Thijs Schreijer

Results 444 comments of 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 mind having a look?

the CI failures seem to be unrelated?

@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