Make node v18 the minimum supported version#1661
Make node v18 the minimum supported version#1661joachimvh merged 2 commits intoversions/next-majorfrom
Conversation
e7f016c to
a89de35
Compare
|
Yesterday we encountered a slight nuance in the node 18 HTTP client. We use CSS locally in development and testing, so most There must be a way to avoid node 18 HTTP client making requests to localhost using ipv6 but we ended up pinning node 16 (we use volta) to be able to work on a demo app rather than wrestling with node 🐱 Today I will try the It looks like node 20 already addressed it: |
|
I didn't check the linked issues yet, but shouldn't that also cause issues in the CSS IDP integration tests then as those also use localhost? |
|
I haven't looked at how you have those tests set up. Yesterday we noticed that ClientID document served using EDIT: It might be also possible that in CI env localhost only resolves to IPv4 and has separate I think this PR is perfectly fine, especially since one should be able to pin node 20 when working on localhost. I just wanted to give heads-up that some developers might start encountering |
|
Just a thought — could whatever generates the requests to |
|
@TallTed I think one could choose to use I can confirm that while mentioned issue occurs with node 18, as mentioned in one of the linked issues node 20 uses RFC 8305 (Happy Eyeballs algorithm) and pining to node 20 solved it for me (just as pinning it to node 16 did yesterday). I would just recommend developers pining their node to 20 while they work on localhost, at least to run CSS, volta, and similar tools make it very easy. |
7a85144 to
3d026db
Compare
3d026db to
e7010a8
Compare

📁 Related issues
Closes #1637
✍️ Description
Adds Node 20 testing and makes Node 18 the minimum supported version. The reason for also immediately cutting of Node 16 is that oidc-provider v8 only supports Node 18 and up.