exceptionfactory commented on code in PR #8906:
URL: https://github.com/apache/nifi/pull/8906#discussion_r1622948663
##########
nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/AccessResource.java:
##########
@@ -412,7 +412,7 @@ private LogoutRequest completeLogoutRequest(final
HttpServletResponse httpServle
}
private String getNiFiLogoutCompleteUri() {
- return getNiFiUri() + "logout-complete";
+ return getNiFiUri() + "#/logout-complete";
Review Comment:
This will be a breaking change for existing deployments given that identity
providers require registration of logout URLs. What do you think about
considering a redirect from `/nifi/logout-complete` in the Jetty Server
configuration?
--
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]
