thenatog commented on a change in pull request #4988:
URL: https://github.com/apache/nifi/pull/4988#discussion_r611944222
##########
File path:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/AccessResource.java
##########
@@ -868,6 +873,10 @@ public void oidcLogout(@Context HttpServletRequest
httpServletRequest, @Context
throw new
IllegalStateException(OPEN_ID_CONNECT_SUPPORT_IS_NOT_CONFIGURED_MSG);
}
+ final String mappedUserIdentity = NiFiUserUtils.getNiFiUserIdentity();
+ removeCookie(httpServletResponse,
JwtAuthenticationFilter.JWT_COOKIE_NAME);
+ logger.info("Successfully invalidated JWT for " + mappedUserIdentity);
Review comment:
I had put this log message here to be a bit clearer about logging out a
JWT user, as at one point this logout was not happening as expected. I've
updated it to be a debug message using a placeholder but with a slightly
different message to the one you suggested as there's still some other steps
for logout after this message depending on the auth mechanism.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
