fix bug that "eval" hangs in named location #1

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@drmingdrmer

I've tested with nginx-1.2.0 there still be such a bug in it.

I just made a fix to make it work but I am not sure if it is a best way to do this.

@agentzh
Member
agentzh commented May 2, 2012

I believe this is a bug in the nginx core. And ngx_eval is not the only one that is affected. The following patch for the nginx core should fix this:

https://github.com/agentzh/ngx_openresty/blob/master/patches/nginx-1.0.15-reset_wev_handler_in_named_locations.patch

I'm reluctant to work-around this issue in ngx_eval because it just "hides" the real issue rather than healthily expose it :)

@agentzh agentzh added a commit that referenced this pull request May 2, 2012
@agentzh agentzh added a test for using ngx_eval in named locations. this exposes an i…
…ssue in the nginx core, reported by drdrxp in github issue #1.
4eb2a02
@drmingdrmer

hmm.. seem like this is the right way.

@drmingdrmer drmingdrmer closed this May 2, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment