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.
          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.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development