[tarantool-patches] [PATCH] httpc: set reason when status is more than 400
Roman Khabibov
roman.habibov at tarantool.org
Mon Feb 4 16:59:27 MSK 2019
Set the reason "Unknown" when it is CURLE_OK and status is more than 400.
Closes #3681
Branch: https://github.com/tarantool/tarantool/tree/romanhabibov/gh-3681-httpc-reason
Issue: https://github.com/tarantool/tarantool/issues/3681
---
src/httpc.c | 8 ++++++--
test/app-tap/http_client.test.lua | 7 +++++--
2 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/src/httpc.c b/src/httpc.c
index 950f8b32f..eb7a5334f 100644
--- a/src/httpc.c
+++ b/src/httpc.c
@@ -325,8 +325,12 @@ httpc_execute(struct httpc_request *req, double timeout)
case CURLE_OK:
curl_easy_getinfo(req->curl_request.easy, CURLINFO_RESPONSE_CODE, &longval);
req->status = (int) longval;
- /* TODO: get real response string from resp->headers */
- req->reason = "Ok";
+
+ if (req->status < 400 && req->status >= 100)
+ req->reason = "Ok";
+ else
+ req->reason = "Unknown";
+
if (req->status == 200) {
++env->stat.http_200_responses;
} else {
diff --git a/test/app-tap/http_client.test.lua b/test/app-tap/http_client.test.lua
index 10a3728f8..d4f3a9f45 100755
--- a/test/app-tap/http_client.test.lua
+++ b/test/app-tap/http_client.test.lua
@@ -62,12 +62,13 @@ local function stop_server(test, server)
end
local function test_http_client(test, url, opts)
- test:plan(9)
+ test:plan(10)
test:isnil(rawget(_G, 'http'), "global namespace is not polluted");
test:isnil(rawget(_G, 'http.client'), "global namespace is not polluted");
local r = client.get(url, opts)
test:is(r.status, 200, 'simple 200')
+ test:is(r.reason, 'Ok', '200 - Ok')
test:is(r.proto[1], 1, 'proto major http 1.1')
test:is(r.proto[2], 1, 'proto major http 1.1')
test:ok(r.body:match("hello") ~= nil, "body")
@@ -104,7 +105,7 @@ local function test_cancel_and_errinj(test, url, opts)
end
local function test_post_and_get(test, url, opts)
- test:plan(19)
+ test:plan(21)
local http = client.new()
test:ok(http ~= nil, "client is created")
@@ -159,6 +160,7 @@ local function test_post_and_get(test, url, opts)
r = responses.absent_get
test:is(r.status, 500, "GET: absent method http code page exists")
+ test:is(r.reason, 'Unknown', '500 - Unknown')
test:is(r.body, "No such method", "GET: absent method right body")
r = responses.empty_post
@@ -180,6 +182,7 @@ local function test_post_and_get(test, url, opts)
r = responses.bad_get
test:is(r.status, 404, "GET: http page not exists")
+ test:is(r.reason, 'Unknown', '404 - Unknown')
test:isnt(r.body:len(), 0, "GET: not empty body page not exists")
test:ok(string.find(r.body, "Not Found"),
"GET: right body page not exists")
--
2.17.1
More information about the Tarantool-patches
mailing list