[Issue] Update Curl to respect case-insensitive headers #29524
Labels
Component: HTTP
Fixed in 2.4.x
The issue has been fixed in 2.4-develop branch
Issue: Confirmed
Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed
Issue: Format is valid
Gate 1 Passed. Automatic verification of issue format passed
Issue: Ready for Work
Gate 4. Acknowledged. Issue is added to backlog and ready for development
Priority: P2
A defect with this priority could have functionality issues which are not to expectations.
Progress: done
Reported on 2.4.0
Indicates original Magento version for the Issue report.
Reproduced on 2.4.x
The issue has been reproduced on latest 2.4-develop branch
Severity: S3
Affects non-critical data or functionality and does not force users to employ a workaround.
This issue is automatically created based on existing pull request: #29274: Update Curl to respect case-insensitive headers
According to RFC 2616 header names are case-insensitive: "Each header field consists of a name followed by a colon (":") and the field value. Field names are case-insensitive." - see https://tools.ietf.org/html/rfc2616#section-4.2
The "Set-Cookie" comparison in the current Curl client is case-sensitive and can cause problems with some (rare) HTTP servers.
Preconditions: (*)
Magento 2.4-develop
According to RFC 2616 header names are case-insensitive: "Each header field consists of a name followed by a colon (":") and the field value. Field names are case-insensitive." - see https://tools.ietf.org/html/rfc2616#section-4.2
The "Set-Cookie" comparison in the previous version is case-sensitive and can cause problems with some (rare) HTTP servers.
Related Pull Requests
None
Fixed Issues (if relevant)
Did not create an issue
Steps to reproduce:
Actual Result: ✖️ It doesn't respect those header names ("set-cookie" or "SET-COOKIE")
Expected Result: ✔️ It should respect those header names
Questions or comments
Please note that I did not provide a unit test for this specific issue. If required please let me know.
Contribution checklist (*)
The text was updated successfully, but these errors were encountered: