[tomee] branch master updated: Revert a small change to make sure we don't break TCK

[jl...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jlmonteiro pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomee.git


The following commit(s) were added to refs/heads/master by this push:
     new efc4e12  Revert a small change to make sure we don't break TCK
efc4e12 is described below

commit efc4e122300a9df1d29a1d77c4cf0e658ccfb3c2
Author: Jean-Louis Monteiro <jl...@tomitribe.com>
AuthorDate: Thu Apr 22 00:26:33 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 =