Wicket
  1. Wicket
  2. WICKET-5131

Problems with cookies disabled when using 301/302 and also 303 (even with cookies)

    Details

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

      Description

      As mentioned in the mailing list by Martin, i open this as a bug...

      Its not possible to use 303 as redirect (SC_SEE_OTHER) because thats not supported, only 302 and 301 are supported but this is defined in RFC HTTP 1.1 from 1997.

      301 will add the Location header - which works as expected when disabling cookies. But a 302 (which is what i prefer) will redirect to the same page because the Location header is missing. When i enable cookies, its working.

      Example can be found here: https://github.com/olze/WicketRedirect

        Issue Links

          Activity

          Hide
          Martin Grigorov added a comment -

          It is planned for next Friday (12.04).

          Show
          Martin Grigorov added a comment - It is planned for next Friday (12.04).
          Hide
          oliver z added a comment -

          Just tested it. SC_SEE_OTHER works like a charm (with and without cookies)
          SC_MOVED_PERMANENTLY also works like a charm with and without cookies.
          SC_MOVED_TEMPORARLY does not work when cookies are disabled.

          Thanks! Is there some release planning when 6.7.0 comes out? Jira said 15.3.

          Show
          oliver z added a comment - Just tested it. SC_SEE_OTHER works like a charm (with and without cookies) SC_MOVED_PERMANENTLY also works like a charm with and without cookies. SC_MOVED_TEMPORARLY does not work when cookies are disabled. Thanks! Is there some release planning when 6.7.0 comes out? Jira said 15.3.
          Martin Grigorov made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee Martin Grigorov [ mgrigorov ]
          Fix Version/s 6.7.0 [ 12323964 ]
          Fix Version/s 1.5.11 [ 12324069 ]
          Resolution Fixed [ 1 ]
          Hide
          Martin Grigorov added a comment -

          Added support for SC_SEE_OTHER.
          Please test it with 6.7.0-SNAPSHOT before next Friday if you can.

          Show
          Martin Grigorov added a comment - Added support for SC_SEE_OTHER. Please test it with 6.7.0-SNAPSHOT before next Friday if you can.
          Martin Grigorov made changes -
          Field Original Value New Value
          Remote Link This issue links to "Mail discussion (Web Link)" [ 12136 ]
          oliver z created issue -

            People

            • Assignee:
              Martin Grigorov
              Reporter:
              oliver z
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development