Mailing-List: contact dev-help@sling.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@sling.apache.org
Date: Wed, 5 Feb 2014 08:54:10 +0000 (UTC)
From: "Felix Meschberger (JIRA)" <jira@apache.org>
To: dev@sling.apache.org
Message-ID: <JIRA.12693268.1391559975887.40711.1391590450416@arcas>
In-Reply-To: <JIRA.12693268.1391559975887@arcas>
References: <JIRA.12693268.1391559975887@arcas>
Subject: [jira] [Commented] (SLING-3374) Overlay sendRedirect to get
 relative Location header
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/SLING-3374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13891929#comment-13891929 ] 

Felix Meschberger commented on SLING-3374:
------------------------------------------

Technically we can (and in an internal discussion I may actually have stipulated that we) implement the sendRedirect method in Sling to return a relative path.

There are two problems, though with respect to specification compliance:

# The Servlet API spec requires the [sendRedirect|http://docs.oracle.com/javaee/6/api/javax/servlet/http/HttpServletResponse.html#sendRedirect%28java.lang.String%29] method to set an absolute URL on the Location header.
# The current HTTP/1.1 spec (RFC 2616) mandates the [Location|http://tools.ietf.org/html/rfc2616#section-14.30] header to be an absolute URI (which is considered a bug to be fixed in the upcoming HTTP spec: http://www.greenbytes.de/tech/webdav/draft-ietf-httpbis-p2-semantics-25.html#header.location).

Given that the HTTP spec is being updated and most if not all modern browsers today support relative URI Location headers, it would be conceivable to wrap the sendRedirect method to always use a relative URL instead of forcing an absolute one.

But then: what would be the benefit of having a relative URL as opposed to an absolute URL ? Could it be that the HttpServletRequest.getServerName and HttpServletRequest.getServerPort yield wrong (or unexpected) values behind a caching proxy ?

> Overlay sendRedirect to get relative Location header
> ----------------------------------------------------
>
>                 Key: SLING-3374
>                 URL: https://issues.apache.org/jira/browse/SLING-3374
>             Project: Sling
>          Issue Type: Bug
>            Reporter: Laurie byrum
>
> Recently, Location headers returned from redirects started being absolute instead of relative. 
> We should try to decorate this in sling.
> Can we replace the sendRedirect with a set-header set-status combination
> in sling?


--
This message was sent by Atlassian JIRA
(v6.1.5#6160)