Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

bugfix: merge location configration #14

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

tomplus commented Aug 29, 2012

Function merge_loc_conf doesn't merge parameters 'queries' and 'literal_query', so I sometimes get error 'no redis2 query specified or the query is empty'.

For example:

    location /redis {
            set $key 'default';
            if ($uri ~ "(special.*)") {
                    set $key $1;
            }
            redis2_query lrange $key 0 -1;
            redis2_pass 127.0.0.1:6379;
    }

You will get HTTP/500 if you send query /redis/special_001. For query '/redis/test' everything work fine.

agentzh added a commit that referenced this pull request Aug 29, 2012

bugfix: directives redis2_query, redis2_literal_raw_query, and redis2…
…_raw_queries could not be inherited automatically by the "location if" blocks, resulting in the "no redis2 query specified or the query is empty" errors. thanks Tomasz Prus for the patch in github pull #14.
Owner

agentzh commented Aug 29, 2012

already merged your patch. thanks for your contribution!

@agentzh agentzh closed this Aug 29, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment