Wicket
  1. Wicket
  2. WICKET-4989

WicketTester should send copies of its cookies

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.9, 6.4.0
    • Fix Version/s: 1.5.10, 6.7.0
    • Component/s: wicket
    • Labels:
      None
    • Environment:
      any

      Description

      There are some bugs related to cookie handling in BaseWicketTester.

      Problem 1) a cookie set in the request should not be in the response unless manually set by the server code

      Problem 2) wicket tester sends the original cookie instance to the server. If the server code manipulates the cookie anyhow (e.g. change its value) but do not send it back explicitly in the response then the original cookie in WicketTester (the request cookie) is updated while it shouldn't be.

      A possible solutions for these problems can be found at:
      https://github.com/apache/wicket/pull/28

        Issue Links

          Activity

          Sven Meier made changes -
          Link This issue is related to WICKET-5147 [ WICKET-5147 ]
          Martin Grigorov made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 6.7.0 [ 12323964 ]
          Resolution Fixed [ 1 ]
          Martin Grigorov made changes -
          Description There are many bugs related to the BaseWicketTester cookie handling. I picket two of then in the attached wicket-quickstart and i did some refactoring on a github-branch to fix this.

          https://github.com/apache/wicket/pull/28
          There are some bugs related to cookie handling in BaseWicketTester.

          Problem 1) a cookie set in the request should not be in the response unless manually set by the server code

          Problem 2) wicket tester sends the original cookie instance to the server. If the server code manipulates the cookie anyhow (e.g. change its value) but do not send it back explicitly in the response then the original cookie in WicketTester (the request cookie) is updated while it shouldn't be.

          A possible solutions for these problems can be found at:
          https://github.com/apache/wicket/pull/28
          Martin Grigorov made changes -
          Summary BaseWicketTester Cookie Bugs WicketTester should send copies of its cookies
          Martin Grigorov made changes -
          Fix Version/s 1.5.10 [ 12323510 ]
          Martin Grigorov made changes -
          Assignee Martin Grigorov [ mgrigorov ]
          Michael Mosmann made changes -
          Field Original Value New Value
          Attachment quickstart-basewickettester-bug.zip [ 12565394 ]
          Michael Mosmann created issue -

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development