Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.4
    • Component/s: None
    • Labels:
      None

      Description

      WSHandler#getPassword tries to get a password by:

      a) Trying to construct a password callback
      b) Trying to get a password callback object from the Message Context
      c) Trying to get the password directly from the Message Context, i.e. delegating this to a subclass of WSHandler

      There are some problems with how the current code handles option (c). In particular, the action to be performed, as well as the username, are not preserved in the WSPasswordCallback object that gets returned. This in turn leads to problems with the UsernameTokenAction, which obtains the username from the WSPasswordCallback.

      See attached for a patch for this problem. In addition, I added some fairly extensive unit testing of the getPassword method.

      1. wss4j_get_password.patch
        15 kB
        Colm O hEigeartaigh

        Activity

        Colm O hEigeartaigh created issue -
        Colm O hEigeartaigh made changes -
        Field Original Value New Value
        Attachment wss4j_get_password.patch [ 12380288 ]
        Fred Dushin made changes -
        Assignee Ruchith Udayanga Fernando [ ruchith ] Fred Dushin [ fdushin ]
        Hide
        Fred Dushin added a comment -

        Applied Colm's fix.

        This change should be noncontroversial – it's a simple refactor of the creation of the WSPasswordCallback object, and does not result in any change in behavior (except for the addition of information to the callback instance created)

        Show
        Fred Dushin added a comment - Applied Colm's fix. This change should be noncontroversial – it's a simple refactor of the creation of the WSPasswordCallback object, and does not result in any change in behavior (except for the addition of information to the callback instance created)
        Fred Dushin made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Fred Dushin added a comment -

        Bulk change of "Fix Version" for all issues fixed or closed for 1.5.4

        Show
        Fred Dushin added a comment - Bulk change of "Fix Version" for all issues fixed or closed for 1.5.4
        Fred Dushin made changes -
        Fix Version/s 1.5.4 [ 12313167 ]
        Colm O hEigeartaigh made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1d 3h 22m 1 Fred Dushin 17/Apr/08 20:08
        Resolved Resolved Closed Closed
        1263d 13h 54m 1 Colm O hEigeartaigh 03/Oct/11 10:02

          People

          • Assignee:
            Fred Dushin
            Reporter:
            Colm O hEigeartaigh
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development