Details

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

      Description

      I have been using the switch user processing filter and there are a number of features that could be added that I think may be useful. I would appreciate it if you would consider these for the future.

      Firstly the user details checker is hard coded to be a AccountStatusUserDetailsChecker. It would be useful to allow control of this so that a custom UserDetailsChecker can be used (even a setter method so that the default can be overwritten in the XML config).

      Secondly there is not any way to handle or respond to any exceptions that are thrown such as the UsernameNotFoundException. The issue may cross reference issue SEC-778 where the exception mapping is going to be addressed.

      Thirdly there is not a way to tell the user what went wrong via an exception message. Would it be possible to add the exception to the session like the abstract processing filter does for the logon.

        Issue Links

          Activity

          Hide
          Luke Taylor added a comment -

          I've modified SwitchUserProcessingFilter to make use of the strategies introduced in SEC-745, so it should be possible to inject any customized navigation behaviour into it for both success and failure conditions.

          Show
          Luke Taylor added a comment - I've modified SwitchUserProcessingFilter to make use of the strategies introduced in SEC-745 , so it should be possible to inject any customized navigation behaviour into it for both success and failure conditions.

            People

            • Assignee:
              Luke Taylor
              Reporter:
              Neil Anderson
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: