search

socialsignin / spring-social-security-demo

Spring Social Security Demo - using the default user local account persistence of Spring-Social-Security
23 stars 23 forks source link

SpringSocialSecurityDemoWebappConfig has multiple PostConstruct methods #5

Closed chrylis closed 10 years ago

chrylis commented 10 years ago

The @Configuration class annotates multiple methods with @PostConstruct, but the annotation's spec requires a maximum of one annotated method. The example may work because one method is a no-op, but this behavior is unspecified.

michaellavelle commented 10 years ago

Thank you chrylis - the @PostConstruct method in question is implemented in SocialConfig and should have been removed from SpringSocialSecurityDemoWebappConfig - I've now removed this duplicate method. - Cheers, Michael