Copilot commented on code in PR #530:
URL: https://github.com/apache/ranger/pull/530#discussion_r2114784335


##########
dev-support/ranger-docker/scripts/ranger-usersync-install.properties:
##########
@@ -215,7 +215,7 @@ SYNC_PAGED_RESULTS_ENABLED=
 # default value: 500
 SYNC_PAGED_RESULTS_SIZE=
 #LDAP context referral could be ignore or follow
-SYNC_LDAP_REFERRAL =ignore
+SYNC_LDAP_REFERRAL=follow

Review Comment:
   [nitpick] Add a `# default value: follow` comment above this line and align 
the spacing around `=` with the surrounding property definitions.



##########
unixauthservice/scripts/install.properties:
##########
@@ -216,7 +216,7 @@ SYNC_PAGED_RESULTS_ENABLED=
 # default value: 500
 SYNC_PAGED_RESULTS_SIZE=
 #LDAP context referral could be ignore or follow
-SYNC_LDAP_REFERRAL =ignore
+SYNC_LDAP_REFERRAL=follow

Review Comment:
   [nitpick] Add a `# default value: follow` comment above this line and adjust 
spacing to match other properties (e.g., `SYNC_LDAP_REFERRAL = follow`).



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@ranger.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to