brentmjohnson opened a new issue, #8017: URL: https://github.com/apache/apisix/issues/8017
### Current Behavior Server-side session stores that support locking remain locked after authentication until the default or configured lock ttl ([https://github.com/bungle/lua-resty-session#redis-storage-adapter](https://github.com/bungle/lua-resty-session#:~:text=set%20%24session_redis_maxlockwait%20%20%20%20%20%20%20%20%20%20%20%20%20%2030%3B%20%20%20%23%20(in%20seconds))) expires. This results in subsequent requests (using the same APISIX session) hanging until the lock expires. Apisix plugin code: https://github.com/apache/apisix/blob/cf8de8ebc2b00f3fb3db50b3d5415a189c7a0680/apisix/plugins/openid-connect.lua#L312 Does not implement explicit session:close() as recommended by zmartzone/lua-resty-openidc [https://github.com/zmartzone/lua-resty-openidc#sessions-and-locking](https://github.com/zmartzone/lua-resty-openidc#sessions-and-locking:~:text=authenticate(opts)-,session%3Aclose(),-Caching) ### Expected Behavior APISIX openid-connect plugin should call explicit session:close() and subsequent requests (using the same APISIX session) should complete immediately. ### Error Logs _No response_ ### Steps to Reproduce 1. Configure APISIX openid-connect plugin for a server-side session store that supports locking 2. Configure route using the openid-connect plugin 3. Authenticate on protected route 4. Immediately refresh route using the same session cookie - request will hang until the default or configured lock ttl ### Environment - APISIX version (run `apisix version`): 2.15.0 - Operating system (run `uname -a`): Linux apisix-6fcbf58999-t8x9r 5.15.0-48-generic # 54-Ubuntu SMP Fri Aug 26 13:26:29 UTC 2022 x86_64 Linux - OpenResty / Nginx version (run `openresty -V` or `nginx -V`): nginx version: openresty/1.21.4.1 - etcd version, if relevant (run `curl http://127.0.0.1:9090/v1/server_info`): N/A - APISIX Dashboard version, if relevant: N/A - Plugin runner version, for issues related to plugin runners: N/A - LuaRocks version, for installation issues (run `luarocks --version`): /usr/local/bin/luarocks 3.8. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
