Thanks for the clarifications Martin, According to [1] 'Ajax-Location' is being set instead of 'Location' (due to redirect happens on ajax button click) This works as expected in Chrome but fails in FF :(((
Will continue debugging and will try to create quickstart [1] https://github.com/apache/wicket/blob/master/wicket-core/src/main/java/org/apache/wicket/protocol/http/servlet/ServletWebResponse.java#L273 On Fri, Oct 6, 2017 at 9:59 PM, Martin Grigorov <mgrigo...@apache.org> wrote: > Maybe I was not clear enough. > HttpServletResponse#sendRedirect() is supposed to set the Location header, > i.e. Tomcat/Jetty. > > The other status codes do it manually because #sendRedirect() would > override the status code. > > IMO you should ask at Tomcat forums why the Location is not being set. > > Martin Grigorov > Wicket Training and Consulting > https://twitter.com/mtgrigorov > > On Fri, Oct 6, 2017 at 4:10 PM, Maxim Solodovnik <solomax...@gmail.com> > wrote: > >> This code: >> https://github.com/apache/wicket/blob/master/wicket- >> request/src/main/java/org/apache/wicket/request/http/handler/ >> RedirectRequestHandler.java >> >> Has special handling of moved temporary >> >> WBR, Maxim >> (from mobile, sorry for the typos) >> >> On Oct 6, 2017 20:05, "Maxim Solodovnik" <solomax...@gmail.com> wrote: >> >> > The status is being set correctly, Location is not being set :( >> > >> > WBR, Maxim >> > (from mobile, sorry for the typos) >> > >> > On Oct 6, 2017 20:04, "Martin Grigorov" <mgrigo...@apache.org> wrote: >> > >> >> Hi, >> >> >> >> Isn't this something that the web container should do for us ? >> >> >> >> javax.servlet.http.HttpServletResponse#sendRedirect() sets the status >> to >> >> 302, that's the reason why the others do it "manually". >> >> >> >> Martin Grigorov >> >> Wicket Training and Consulting >> >> https://twitter.com/mtgrigorov >> >> >> >> On Fri, Oct 6, 2017 at 2:18 PM, Maxim Solodovnik <solomax...@gmail.com> >> >> wrote: >> >> >> >> > Hello All, >> >> > >> >> > Recently I found RedirectToUrlException is not working in FF (works in >> >> > Chrome) >> >> > Redirect URL is being displayed for microsecond, then page is just >> >> > being refreshed >> >> > >> >> > What I found during debugging: >> >> > In case code is SC_MOVED_TEMPORARILY Location header is not set. >> >> > >> >> > FF site [1] clearly states Location should be set >> >> > >> >> > Tested on master branch. >> >> > >> >> > Should I prepare PR or this is done on purpose? >> >> > >> >> > [1] https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/302 >> >> > >> >> > -- >> >> > WBR >> >> > Maxim aka solomax >> >> > >> >> >> > >> -- WBR Maxim aka solomax
