Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v7 #3386

Open
ronag opened this issue Jul 3, 2024 · 16 comments
Open

v7 #3386

ronag opened this issue Jul 3, 2024 · 16 comments
Assignees
Labels
semver-major Features or fixes that will be included in the next semver major release

Comments

@ronag
Copy link
Member

ronag commented Jul 3, 2024

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

@ronag ronag added the semver-major Features or fixes that will be included in the next semver major release label Jul 3, 2024
@metcoder95
Copy link
Member

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

@mcollina
Copy link
Member

mcollina commented Jul 3, 2024

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

Yes, I think so.

@Uzlopak
Copy link
Contributor

Uzlopak commented Jul 3, 2024

How can i support?

@KhafraDev
Copy link
Member

  • diagnostics channels

is this something we're absolutely landing?

Also hopefully #2727 is included in the fetch stuff, it's slightly older so I'm hoping it won't get missed.

@metcoder95
Copy link
Member

How can i support?

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

@metcoder95
Copy link
Member

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
Copy link

b2whats commented Jul 21, 2024

Why was dropped interceptors ? What should i use instead ?

@metcoder95
Copy link
Member

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

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 24, 2024

can the npm package next point to v7?

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 24, 2024

image

@mcollina
Copy link
Member

mcollina commented Sep 3, 2024

That's done.


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

@metcoder95
Copy link
Member

Backported some of the work done for interceptors and smaller fixes at core.

@mcollina
Copy link
Member

Do we want to make v7 before v23 is cut? nodejs/Release#1034

@Uzlopak
Copy link
Contributor

Uzlopak commented Sep 19, 2024

Yes.

@trivikr
Copy link
Member

trivikr commented Oct 4, 2024

Is undici@v7 going to drop support for Node.js 18.x?

@mcollina
Copy link
Member

mcollina commented Oct 4, 2024

Not right now. However, if we miss the deadline for Node.js v23 (seems likely), we will definitely be dropping it, as Node.js v18 goes EOL April 2025.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-major Features or fixes that will be included in the next semver major release
Projects
None yet
Development

No branches or pull requests

7 participants