ZCS-20697 Added zimbraMobileAppFCMServerJson, zimbraMobileAppFCMDeviceJson and relevant callback by ashishkhms · Pull Request #1894 · Zimbra/zm-mailbox

ashishkhms · 2026-09-10T05:46:54Z

Added zimbraMobileAppFCMServerJson, zimbraMobileAppFCMDeviceJson LDAP attributes and relevant callback

ZCS-20697 Changed the description and names of LDAP attributes.

…eJson and relevant callback ZCS-20697 Changed the description and names of LDAP attributes

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

checkstyle

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck> _{reported by reviewdog 🐶}
Line exceeds 120 characters in comments

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37809

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 120 characters (found 136).

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37809

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37810

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37810

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37827

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37827

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck> _{reported by reviewdog 🐶}
Line exceeds 120 characters in comments

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37828

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 120 characters (found 126).

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37828

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37828

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37828

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> _{reported by reviewdog 🐶}
'if' construct must use '{}'s.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37829

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37829

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37844

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37844

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37846

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37846

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37862

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37862

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37863

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37863

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> _{reported by reviewdog 🐶}
'if' construct must use '{}'s.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37864

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrConfig.java#L37864

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19632

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19632

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19648

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19648

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck> _{reported by reviewdog 🐶}
Line exceeds 120 characters in comments

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19649

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 120 characters (found 136).

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19649

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19650

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19650

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19667

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19667

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck> _{reported by reviewdog 🐶}
Line exceeds 120 characters in comments

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19668

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 120 characters (found 126).

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19668

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19668

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19668

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> _{reported by reviewdog 🐶}
'if' construct must use '{}'s.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19669

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19669

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19684

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19684

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19686

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19686

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19702

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19702

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19703

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19703

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> _{reported by reviewdog 🐶}
'if' construct must use '{}'s.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19704

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19704

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19719

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19719

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19735

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19735

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck> _{reported by reviewdog 🐶}
Line exceeds 120 characters in comments

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19736

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 120 characters (found 136).

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19736

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19737

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19737

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19754

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19754

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck> _{reported by reviewdog 🐶}
Line exceeds 120 characters in comments

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19755

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 120 characters (found 126).

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19755

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19755

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19755

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> _{reported by reviewdog 🐶}
'if' construct must use '{}'s.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19756

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19756

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19771

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19771

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19773

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19773

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19789

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19789

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19790

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19790

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> _{reported by reviewdog 🐶}
'if' construct must use '{}'s.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19791

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

https://github.com/Zimbra/zm-mailbox/blob/a66cd9fa990fdb0e7a2ac6d42aafd4276b56e272/store/src/java/com/zimbra/cs/account/ZAttrDomain.java#L19791

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

zm-mailbox/common/src/java/com/zimbra/common/account/ZAttrProvisioning.java

Line 10927 in a66cd9f

@ZAttr(id=4171)

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

zm-mailbox/common/src/java/com/zimbra/common/account/ZAttrProvisioning.java

Line 10927 in a66cd9f

@ZAttr(id=4171)

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.ConstantNameCheck> _{reported by reviewdog 🐶}
Name 'A_zimbraMobileAppFCMDeviceDetails' must match pattern '^[A-Z][A-Z0-9](_[A-Z0-9]+)$'.

zm-mailbox/common/src/java/com/zimbra/common/account/ZAttrProvisioning.java

Line 10928 in a66cd9f

    
           public static final String A_zimbraMobileAppFCMDeviceDetails = "zimbraMobileAppFCMDeviceDetails";

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

zm-mailbox/common/src/java/com/zimbra/common/account/ZAttrProvisioning.java

Line 10938 in a66cd9f

@ZAttr(id=4170)

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

zm-mailbox/common/src/java/com/zimbra/common/account/ZAttrProvisioning.java

Line 10938 in a66cd9f

@ZAttr(id=4170)

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.ConstantNameCheck> _{reported by reviewdog 🐶}
Name 'A_zimbraMobileAppFCMServerDetails' must match pattern '^[A-Z][A-Z0-9](_[A-Z0-9]+)$'.

zm-mailbox/common/src/java/com/zimbra/common/account/ZAttrProvisioning.java

Line 10939 in a66cd9f

    
           public static final String A_zimbraMobileAppFCMServerDetails = "zimbraMobileAppFCMServerDetails";

github-actions · 2026-09-10T05:48:19Z

+     *
+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4171)


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

github-actions · 2026-09-10T05:48:19Z

+     *
+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4171)


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

github-actions · 2026-09-10T05:48:19Z

+     *
+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4171)


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

github-actions · 2026-09-10T05:48:19Z

+     *
+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4171)


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

github-actions · 2026-09-10T05:48:19Z

+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4171)
+    public void setMobileAppFCMDeviceDetails(String zimbraMobileAppFCMDeviceDetails) throws com.zimbra.common.service.ServiceException {


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck> _{reported by reviewdog 🐶}
Line exceeds 120 characters in comments

github-actions · 2026-09-10T05:48:20Z

+     */
+    @ZAttr(id=4171)
+    public Map<String,Object> unsetMobileAppFCMDeviceDetails(Map<String,Object> attrs) {
+        if (attrs == null) attrs = new HashMap<String,Object>();


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck> _{reported by reviewdog 🐶}
',' is not followed by whitespace.

github-actions · 2026-09-10T05:48:20Z

+     *
+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4170)


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

github-actions · 2026-09-10T05:48:20Z

+     *
+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4170)


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

github-actions · 2026-09-10T05:48:21Z

+     *
+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4170)


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not followed by whitespace.

github-actions · 2026-09-10T05:48:21Z

+     *
+     * @since ZCS 10.1.22
+     */
+    @ZAttr(id=4170)


🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck> _{reported by reviewdog 🐶}
'=' is not preceded with whitespace.

ZCS-20697 Added zimbraMobileAppFCMServerJson, zimbraMobileAppFCMDevic…

a66cd9f

…eJson and relevant callback ZCS-20697 Changed the description and names of LDAP attributes

ashishkhms requested review from Gopal-Moolchandani, GuruanthDeshmukh, aakashsingh2000, abhi-zimbra, ashwinsahu19, bharat-kumar-c, chitranshu198, dasiyogesh, desouzas, elavarasan-synacor, ganeshVaradi18, jiteshsojitra, k-kato, omkar-oak-01 and preshitaagrawal September 10, 2026 05:46

github-actions Bot reviewed Sep 10, 2026

View reviewed changes

GuruanthDeshmukh mentioned this pull request Sep 10, 2026

ZCS-20759 : Addition of LDAP Attributes #1895

Open

Navigation Menu

Uh oh!

ZCS-20697 Added zimbraMobileAppFCMServerJson, zimbraMobileAppFCMDeviceJson and relevant callback - #1894

ZCS-20697 Added zimbraMobileAppFCMServerJson, zimbraMobileAppFCMDeviceJson and relevant callback#1894
ashishkhms wants to merge 1 commit into
developfrom
ZCS-20697

ashishkhms commented Sep 10, 2026

Uh oh!

github-actions Bot left a comment

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

github-actions Bot Sep 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

ashishkhms commented Sep 10, 2026

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 10, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant