fix: keepalive_ready mark only after connection

Author: GuyLewin

lib/resty/http.lua

@@ -135,7 +135,7 @@ function _M.new(_)
         return nil, err
     end
     return setmetatable({
-        sock = sock, keepalive_supported = true, keepalive_ready = true, pool_only_after_response = false
+        sock = sock, keepalive_supported = true, keepalive_ready = false, pool_only_after_response = false
     }, mt)
 end
 

lib/resty/http_connect.lua

@@ -263,6 +263,8 @@ local function connect(self, options)
 
     self.host = request_host
     self.port = request_port
+    -- Immediately after connection - keepalive should be possible
+    self.keepalive_ready = true
     self.keepalive_supported = true
     self.ssl = ssl
     -- set only for http, https has already been handled

t/07-keepalive.t

@@ -524,13 +524,19 @@ response not fully read
 [error]
 [warn]
 
-=== TEST 10 Pooling connection immediately after creation should work
+=== TEST 10 Pooling connection immediately after connecting should work
 --- http_config eval: $::HttpConfig
 --- config
     location = /a {
         content_by_lua '
             local http = require "resty.http"
             local httpc = http.new()
+            httpc:connect({
+                scheme = "http",
+                host = "127.0.0.1",
+                port = ngx.var.server_port,
+                pool_only_after_response = true
+            })
             ngx.say(httpc:set_keepalive())
         ';
     }