bug report: SSI -> signal 10 (nginx 0.7.10, Solaris 10)

[Igor Sysoev]

On Thu, Nov 06, 2008 at 02:17:43PM +0300, Maxim Dounin wrote:

> Hello!
> 
> On Thu, Nov 06, 2008 at 01:51:22PM +0300, Igor Sysoev wrote:
> 
> > On Thu, Nov 06, 2008 at 01:50:22PM +0300, Maxim Dounin wrote:
> > 
> > > Hello!
> > > 
> > > On Thu, Nov 06, 2008 at 08:01:58AM +0100, Hannes Haug wrote:
> > > 
> > > > Yes Maxim,
> > > > 
> > > > your patch fixes the problem.
> > > 
> > > Thanks for testing.
> > > 
> > > > But there remains an issue:
> > > > Errors on subrequests (e.g. 404) result in the inclusion of the error
> > > > pages in the main page.
> > > > I.e. in the body of the main page there starts a new html document
> > > > with the error message.
> > > 
> > > It's by design.  Subrequests are just included into main page as 
> > > is, regardless of the error code.  If you don't want error replies 
> > > to be included you have to write in your config something like 
> > > this:
> > > 
> > >     location /include/ {
> > >         error_page  404  /empty;
> > 
> > -         error_page  404  /empty;
> > +         error_page  404  = /empty;
> 
> It does it's job in both cases, actually.  From semantic point of 
> view '=' may be better, but it shouldn't influence real work 
> anyway.  Or I missed something?

You are right.

> BTW, did you picked alignment patch?

Yes, thank you.


-- 
Igor Sysoev
http://sysoev.ru/en/