mattrpav commented on PR #1945: URL: https://github.com/apache/karaf/pull/1945#issuecomment-3181003565
@globalbus the cache needs to be cleared on any config change, data change and connection status change. If the LDAP server changes (connect-disconnect), it is not safe to presume the data will be the same. I'm not married to the static class, but injecting anything into Karaf's JAAS module system is complicated to get consistent via all the provisioning methods: cfg file, blueprint deployment or scr deployment. I'm leaning to keep this change to be "fix" what is broken and then discuss a refactor/improvement as a follow-on activity. -- 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]
