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

Consolidate OAuthSecurityConfiguration #91

Closed onobc closed 2 years ago

onobc commented 2 years ago

I missed this on the previous commit to OAuthSecurityConfiguration, but we can move to bean method injection rather than constructor injection on the static configuration class wrappers (reduces lines of code quite a bit).

I ran SCDF and Skipper tests locally w/ this updated version - all was well.

@corneil I notice this when I was stepping through debugger in figuring out the 1 @Ignore test in SCDF.