You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by jl...@apache.org on 2021/04/21 22:25:53 UTC
[tomee] 01/01: Revert a small change to make sure we don't break TCK
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 <jl...@tomitribe.com>
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 =