Here is the quick start: https://github.com/solomax/ajax-download
On Sat, Oct 7, 2017 at 12:40 PM, Maxim Solodovnik <solomax...@gmail.com> wrote: > 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 -- WBR Maxim aka solomax
