pat-lego commented on code in PR #16:
URL:
https://github.com/apache/sling-org-apache-sling-auth-core/pull/16#discussion_r1722279016
##########
src/main/java/org/apache/sling/auth/core/impl/SlingAuthenticator.java:
##########
@@ -406,6 +406,9 @@ public static String getHttpAuth(final Config config) {
@Override
public boolean handleSecurity(HttpServletRequest request,
HttpServletResponse response) {
+ if(request.getHeader("x-request-id") != null) {
Review Comment:
The request path and method would still persist as we are not overriding the
function call made here
https://github.com/apache/sling-org-apache-sling-engine/blob/master/src/main/java/org/apache/sling/engine/impl/SlingMainServlet.java#L107
but instead we are simply adding the ability to trace the authentication logs
to the edge logs allowing for visibility into the authentication process.
--
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]
