cshannon opened a new pull request, #1879: URL: https://github.com/apache/activemq/pull/1879
This is an update to #1874 which fixes the previous issue with wildcard principal handling which caused things to break This method was iterating over a set to find matching entries instead of just using contains(). This update switches to using contains() which will be faster and also cleans up the code with a for each loop. The wildcard principal handling was simplified inside DefaultAuthorizationMap by just using a predefined static object to mark that everything should be allowed if the name is "*" A small test was added but there are several other authorization tests that already exist that also exercise this method. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information, visit: https://activemq.apache.org/contact
