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:52 UTC
[tomee] branch tomee-security_end created (now 52463e2)
This is an automated email from the ASF dual-hosted git repository.
jlmonteiro pushed a change to branch tomee-security_end
in repository https://gitbox.apache.org/repos/asf/tomee.git.
at 52463e2 Revert a small change to make sure we don't break TCK
This branch includes the following new commits:
new 52463e2 Revert a small change to make sure we don't break TCK
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
[tomee] 01/01: Revert a small change to make sure we don't break TCK
Posted by jl...@apache.org.
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 =