Undo the last commit and add a potential fix for @issue('11673')

Author: sebastien matrion

grails-web-url-mappings/src/main/groovy/grails/web/mapping/ResponseRedirector.groovy

@@ -94,12 +94,14 @@ class ResponseRedirector {
         boolean permanent = getBooleanArgument(ARGUMENT_PERMANENT, arguments)
         boolean moved = getBooleanArgument(ARGUMENT_MOVED, arguments, true)
 
+
+        boolean absolute = getBooleanArgument(ARGUMENT_ABSOLUTE, arguments, true)
+
         final Map namedParameters = new LinkedHashMap<>(arguments)
         // we generate a relative link with no context path so that the absolute can be calculated by combining the serverBaseURL
         // which includes the contextPath
-        namedParameters.put(LinkGenerator.ATTRIBUTE_CONTEXT_PATH, BLANK)
-
-        boolean absolute = getBooleanArgument(ARGUMENT_ABSOLUTE, arguments, true)
+        // @Issue('11673')
+        if(absolute) namedParameters.put(LinkGenerator.ATTRIBUTE_CONTEXT_PATH, BLANK)
 
         // If the request parameters contain "keepParamsWhenRedirect = true", then we add the original params. The
         // new attribute can be used from UrlMappings to redirect from old URLs to new ones while keeping the params

grails-web-url-mappings/src/test/groovy/grails/web/mapping/RedirectNonAbsoluteURISpec.groovy

@@ -94,6 +94,26 @@ class RedirectNonAbsoluteURISpec extends AbstractUrlMappingsSpec {
         RequestContextHolder.setRequestAttributes(null)
     }
 
+    @Issue('11673')
+    void 'An "absolute=false" redirect includes context-path in Location header'() {
+        given:
+        def linkGenerator = getLinkGeneratorWithContextPath {
+            "/$controller/$action?/$id?"()
+        }
+        def responseRedirector = new ResponseRedirector(linkGenerator)
+        HttpServletRequest request = Mock(HttpServletRequest) { lookup() >> GrailsWebMockUtil.bindMockWebRequest() }
+        HttpServletResponse response = Mock(HttpServletResponse)
+
+        when: 'redirecting with absolute=false where context-path is set'
+        responseRedirector.redirect(request, response, [controller: 'test', action: 'foo', absolute: false])
+
+        then: 'the partial URI includes context-path'
+        1 * response.setStatus(302)
+        1 * response.setHeader(HttpHeaders.LOCATION, CONTEXT_PATH + '/test/foo')
+
+        cleanup:
+        RequestContextHolder.setRequestAttributes(null)
+    }
 
     @Issue('15132')
     void 'An "absolute=false" redirect not added context-path with url params'() {