Hockenba commented on PR #13651: URL: https://github.com/apache/apisix/pull/13651#issuecomment-5680591235
> This introduces a Redis lookup on every request that opens an existing cookie session when session.revocation is redis, not only on logout. For example, 100 requests using the same session within 3–5 seconds can generate roughly 100 Redis GETs. > > Could we add a small, short-lived local cache keyed by the hashed session ID? On a cache miss, query Redis and cache the result for around 3–5 seconds. Logout should update or invalidate the local cache after writing the revocation marker. This keeps the implementation simple while avoiding dense repeated lookups for the same session. The TTL would be the maximum local stale window for a just-revoked session. @membphis PR for this: https://github.com/bungle/lua-resty-session/pull/213. Please take a quick peak when you have the time to review. -- 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]
