jdaugherty commented on PR #15405: URL: https://github.com/apache/grails-core/pull/15405#issuecomment-3916011158
Wait, I just looked at the referenced spring security config - this PR makes it sound like we agreed on this approach and this PR is just following that approach. But that approach doesn't appear to be existing; it's new via https://github.com/apache/grails-spring-security/pull/1205/changes As an optional, external plugin I understand that it needs to implement the exclusion logic, but we've approached the Grails autoconfiguration exclusion differently in core. We need to decide on one way and not adopt multiple ways. ApplicationClassInjector is the class that has built in excludes. The database plugin should be added here if it's present on the classpath. @matrei what are your thoughts? -- 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]
