Re: 请教一个nginx问题

agentzh · 2012-06-18T19:51:04+00:00

Hello! 2012/6/18 agentzh <age...@gmail.com>: > 2012/6/18 Mr.SlUmP <marc...@qq.com>: >> 看到agentzh的模块&...

Re: 请教一个nginx问题

agentzh

Hello!

2012/6/18 level077 <lev...@163.com>:
> 根据你这篇文章http://agentzh.blogspot.com/2011/03/how-nginx-location-if-works.html，;
> 做了测试，以下配置
>               location /proxy {
>                         set $a 1;
>                         if ($a = 1) {
>                                 set $a 2;

这里需要加上 break 语句。

>                         }
>                         set $a 3;
>                         proxy_pass http://127.0.0.1:8080/$a.htm ;
>                 }
> 如果proxy_pass中不使用变量，如写成http://127.0.0.1:8080/2.htm,则请求会返回404.
>
> 但是如果配置直接写成这样：
>              location /proxy {
>                   proxy_pass http://127.0.0.1:8080/2.htm ;
>              }
> 请求是没有问题的
> 不知这原因是什么？

你的配置有一个问题，即 if 指令的条件匹配之后，其 if 配置块内部如果不使用 break 指令，仍然会继续执行 if 外面接下来的
set $a 3 等其他 ngx_rewrite 的指令。

我的文中指的"if 配置块进去就出不来"其实并不包括当前正在运行中的 ngx_rewrite 模块的指令序列 :)

Regards,
-agentzh

P.S. 抄送给 openresty 邮件列表：https://groups.google.com/group/openresty
也欢迎你加入那里讨论 nginx 或 openresty 相关的问题 :)

level077

大神回复好快，激动:)

不好意思，我文中没表述清楚。

对于以下两个配置：

location /proxy {

set $a 1;

if ($a=1) {

set $a 2;

break;

}

set $a 3;

proxy_pass http://127.0.0.1:8080/$a.htm;

}

location /proxy {

set $a 1;

if ($a=1) {

set $a 2;

break;

}

set $a 3;

proxy_pass http://127.0.0.1:8080/2.htm;

}

这两个配置就proxy_pass处不一样，但对于第一个配置，请求/proxy返回200，第二个则是404，这是什么原因？

在http://wiki.nginx.org/IfIsEvil中有如下例子：

# request will be sent to backend without uri changed
        # to '/' due to if
 
        location /proxy-pass-uri {
            proxy_pass http://127.0.0.1:8080/;
 
            set $true 1;
 
            if ($true) {
                # nothing
            }
        }

感觉跟我说的问题是同一种情况。


在 2012-06-18 19:51:03，agentzh <age...@gmail.com> 写道：
>Hello!
>
>2012/6/18 level077 <lev...@163.com>:
>> 根据你这篇文章http://agentzh.blogspot.com/2011/03/how-nginx-location-if-works.html，
>> 做了测试，以下配置
>>               location /proxy {
>>                         set $a 1;
>>                         if ($a = 1) {
>>                                 set $a 2;
>
>这里需要加上 break 语句。
>
>>                         }
>>                         set $a 3;
>>                         proxy_pass http://127.0.0.1:8080/$a.htm ;
>>                 }
>> 如果proxy_pass中不使用变量，如写成http://127.0.0.1:8080/2.htm,则请求会返回404.
>>
>> 但是如果配置直接写成这样：
>>              location /proxy {
>>                   proxy_pass http://127.0.0.1:8080/2.htm ;
>>              }
>> 请求是没有问题的
>> 不知这原因是什么？
>
>你的配置有一个问题，即 if 指令的条件匹配之后，其 if 配置块内部如果不使用 break 指令，仍然会继续执行 if 外面接下来的
>set $a 3 等其他 ngx_rewrite 的指令。
>
>我的文中指的"if 配置块进去就出不来"其实并不包括当前正在运行中的 ngx_rewrite 模块的指令序列 :)
>
>Regards,
>-agentzh
>
>P.S. 抄送给 openresty 邮件列表：https://groups.google.com/group/openresty
>也欢迎你加入那里讨论 nginx 或 openresty 相关的问题 :)

agentzh

Hello!

2012/6/18 level077 <lev...@163.com>:
>
> location /proxy {
> set $a 1;
> if ($a=1) {
> set $a 2;
> break;
> }
> set $a 3;
> proxy_pass http://127.0.0.1:8080/2.htm;
> }
>
> 这两个配置就proxy_pass处不一样，但对于第一个配置，请求/proxy返回200，第二个则是404，这是什么原因？
>

第二个例子不能如期工作的原因是因为 ngx_proxy 模块中有一个小问题，让 if 配置块在此种情况下未能恰当继承 location
/proxy 配置块中的 ngx_proxy 的配置（在源码级别上即是没有继承 plcf->location 这个字符串，在上例中便是
"/proxy"），于是导致 ngx_proxy 生成的请求 URI 是 /2.html/proxy，而不是期望中的 /2.html.

解决的办法是给标准的 ngx_proxy 模块打一个小补丁，这样你的第二个例子就可以如期工作了（见邮件的末尾或附件）。

这个补丁生成自标准 nginx 1.0.15，但也可以应用于 nginx 1.2.1.

如果你不想打补丁的话，也可以通过在 proxy_pass 参数串中引入 nginx 变量的办法绕过。比如你第一个例子那样），或者更一般地：

    proxy_pass http://127.0.0.1:$server_port/2.htm;

Best regards,
-agentzh

--- nginx-1.0.15/src/http/modules/ngx_http_proxy_module.c	2012-03-15
19:41:43.000000000 +0800
+++ nginx-1.0.15-patched/src/http/modules/ngx_http_proxy_module.c	2012-06-18
22:27:32.715568165 +0800
@@ -2046,7 +2046,7 @@

     if (conf->upstream.upstream || conf->proxy_lengths) {
         clcf = ngx_http_conf_get_module_loc_conf(cf, ngx_http_core_module);
-        if (clcf->handler == NULL && clcf->lmt_excpt) {
+        if (clcf->handler == NULL) {
             clcf->handler = ngx_http_proxy_handler;
             conf->location = prev->location;
         }

Attachment: nginx-1.0.15-fix_proxy_pass_in_location_if.patch
Description: Binary data