Uploaded image for project: 'Spring Security'
  1. Spring Security
  2. SEC-1008

UserDetailsChecker.check(UserDetails, Authentication) or UserDetailsChecker.check(UserDetails, Object authenticationDetails) (on your taste)

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: 3.0.0 M1
    • Component/s: Core
    • Labels:
      None

      Description

      Please, add Authentication (or authenticationDetails ) parameter to UserDetailsChecker.check(..) method. I need remote address for some sort checks.

        Activity

        Hide
        luke Luke Taylor added a comment -

        I think this would ultimately be a misplacement of authentication logic which should probably go in an extended AuthenticationProvider. This interface is intended for simple checking of persistent data (for locked flags etc). Changing it would break existing implementations and it is also used from situations where an Authentication object isn't actually available.

        Show
        luke Luke Taylor added a comment - I think this would ultimately be a misplacement of authentication logic which should probably go in an extended AuthenticationProvider. This interface is intended for simple checking of persistent data (for locked flags etc). Changing it would break existing implementations and it is also used from situations where an Authentication object isn't actually available.

          People

          • Assignee:
            luke Luke Taylor
            Reporter:
            gerr_mes Gerr Magnus Mes
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: