<div dir="ltr">Thanks! Could you please explain why this is done?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 14, 2014 at 4:41 PM, Maxim Dounin <span dir="ltr"><<a href="mailto:mdounin@mdounin.ru" target="_blank">mdounin@mdounin.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello!<br>
<div><div class="h5"><br>
On Tue, Jan 14, 2014 at 04:15:32PM +0530, Fasih wrote:<br>
<br>
> Hi<br>
><br>
> I have a custom plugin that handles rewrite (NGX_HTTP_REWRITE_PHASE). There<br>
> is another plugin compiled before my plugin that also handles rewrite<br>
> (HttpLuaModule). I was expecting to see that my module would rewrite after<br>
> lua is done, however that is not the case. Some debugging showed that<br>
> whereas my module pushed into the<br>
> cmcf->phases[NGX_HTTP_REWRITE_PHASE].handlers after lua, the<br>
> cmcf.phase_engine.handlers had lua *after* my module. The culprit seems to<br>
> be the following:<br>
><br>
> static ngx_int_t<br>
> ngx_http_init_phase_handlers(ngx_conf_t *cf, ngx_http_core_main_conf_t<br>
> *cmcf)<br>
> {<br>
> ..<br>
>         ph = cmcf->phase_engine.handlers;<br>
> ..<br>
>         n += cmcf->phases[i].handlers.nelts;<br>
><br>
>         for (j = cmcf->phases[i].handlers.nelts - 1; j >=0; j--) {<br>
>             ph->checker = checker;<br>
>             ph->handler = h[j];<br>
>             ph->next = n;<br>
>             ph++;<br>
>         }<br>
> }<br>
><br>
> The order is inverted here (h[j] before h[j-1]). Is this intentional or a<br>
> bug?<br>
<br>
</div></div>It's intentional.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Maxim Dounin<br>
<a href="http://nginx.org/" target="_blank">http://nginx.org/</a><br>
<br>
_______________________________________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" target="_blank">http://mailman.nginx.org/mailman/listinfo/nginx-devel</a><br>
</font></span></blockquote></div><br></div>