chore: upgrade llhttp to 9.2.0

[Uzlopak]

To check if this closes #2678 you can run

import { fetch } from './index-fetch.js'

const response = await fetch('https://u-vpn.unilim.fr/remote/logincheck', {
  headers: {
    accept: '*/*',
    'cache-control': 'no-store, no-cache, must-revalidate',
    'content-type': 'text/plain;charset=UTF-8',
    'if-modified-since': 'Sat, 1 Jan 2000 00:00:00 GMT',
    pragma: 'no-cache',
    Referer: 'https://u-vpn.unilim.fr/remote/login?lang=en',
    'Referrer-Policy': 'strict-origin-when-cross-origin'
  },
  // Fake credentials, still throws the error though
  body: 'ajax=1&username=a&realm=&credential=b',
  method: 'POST'
})

// Only happens here.
console.log(await response.text())

@mcollina

Can your run some benchmarks on this please?

@ShogunPanda

[codecov-commenter]

Codecov Report

Attention: 220 lines in your changes are missing coverage. Please review.

Comparison is base (e39a632) 85.54% compared to head (ff3ed56) 85.28%.
Report is 277 commits behind head on main.

Files	Patch %	Lines
lib/fetch/util.js	60.41%	57 Missing ⚠️
lib/fetch/index.js	66.21%	50 Missing ⚠️
lib/handler/RetryHandler.js	74.35%	30 Missing ⚠️
lib/cache/cache.js	12.12%	29 Missing ⚠️
lib/fetch/dataURL.js	79.31%	12 Missing ⚠️
lib/api/readable.js	83.92%	9 Missing ⚠️
lib/core/diagnostics.js	84.74%	9 Missing ⚠️
lib/eventsource/eventsource.js	96.09%	5 Missing ⚠️
lib/core/request.js	94.36%	4 Missing ⚠️
lib/client.js	95.83%	3 Missing ⚠️
... and 6 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2705      +/-   ##
==========================================
- Coverage   85.54%   85.28%   -0.27%     
==========================================
  Files          76       84       +8     
  Lines        6858     7592     +734     
==========================================
+ Hits         5867     6475     +608     
- Misses        991     1117     +126     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ShogunPanda]

LGTM!

[Uzlopak]

fixed the tests

imho ready for merge.

[Uzlopak]

@mcollina

Maybe we should first release a undici 6.6.2 before we merge this?

[ShogunPanda]

I'm still not sure about the right semver change but yes, let's merge this first.

[Uzlopak]

@ShogunPanda

I took the version from HEAD of the default branch of llhttp. Can you make a release of llhttp? THan we just fix the version in this PR.

[ShogunPanda]

Sure, I'll release llhttp after lunch.

[ShogunPanda]

@Uzlopak llhttp 9.2.0. has just been released. You can quickly rework this PR and then we're ready to go.

[Uzlopak]

@ShogunPanda

Thanks. I just patched the version in the header file.

[ShogunPanda]

LGTM!

[mcollina]

lgtm

I think we should add support/expose the leniency flags, and potentially read those values from node.js core itself. That can be done in a follow-up PR.

[Uzlopak]

@mcollina
@ShogunPanda
@metcoder95
@KhafraDev

Target changed to next branch

[mcollina]

lgtm