[Groonga-commit] groonga/groonga at 60fd5fc [master] groonga-httpd: fix wrong HTTP status is set on success

Back to archive index

Kouhei Sutou null+****@clear*****
Fri Aug 19 16:02:31 JST 2016


Kouhei Sutou	2016-08-19 16:02:31 +0900 (Fri, 19 Aug 2016)

  New Revision: 60fd5fcf1595e02908d977013aefa34c3f07fff2
  https://github.com/groonga/groonga/commit/60fd5fcf1595e02908d977013aefa34c3f07fff2

  Message:
    groonga-httpd: fix wrong HTTP status is set on success

  Modified files:
    src/httpd/nginx-module/ngx_http_groonga_module.c

  Modified: src/httpd/nginx-module/ngx_http_groonga_module.c (+3 -3)
===================================================================
--- src/httpd/nginx-module/ngx_http_groonga_module.c    2016-08-19 15:47:15 +0900 (81511e9)
+++ src/httpd/nginx-module/ngx_http_groonga_module.c    2016-08-19 16:02:31 +0900 (532ac03)
@@ -134,11 +134,11 @@ ngx_http_groonga_get_thread_limit(void *data)
 }
 
 static ngx_int_t
-ngx_http_groonga_grn_rc_to_ngx_rc(grn_rc rc)
+ngx_http_groonga_grn_rc_to_http_status(grn_rc rc)
 {
   switch (rc) {
   case GRN_SUCCESS :
-    return NGX_OK;
+    return NGX_HTTP_OK;
   case GRN_INVALID_ARGUMENT :
   case GRN_FUNCTION_NOT_IMPLEMENTED :
   case GRN_SYNTAX_ERROR :
@@ -945,7 +945,7 @@ ngx_http_groonga_handler_send_response(ngx_http_request_t *r,
   output_chain = ngx_http_groonga_attach_chain(output_chain, &foot_chain);
 
   /* set the status line */
-  r->headers_out.status = ngx_http_groonga_grn_rc_to_ngx_rc(data->rc);
+  r->headers_out.status = ngx_http_groonga_grn_rc_to_http_status(data->rc);
   r->headers_out.content_length_n = GRN_TEXT_LEN(&(data->typed.head)) +
                                     GRN_TEXT_LEN(&(data->typed.body)) +
                                     GRN_TEXT_LEN(&(data->typed.foot));
-------------- next part --------------
HTML����������������������������...
Download 



More information about the Groonga-commit mailing list
Back to archive index