mmodzelewski opened a new pull request, #2910: URL: https://github.com/apache/iggy/pull/2910
Pooled connections authenticated with a simple boolean flag failed to re-authenticate after logout/re-login cycles. Channels from the prior session appeared authenticated but held stale server-side state, causing silent failures. Replace the boolean auth attribute with an AtomicLong generation counter. Each login increments the generation; logout invalidates all channels by advancing it. Channels compare their stored generation on acquire and transparently re-authenticate when stale. Additional cleanup: move pool initialization into the constructor with eager connectivity validation, extract PoolChannelHandler as a static inner class, fix payload leak on acquire failure by releasing in finally, fix login capture to include PAT login (was incorrectly capturing UPDATE), reduce IggyAuthenticator/FrameEncoder visibility to package-private, and migrate to Netty 5 IoEventLoopGroup API. -- 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]
