Wicket
  1. Wicket
  2. WICKET-5123

Component.continueToOriginalDestination() can redirect to ./.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.6.0, 1.5.10
    • Fix Version/s: 6.7.0, 1.5.11
    • Component/s: wicket
    • Labels:
      None

      Description

      Following scenario:

      Home page redirects to login page via RestartResponseException if no user is logged in.
      The user logs in, and is led back to the originally called page via continueToOriginalDestination().

      If the original page was the home page, Wicket's UrlRenderer produces a target url of ./. here (which IE browsers can't handle).

      1. urlrenderer.tar.gz
        22 kB
        Martin Dietze

        Activity

        Martin Dietze created issue -
        Hide
        Martin Dietze added a comment -

        Browse to 'http://localhost'. You will be redirected to the LoginPage on which the target URL is printed as it would be generated inside of continueToOriginalDestination().

        Show
        Martin Dietze added a comment - Browse to 'http://localhost'. You will be redirected to the LoginPage on which the target URL is printed as it would be generated inside of continueToOriginalDestination().
        Martin Dietze made changes -
        Field Original Value New Value
        Attachment urlrenderer.tar.gz [ 12575899 ]
        Sven Meier made changes -
        Affects Version/s 1.5.10 [ 12323510 ]
        Affects Version/s 6.6.0 [ 12323963 ]
        Priority Minor [ 4 ] Major [ 3 ]
        Assignee Sven Meier [ svenmeier ]
        Hide
        Sven Meier added a comment -

        url is now rendered as "." instead of "./."

        Show
        Sven Meier added a comment - url is now rendered as "." instead of "./."
        Sven Meier made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 6.7.0 [ 12323964 ]
        Fix Version/s 1.5.11 [ 12324069 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        3h 1 Sven Meier 28/Mar/13 20:40

          People

          • Assignee:
            Sven Meier
            Reporter:
            Martin Dietze
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 4h
              4h
              Remaining:
              Remaining Estimate - 4h
              4h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development