fix: more informative error message to tell that the server doesn't match http/1.1 protocol

[songkeys]

This PR just tries to make the error message more informative for http/2 server response.

fix #2043

Error message after this change:

HTTPParserError: Response does not match the HTTP/1.1 protocol (Expected HTTP/)
    at Parser.execute (/Users/songkeys/GitHub/Songkeys/undici/lib/client.js:575:15)
    at Parser.readMore (/Users/songkeys/GitHub/Songkeys/undici/lib/client.js:516:12)
    at TLSSocket.onSocketReadable (/Users/songkeys/GitHub/Songkeys/undici/lib/client.js:943:10)
    at TLSSocket.emit (node:events:513:28)
    at emitReadable_ (node:internal/streams/readable:590:12)
    at processTicksAndRejections (node:internal/process/task_queues:81:21) {
  code: 'HPE_INVALID_CONSTANT',
  data: '...'
}

P.S. I was confused by the Expected HTTP/ error message thrown from the llhttp module. I thought it should be Expected HTTP/1.1? But I didn't change them cuz they are in serveral places so might involve in other issues. I included them in the brackets on the tail.

[mcollina]

lgtm

[KhafraDev]

actually, a test is failing

 # Subtest: test/parser-issues.js
    # Subtest: https://github.com/mcollina/undici/issues/268
        1..2
        ok 1 - should not error
        ok 2 - (unnamed test)
    ok 1 - https://github.com/mcollina/undici/issues/268 # time=2041.849ms
    
    # Subtest: parser fail
        1..3
        ok 1 - expect truthy value
        ok 2 - should be equal
        not ok 3 - should be equal
          ---
          compare: ===
          at:
            line: 60
            column: 9
            file: test/parser-issues.js
          stack: |
            test/parser-issues.js:60:9
            lib/api/api-request.js:2:113
          source: |2
                  t.equal(err.code, 'HPE_INVALID_CONSTANT')
                  t.equal(err.message, 'Expected HTTP/')
            --------^
                })
              })
          diff: |
            --- expected
            +++ actual
            @@ -1,1 +1,1 @@
            -Expected HTTP/
            +Response does not match the HTTP/1.1 protocol (Expected HTTP/)
          ...
        

[songkeys]

Tried to fix with t.type instead of pure string matching. PTAL.

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (472c40e) 90.04% compared to head (04303dc) 90.04%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2055   +/-   ##
=======================================
  Coverage   90.04%   90.04%           
=======================================
  Files          72       72           
  Lines        6219     6219           
=======================================
  Hits         5600     5600           
  Misses        619      619           

Impacted Files	Coverage Δ
lib/client.js	96.93% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.