Bug 61778

Summary: AuthConfigFactoryImpl.detachListener() alwas returns an empty string array
Product: Tomcat 8 Reporter: Lazar Kirchev <lazar.kirchev>
Component: CatalinaAssignee: Tomcat Developers Mailing List <dev>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P2    
Version: 8.5.23   
Target Milestone: ----   
Hardware: PC   
OS: All   

Description Lazar Kirchev 2017-11-18 13:51:33 UTC
RegistrationContextImpl.removeListener() always returns false - it initialises the return value in the beginning, but never changes it even if the listener for removal is found. The result is that when detachListener() calls removeListener, it allways gets false, and finally does not return the correct registration id.
Comment 1 Lazar Kirchev 2017-11-18 13:52:50 UTC
This pull request contains a test case, which illustrates the problem and a fix for it: https://github.com/apache/tomcat/pull/86
Comment 2 Mark Thomas 2017-11-20 12:58:21 UTC
Again, many thanks for the report, patch and test case.

Fixed in:
- trunk for 9.0.2 onwards
- 8.5.x for 8.5.24 onwards