Thijs Schreijer
Thijs Schreijer
accidentally ran into this PR while browsing the lib. I think it is worth noting that the library will only have a single worker do the healthchecks. Hence the callbacks...
@thibaultcha this PR is correct and helpful
@thibaultcha mind having a look?
the CI failures seem to be unrelated?
ping @thibaultcha
@thibaultcha @spacewander just ran into this old PR. Rebased it. Can we either merge it, or close it if there is no intention to merge it?
the default might reuse a connection opened with different properties, and hence is a security issue
probably this isn't even enough, see https://github.com/ledgetech/lua-resty-http/pull/206/files#diff-654e4e7b65177f9d0e6eb1d4b1c259dbR133-R143
If, in the above example, I change: ```lua sema:post(1000) ``` into ```lua sema:post(1000) ngx.timer.at(0, function() ngx.log(ngx.WARN,"please ignore") end) ``` then it works as expected. ping @javierguerragiraldez