smolnar82 commented on code in PR #1400:
URL: https://github.com/apache/knox/pull/1400#discussion_r4023728918
##########
gateway-service-knoxtoken/src/main/java/org/apache/knox/gateway/service/knoxtoken/TokenResource.java:
##########
@@ -945,6 +972,11 @@ protected TokenResponseContext
getTokenResponse(UserContext context) {
} else {
response = new TokenResponseContext(null, null,
Response.serverError());
}
+ } catch (ActorChainDepthExceededException e) {
+ log.rejectedTokenExchange(e.getMessage());
Review Comment:
Fixed.
##########
gateway-service-knoxtoken/src/main/java/org/apache/knox/gateway/service/knoxtoken/TokenResource.java:
##########
@@ -155,6 +155,8 @@ public class TokenResource {
private static final String KNOX_TOKEN_HARDCODED_CLAIM_MAPPINGS =
TOKEN_PARAM_PREFIX + "hardcoded.claim.mappings";
private static final String METADATA_QUERY_PARAM_PREFIX = "md_";
private static final String TOKEN_ENABLE_DELEGATED_AUTH = TOKEN_PARAM_PREFIX
+ "enable.delegated.auth";
+ static final String DELEGATION_MAX_ACTOR_CHAIN_DEPTH =
"delegation.max.actor.chain.depth";
+ static final int DELEGATION_MAX_ACTOR_CHAIN_DEPTH_DEFAULT = 10;
Review Comment:
Fixed.
--
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]
