From c2188a840bc4153ae92112b04b2e06a90d3944aa Mon Sep 17 00:00:00 2001 From: Paul Martin <paul@paulsputer.com> Date: Wed, 27 Apr 2016 18:58:06 -0400 Subject: [PATCH] Ticket Reference handling #1048 --- src/main/java/com/gitblit/servlet/AccessRestrictionFilter.java | 11 ++++++----- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/src/main/java/com/gitblit/servlet/AccessRestrictionFilter.java b/src/main/java/com/gitblit/servlet/AccessRestrictionFilter.java index bfbc089..e1d76db 100644 --- a/src/main/java/com/gitblit/servlet/AccessRestrictionFilter.java +++ b/src/main/java/com/gitblit/servlet/AccessRestrictionFilter.java @@ -133,10 +133,11 @@ /** * Allows authentication header to be altered based on the action requested * Default is WWW-Authenticate + * @param httpRequest * @param action * @return authentication type header */ - protected String getAuthenticationHeader(String action) { + protected String getAuthenticationHeader(HttpServletRequest httpRequest, String action) { return "WWW-Authenticate"; } @@ -192,7 +193,7 @@ logger.info(MessageFormat.format("ARF: CREATE CHALLENGE {0}", fullUrl)); } - httpResponse.setHeader(getAuthenticationHeader(urlRequestType), CHALLENGE); + httpResponse.setHeader(getAuthenticationHeader(httpRequest, urlRequestType), CHALLENGE); httpResponse.sendError(HttpServletResponse.SC_UNAUTHORIZED); return; } else { @@ -239,7 +240,7 @@ if (runtimeManager.isDebugMode()) { logger.info(MessageFormat.format("ARF: CHALLENGE {0}", fullUrl)); } - httpResponse.setHeader(getAuthenticationHeader(urlRequestType), CHALLENGE); + httpResponse.setHeader(getAuthenticationHeader(httpRequest, urlRequestType), CHALLENGE); httpResponse.sendError(HttpServletResponse.SC_UNAUTHORIZED); return; } else { @@ -248,8 +249,8 @@ // authenticated request permitted. // pass processing to the restricted servlet. newSession(authenticatedRequest, httpResponse); - logger.info(MessageFormat.format("ARF: {0} ({1}) authenticated", fullUrl, - HttpServletResponse.SC_CONTINUE)); + logger.info(MessageFormat.format("ARF: authenticated {0} to {1} ({2})", user.username, + fullUrl, HttpServletResponse.SC_CONTINUE)); chain.doFilter(authenticatedRequest, httpResponse); return; } -- Gitblit v1.9.1