chore: upgrade to node v24 (#3854)#3856
Conversation
- Update .nvmrc
- Update engines in root package.json
- Update @types/node package
- Update all Dockerfiles to use node:24-alpine3.23
- Update workflows/rafiki/env-setup/action.yml to use Node 24
- Fix Invalid URL error (new URL("")) by filtering empty env string arrays
✅ Deploy Preview for brilliant-pasca-3e80ec canceled.
|
github-actions
Bot
added
type: ci
| loggingPlugin, | ||
| ApolloServerPluginDrainHttpServer({ httpServer }) | ||
| ], | ||
| stopOnTerminationSignals: false, |
There was a problem hiding this comment.
Good idea, let's do the same for the auth package (in packages/auth/src/app.ts)
There was a problem hiding this comment.
Sorry this was not meant for this PR, I pushed to my main forgetting I had this PR open. I'll withdraw it. I'll make a separate PR for it once this one is merged.
|
Thank you for the contribution @Mwni! |
|
@Mwni thank you for this contribution 🫶🏻 I'd love to personally welcome you in our Interledger Slack community workspace, where you can ask questions, meet the team and find other ways to contribute and claim prizes (we have a 'scavenger hunt' activity happening next week with swag up for grabs!). Here's the link to join us: https://communityinviter.com/apps/interledger/interledger-working-groups-slack hope to see you there and feel free to DM me! I'm Bibi Souza in there :) |
3 participants
Changes proposed in this pull request
Context
This change updates Rafiki to Node 24 as the supported runtime across local development, CI, and containerized services.
While upgrading, Node 24 surfaced stricter URL/input behavior and type expectations in HTTP mocks. This PR includes targeted fixes so existing configuration parsing and tests continue to work under the new runtime.
fixes #3854
Checklist
fixes #numberuser-docslabel (if necessary)