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

[Maxim Dounin]

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?

BTW, did you picked alignment patch?

Maxim Dounin