Merge branch 'master' of github.com:chaoslawful/drizzle-nginx-module

openresty · Dec 17, 2010 · 397ddcb · 397ddcb
2 parents 3df6542 + f59b991
commit 397ddcb
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/ngx_http_drizzle_handler.c b/src/ngx_http_drizzle_handler.c
@@ -118,7 +118,9 @@ ngx_http_drizzle_handler(ngx_http_request_t *r)
         dlcf->upstream.upstream = ngx_http_upstream_drizzle_add(r, &url);
 
         if (dlcf->upstream.upstream == NULL) {
-            //dd("upstream \"%.*s\" not found", target.len, target.data);
+            ngx_log_error(NGX_LOG_ERR, r->connection->log, 0,
+                   "drizzle: upstream \"%V\" not found", &target);
+
             return NGX_HTTP_INTERNAL_SERVER_ERROR;
         }
     }

diff --git a/test/t/errors.t b/test/t/errors.t
@@ -156,3 +156,20 @@ little-endian systems only
 GET /mysql
 --- error_code: 500
 
+
+
+=== TEST 8: empty pass
+little-endian systems only
+
+--- http_config eval: $::http_config
+--- config
+    location /mysql {
+        set $backend "not-exist";
+        drizzle_pass $backend;
+        drizzle_module_header off;
+        drizzle_query "update cats set name='bob' where name='bob'";
+    }
+--- request
+GET /mysql
+--- error_code: 500
+