lmccay commented on code in PR #1011:
URL: https://github.com/apache/knox/pull/1011#discussion_r2021744683
##########
gateway-service-knoxtoken/src/main/java/org/apache/knox/gateway/service/knoxtoken/ClientCredentialsResource.java:
##########
@@ -75,7 +75,7 @@ public Response getAuthenticationToken() {
TokenResponseContext resp = getTokenResponse(context);
if (resp.responseMap != null) {
- String passcode = resp.responseMap.passcode;
+ String passcode = (String) resp.responseMap.map.get(PASSCODE);
Review Comment:
Yeah, this was the source of the initial bug in the implementation. For some
reason the tokenid and the passcode are surfaced at the top level but the
passcode there is just the passcode portion of what ends up being the
client_secret which contains both. I didn't really dig into why there are two
levels but the one inside the map is put together and base64 encoded properly
for use where the other is just the UUID format which can't be used directly.
--
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: dev-unsubscr...@knox.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
