This is an automated email from the ASF dual-hosted git repository. jlmonteiro pushed a commit to branch tomee-security_end in repository https://gitbox.apache.org/repos/asf/tomee.git
commit 52463e26b682cba9f871bc555693918e8482d210 Author: Jean-Louis Monteiro <[email protected]> AuthorDate: Thu Apr 22 00:25:25 2021 +0200 Revert a small change to make sure we don't break TCK --- .../apache/tomee/security/identitystore/TomEEIdentityStoreHandler.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tomee/tomee-security/src/main/java/org/apache/tomee/security/identitystore/TomEEIdentityStoreHandler.java b/tomee/tomee-security/src/main/java/org/apache/tomee/security/identitystore/TomEEIdentityStoreHandler.java index 122b277..b0b7af7 100644 --- a/tomee/tomee-security/src/main/java/org/apache/tomee/security/identitystore/TomEEIdentityStoreHandler.java +++ b/tomee/tomee-security/src/main/java/org/apache/tomee/security/identitystore/TomEEIdentityStoreHandler.java @@ -105,6 +105,7 @@ public class TomEEIdentityStoreHandler implements IdentityStoreHandler { // Ask all stores that were configured for group providing only to get the groups for the // authenticated caller + /* Comment out because it seems to be adding more roles and breaking TCK final CredentialValidationResult finalResult = validationResult; // compiler didn't like validationResult in the enclosed scope AccessController.doPrivileged(new PrivilegedAction<Void>() { public Void run() { @@ -114,6 +115,7 @@ public class TomEEIdentityStoreHandler implements IdentityStoreHandler { return null; } }); + */ final CredentialValidationResult authorizedValidationResult = validationResult; final Set<String> additionalGroups =
