The problem is that the status header is rewritten - sendRedirect()
implicitly sets 302 by the spec. Set the headers yourself and flush to
commit:
servletResponse.setStatus(HttpServletResponse.SC_MOVED_PERMANENTLY);
servletResponse.setHeader("Location",
servletResponse.encodeRedirectURL(requestUri));
servletResponse.flushBuffer();
Kalle
On Tue, Oct 28, 2014 at 11:18 AM, George Christman <gchrist...@cardaddy.com>
wrote:
> I found componentRequestHandler in some tapestry code and when I removed
> it, no page. I guess I just needed to return false ;)
>
> Yup the code is definitely running, I can remove Secure_Page and the page
> redirects without issue, however no matter what I setStatus too, it
> continues to provide a 302 response. If I sendError to a 301, it will
> redirect the browser to a 301 error like it should. Is setStatus the
> correct method to use?
>
> http://localhost:8080/etss/login
>
> GET /etss/login HTTP/1.1
> Host: localhost:8080
> User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101
> Firefox/32.0
> Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
> Accept-Language: en-US,en;q=0.5
> Accept-Encoding: gzip, deflate
> DNT: 1
> Cookie: shiroSavedRequest=/etss/
> Connection: keep-alive
>
> HTTP/1.1 302 Found
> Location: https://localhost/etss/login
> Content-Length: 0
> Server: Jetty(8.1.14.v20131031)
>
>
> On Tue, Oct 28, 2014 at 1:42 PM, Thiago H de Paula Figueiredo <
> thiag...@gmail.com> wrote:
>
> > On Tue, 28 Oct 2014 15:11:13 -0200, George Christman <
> > gchrist...@cardaddy.com> wrote:
> >
> > Hi Thiago,
> >>
> >
> > Hi!
> >
> > I'm assuming you mean something like this, but I'm not seeing
> >> the 301 response. Could you take a look and see if you can spot what I'm
> >> doing wrong. Below is the response as well. Thank you.
> >>
> >
> > Are you sure your code was actually called?
> >
> >
> >>
> >> public class AccessController implements Dispatcher {
> >>
> >> @Override
> >> public boolean dispatch(Request request, final Response response)
> >> throws IOException {
> >> int port = request.getLocalPort();
> >>
> >> PageRenderRequestParameters parameters =
> >> linkEncoder.decodePageRenderRequest(request);
> >>
> >> if (port == 80) {
> >> Link link =
> >> linkSource.createPageRenderLinkWithContext(parameters.
> >> getLogicalPageName(),
> >> parameters.getActivationContext());
> >>
> response.setStatus(HttpServletResponse.SC_MOVED_PERMANENTLY);
> >> link.setSecurity(LinkSecurity.SECURE);
> >> response.sendRedirect(link.toAbsoluteURI());
> >> return true;
> >> }
> >>
> >> componentRequestHandler.handlePageRender(parameters);
> >> return true;
> >>
> >
> > You should return false when a Dispatcher didn't provide a response, so
> > the last line should return false, otherwise no page will ever be
> rendered.
> > I'm not sure why you've added the componentRequestHandler line.
> >
> >
> > --
> > Thiago H. de Paula Figueiredo
> > Tapestry, Java and Hibernate consultant and developer
> > http://machina.com.br
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> > For additional commands, e-mail: users-h...@tapestry.apache.org
> >
> >
>
>
> --
> George Christman
> www.CarDaddy.com
> P.O. Box 735
> Johnstown, New York
>
