spring-cloud / spring-cloud-common-security-config

A common security infrastructure used by Spring Cloud Data Flow and the projects in its ecosystem
19 stars 32 forks source link

Added configuration test. #90

Closed corneil closed 2 years ago

corneil commented 2 years ago

Added application and config test. Assuming the changes on 25th to main resolved the circular dependency.

Cannot reproduce #89

corneil commented 2 years ago

I moved most of the bean methods into separate static classes that are imported into the original, ensuring that nothing that is needed by OAuthSecurityConfiguration has to come from a method annotated with @Bean except for those which don't need external fields and creates everything needed. ProviderManager was also changed to required = false since it is only required when opaqueTokenIntrospector is present. Both config classes have the same @ConditionalOnProperty annotation.

onobc commented 2 years ago

Closed via https://github.com/spring-cloud/spring-cloud-common-security-config/commit/9bcf040ea02a33016594b5c7092621946967a759