Make ConnectController and ProviderSignInController more extendable

[michaellavelle]

It would be really useful to be able to extend ConnectController and ProviderSignInController to add custom callback methods. For example, the callbacks from LastFm's api are neither OAuth1 or OAuth2 but are similar (using a different request parameter name). Adding new callback methods means requiring much of the functionality of the existing callback methods, but the existing callback methods use private helper methods and attributes. Making the visibility of these helper methods and attributes protected rather than private would make extending these controllers easier.

See

https://github.com/michaellavelle/spring-social-lastfm-non-oauth-web/blob/master/src/main/java/org/springframework/social/lastfm/nonoauth/connect/web/LastFmNonOAuthSignInController.java

for an example of this use case.

This controller extends a custom controller class

https://github.com/socialsignin/spring-social-web-non-oauth-extension/blob/master/src/main/java/org/springframework/social/connect/web/ExtensibleProviderSignInController.java

but could simply extend the existing ProviderSignInController if these visibility modifications were made.

[axelhzf]

Why it isn't add to master?? I'm working on some redirect after login and I need overwrite this methods

[pivotal-issuemaster]

@michaellavelle Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.