Spring Security
  1. Spring Security
  2. SEC-2204

Problems with the spring security annotation config social sample

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Complete
    • Affects Version/s: 3.2.0.M2
    • Fix Version/s: None
    • Component/s: Samples

      Description

      As per Craig's request, creating this ticket. This is related to the problems detailed in this forum post.

      http://forum.springsource.org/showthread.php?139768-Spring-social-with-spring-security-annotation-config-No-authentication-provider-found

      Those are namely;
      1- Small configuration problem with logout. Logout doesn't work without logoutSuccessUrl.
      2- UserIdSource needs to return AuthenticationUserIdExtractor otherwise userId passed in is null.
      3- registerAuthentication is missing socialAuthenticationProvider. Must be registered otherwise gives "No AuthenticationProvider found". Detailed in forum post.
      4- Minor, "accessToken" column on userConnection table didn't fit for FB response data I had to expand it to 384 chars.
      5- Last but not least I see null value for userDetailsService which is passed into SocialUserDetailsService, so second time around trying to login with Face book throws NPE.

      Caveat: I have a muti-module project so I shuffled things around in between service and config and web modules. If you believe some of the items above are working fine, it could be my mistake, please disregard those.

      Thanks for all your hard work on security annotation config.

        Activity

        Hide
        Rob Winch added a comment -

        Thank you for your feedback. All of the feedback appears to have been relevant. The samples don't yet exist in Spring Security's codebase so I have pushed fixes to the spring-security-javaconfig modules project. You can find the commit here https://github.com/SpringSource/spring-security-javaconfig/commit/d2949f7efc9265c23691af963eae31b426dd6fc7 Thanks again for taking the time to create a JIRA.

        Show
        Rob Winch added a comment - Thank you for your feedback. All of the feedback appears to have been relevant. The samples don't yet exist in Spring Security's codebase so I have pushed fixes to the spring-security-javaconfig modules project. You can find the commit here https://github.com/SpringSource/spring-security-javaconfig/commit/d2949f7efc9265c23691af963eae31b426dd6fc7 Thanks again for taking the time to create a JIRA.
        Hide
        Adam Aytul added a comment -

        Thanks for the quick turnaround, seems all problems fixed.

        Show
        Adam Aytul added a comment - Thanks for the quick turnaround, seems all problems fixed.

          People

          • Assignee:
            Rob Winch
            Reporter:
            Adam Aytul
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: