lua-resty-acme icon indicating copy to clipboard operation
lua-resty-acme copied to clipboard
verified publisher icon fffonion

Use atomic set and redis pool size

Open kozak opened this issue 1 year ago • 2 comments

kozak avatar Nov 12 '24 12:11 kozak

@kozak Thank you for keep this updated. Could you rebase to latest master? There's some changes to redis driver.

fffonion avatar Nov 12 '24 12:11 fffonion

@jborkowski has just done a rebase.

Btw. the atomic set changes have eliminated the locking issue, so it would be nice if they could be merged into master :)

The redis pool size change doesn't seem to be controversial, but does come with a requirement for ngx lua 0.10.14 (https://github.com/openresty/lua-resty-redis?tab=readme-ov-file#connect)

kozak avatar Nov 13 '24 12:11 kozak