[GitHub] [activemq-artemis] jbertram opened a new pull request, #4209: ARTEMIS-3984 LegacyLDAPSecuritySettingPlugin shouldn't modify existin…

[GitBox <gi...@apache.org>]

jbertram opened a new pull request, #4209:
URL: https://github.com/apache/activemq-artemis/pull/4209

   …g matches
   
   When the LegacyLDAPSecuritySettingPlugin has enableListener set to true and a new permission is added it will try to modify the existing match if one exists. This is problematic if there's a more generic wildcard match than the specific one that's modified.
   
   This commit fixes that problem so that instead of modifying the existing match(es) it simply adds a new one. The plugin never should have tried modifying the existing match in the first place as two identical matches would be a configuration error.


-- 
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: gitbox-unsubscribe@activemq.apache.org

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

[GitHub] [activemq-artemis] clebertsuconic merged pull request #4209: ARTEMIS-3984 LegacyLDAPSecuritySettingPlugin shouldn't modify existin…

[GitBox <gi...@apache.org>]

clebertsuconic merged PR #4209:
URL: https://github.com/apache/activemq-artemis/pull/4209


-- 
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: gitbox-unsubscribe@activemq.apache.org

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