Merge the diff for start_response_length/4
Project: http://git-wip-us.apache.org/repos/asf/couchdb-couch-httpd/repo Commit: http://git-wip-us.apache.org/repos/asf/couchdb-couch-httpd/commit/305e54b7 Tree: http://git-wip-us.apache.org/repos/asf/couchdb-couch-httpd/tree/305e54b7 Diff: http://git-wip-us.apache.org/repos/asf/couchdb-couch-httpd/diff/305e54b7 Branch: refs/heads/split_out_httpd_stack Commit: 305e54b7b8d3f4f45bc3b3e11544c58f26105e73 Parents: 09d0245 Author: ILYA Khlopotov <iil...@ca.ibm.com> Authored: Thu Feb 25 11:27:35 2016 -0800 Committer: ILYA Khlopotov <iil...@ca.ibm.com> Committed: Tue Mar 1 08:35:07 2016 -0800 ---------------------------------------------------------------------- src/couch_httpd.erl | 1 + src/couch_httpd_original.erl | 17 ++--------------- 2 files changed, 3 insertions(+), 15 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/couchdb-couch-httpd/blob/305e54b7/src/couch_httpd.erl ---------------------------------------------------------------------- diff --git a/src/couch_httpd.erl b/src/couch_httpd.erl index 3928c3a..5db4989 100644 --- a/src/couch_httpd.erl +++ b/src/couch_httpd.erl @@ -116,6 +116,7 @@ }). start_response_length(#httpd{mochi_req=MochiReq}=Req, Code, Headers0, Length) -> + log_request(Req, Code), couch_stats:increment_counter([couchdb, httpd_status_codes, Code]), Headers1 = Headers0 ++ server_header() ++ couch_httpd_auth:cookie_auth_header(Req, Headers0), http://git-wip-us.apache.org/repos/asf/couchdb-couch-httpd/blob/305e54b7/src/couch_httpd_original.erl ---------------------------------------------------------------------- diff --git a/src/couch_httpd_original.erl b/src/couch_httpd_original.erl index fb56c74..58937f6 100644 --- a/src/couch_httpd_original.erl +++ b/src/couch_httpd_original.erl @@ -22,7 +22,6 @@ -export([etag_maybe/2]). -export([start_chunked_response/3,send_chunk/2]). --export([start_response_length/4]). -export([send_response/4,send_error/2,send_error/4, send_chunked_error/2]). -export([accepted_encodings/1,handle_request_int/5,validate_referer/1]). -export([http_1_0_keep_alive/2]). @@ -68,7 +67,8 @@ log_request/2, etag_respond/3, etag_match/2, - start_reponse/3 + start_reponse/3, + start_response_length/4 ]). -define(HANDLER_NAME_IN_MODULE_POS, 6). @@ -494,19 +494,6 @@ verify_is_server_admin(#user_ctx{roles=Roles}) -> false -> throw({unauthorized, <<"You are not a server admin.">>}) end. -start_response_length(#httpd{mochi_req=MochiReq}=Req, Code, Headers, Length) -> - log_request(Req, Code), - couch_stats:increment_counter([couchdb, httpd_status_codes, Code]), - Headers1 = Headers ++ server_header() ++ - couch_httpd_auth:cookie_auth_header(Req, Headers), - Headers2 = couch_httpd_cors:cors_headers(Req, Headers1), - Resp = MochiReq:start_response_length({Code, Headers2, Length}), - case MochiReq:get(method) of - 'HEAD' -> throw({http_head_abort, Resp}); - _ -> ok - end, - {ok, Resp}. - no_resp_conn_header([]) -> true;