Tapestry itself does not contribute any configuration to the RequestExceptionHandler service.
One of the project that contributes a configuration is tapestry-security "configuration.add(ShiroException.class, SecurityExceptionHandlerAssistant.class);" So if the exceptions you are testing are not of type ShiroException and you didnt do any contribution to RequestExceptionHandler then it will propagate to renderException What ever the case is it would be nice to create an issue to JIRA regarding the absoluteUrl because event if you didn't hit that part of code its still an issue that has to be fixed Thanks Dimitris Zenios On Mon, Apr 13, 2015 at 3:25 AM, George Christman <gchrist...@cardaddy.com> wrote: > Thanks Kalle and yes I did try overriding the service with my own > implementation, however I must have been missing something some wheres > because the configuration always appeared to be empty triggering a > renderException. > > On Fri, Apr 10, 2015 at 7:14 PM, Kalle Korhonen < > kalle.o.korho...@gmail.com> > wrote: > > > On Fri, Apr 10, 2015 at 5:56 AM, George Christman < > gchrist...@cardaddy.com > > > > > wrote: > > > > > I'm using relative links throughout my application and allowing our > > reverse > > > proxy to handle our secure port forwarding. > > > > > > I just discovered that DefaultRequestExceptionHandler on line 210 is > > > creating an absolute link for all xhr request. > > > > > > > > > > > > https://github.com/apache/tapestry-5/blob/840533975a33d9094789bc09e40557a5c3ec7ba8/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/DefaultRequestExceptionHandler.java > > > > > > The problem I face with this is we need to have secure links and it's > > > creating an non secure absolute link. Is there a way to override this > > > behavior so that I don't need to depend on our reverse proxy to forward > > the > > > request? > > > > > > > Obviously the DefaultRequestExceptionHandler can be replaced like the > name > > implies. But yes, please open an issue and I'll fix it. > > > > @Dimitris, yes your analysis is completely correct, should have gone > > through it more carefully when I merged it in. > > > > Kalle > > > > > > -- > George Christman > CEO > www.CarDaddy.com > P.O. Box 735 > Johnstown, New York >
