[Webkit-unassigned] [Bug 258389] New: [Curl] Replace handling of reason-phrase in ResourceResponseCurl

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=258389

            Bug ID: 258389
           Summary: [Curl] Replace handling of reason-phrase in
                    ResourceResponseCurl
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Platform
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: kenji.shukuwa at open-tec.co.jp

In ResourceResponseCurl, the reason-phrase of the HTTP status line is obtained
in the ResourceResponse::setStatusLine function, but in order to reduce the code
to be maintained, replace it with extractReasonPhraseFromHTTPStatusLine of HTTPParsers.

Additionally, the reason-phrase can contain ":". Therefore, there is a possibility
that the reason-phrase cannot be obtained with the current processing.
(for example: HTTP/1.1 999 ABC:DEF)

https://github.com/WebKit/WebKit/blob/e45a46fae1a59709b4d5dd5c043352129364d150/Source/WebCore/platform/network/curl/ResourceResponseCurl.cpp#L117-L129
---
    auto splitPosition = header.find(':');
    if (splitPosition != notFound) {
        ...
    } else if (startsWithLettersIgnoringASCIICase(header, "http"_s)) {
        // This is the first line of the response.
        setStatusLine(header);
    }
---

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230622/2e25cc51/attachment.htm>