nodejs / undici

An HTTP/1.1 client, written from scratch for Node.js
https://nodejs.github.io/undici
MIT License
6.01k stars 527 forks source link

v7 #3386

Open ronag opened 2 months ago

ronag commented 2 months ago

We start with merging the next branch into main (@metcoder95) and then start working actively on v7 with a deadline relative to Node v23.

metcoder95 commented 2 months ago

Shall we consider the exposure of the lenient flags from lhttp as part of this?

mcollina commented 2 months ago

Shall we consider the exposure of the lenient flags from lhttp as part of this?

Yes, I think so.

Uzlopak commented 2 months ago

How can i support?

KhafraDev commented 2 months ago
  • diagnostics channels

is this something we're absolutely landing?

Also hopefully https://github.com/nodejs/undici/pull/2727 is included in the fetch stuff, it's slightly older so I'm hoping it won't get missed.

metcoder95 commented 2 months ago

How can i support?

Would you like to support adding the lenient flags for llhttp @Uzlopak?

metcoder95 commented 1 month ago

I was kicking off the work for the lenient flags, but I just noticed that most of them as marked as opening a security vector if enabled; do we want to enable that within undici?

By default they are disabled sure, but do we want to expose the users to that possibility? I'd consider to not do so, cannot imagine under which circumstances (beyond legacy servers) you might want to enable this

b2whats commented 1 month ago

Why was dropped interceptors ? What should i use instead ?

metcoder95 commented 1 month ago

Interceptors hasn't been dropped, but rather refactored into a new API. Check: https://undici.nodejs.org/#/docs/api/Dispatcher?id=dispatchercomposeinterceptors-interceptor

Uzlopak commented 1 week ago

can the npm package next point to v7?

Uzlopak commented 1 week ago

image

mcollina commented 2 days ago

That's done.


On a different topic, I think we might want to release v7 before Node.js v23, to be included there.