Don't log poolname

README.md

@@ -170,6 +170,7 @@ The options table has the following fields:
 * `host`: target host, or path to a unix domain socket
 * `port`: port on target host, will default to `80` or `443` based on the scheme
 * `pool`: custom connection pool name. Option as per [OpenResty docs](https://github.com/openresty/lua-nginx-module#tcpsockconnect), except that the default will become a pool name constructed using the SSL / proxy properties, which is important for safe connection reuse. When in doubt, leave it blank!
+* `pool_debug`: set to `true` to log the connection pool name at `DEBUG` level, off by default
 * `pool_size`: option as per [OpenResty docs](https://github.com/openresty/lua-nginx-module#tcpsockconnect)
 * `backlog`: option as per [OpenResty docs](https://github.com/openresty/lua-nginx-module#tcpsockconnect)
 * `proxy_opts`: sub-table, defaults to the global proxy options set, see [set\_proxy\_options](#set_proxy_options).

lib/resty/http_connect.lua

@@ -38,6 +38,7 @@ client:connect {
     host = "myhost.com",    -- target machine, or a unix domain socket
     port = nil,             -- port on target machine, will default to 80/443 based on scheme
     pool = nil,             -- connection pool name, leave blank! this function knows best!
+    pool_debug = nil,       -- set to 'true' to log the connection pool name at DEBUG level
     pool_size = nil,        -- options as per: https://github.com/openresty/lua-nginx-module#tcpsockconnect
     backlog = nil,
 
@@ -250,7 +251,9 @@ local function connect(self, options)
         -- with a plain http request the authorization is part of the actual request.
     end
 
-    ngx_log(ngx_DEBUG, "poolname: ", poolname)
+    if options.pool_debug == true then
+        ngx_log(ngx_DEBUG, "poolname: ", poolname)
+    end
 
     -- do TCP level connection
     local tcp_opts = { pool = poolname, pool_size = pool_size, backlog = backlog }