Uploaded image for project: 'Couchbase Lite'
  1. Couchbase Lite
  2. CBL-1908

Add cookie using setValue instead of addValue

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.8.0
    • 3.0
    • iOS
    • Security Level: Public
    • None
    • Jay 47
    • 2

    Description

      When using the 'addValue:forHTTPHeaderField', it incrementally appends the key-value using a comma.

      Multiple message-header fields with the same field-name MAY be present in a message if and only if the entire field-value for that header field is defined as a comma-separated list
      // https://tools.ietf.org/html/rfc2616#page-32

      May be we should use 'setValue: forHTTPHeaderField', which avoids the issue of cookies with comma delimiter, instead semi-colon delimiter.

      Related GH issue: https://github.com/couchbase/couchbase-lite-ios/issues/2828

      Validate fix using proxy app?

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            No work has yet been logged on this issue.

            People

              Jayahari.Vavachan Jay Vavachan
              Jayahari.Vavachan Jay Vavachan
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty