Mercurial > hg > nginx

changeset 1786:adca43955f79

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
return NGX_DECLINED if access directives are not active, this fixes case when satisfy any does not test active directives at all
author Igor Sysoev <igor@sysoev.ru>
date Thu, 27 Dec 2007 14:15:34 +0000
parents 0673b54f34f7
children a37fe5ceacc4
files src/http/modules/ngx_http_access_module.c src/http/modules/ngx_http_auth_basic_module.c
diffstat 2 files changed, 3 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/src/http/modules/ngx_http_access_module.c
+++ b/src/http/modules/ngx_http_access_module.c
@@ -98,7 +98,7 @@ ngx_http_access_handler(ngx_http_request
     alcf = ngx_http_get_module_loc_conf(r, ngx_http_access_module);
 
     if (alcf->rules == NULL) {
-        return NGX_OK;
+        return NGX_DECLINED;
     }
 
     /* AF_INET only */
@@ -128,7 +128,7 @@ ngx_http_access_handler(ngx_http_request
         }
     }
 
-    return NGX_OK;
+    return NGX_DECLINED;
 }
 
 
--- a/src/http/modules/ngx_http_auth_basic_module.c
+++ b/src/http/modules/ngx_http_auth_basic_module.c
@@ -113,7 +113,7 @@ ngx_http_auth_basic_handler(ngx_http_req
     alcf = ngx_http_get_module_loc_conf(r, ngx_http_auth_basic_module);
 
     if (alcf->realm.len == 0 || alcf->user_file.len == 0) {
-        return NGX_OK;
+        return NGX_DECLINED;
     }
 
     ctx = ngx_http_get_module_ctx(r, ngx_http_auth_basic_module);