Skip to content

Conversation

@conwuegb
Copy link

@conwuegb conwuegb commented Nov 21, 2025

Updated form-data to patch which removes use of unsafe random function: Advisory

@changeset-bot
Copy link

changeset-bot bot commented Nov 21, 2025

⚠️ No Changeset found

Latest commit: 19fd316

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@apollo-librarian
Copy link

apollo-librarian bot commented Nov 21, 2025

✅ Docs preview has no changes

The preview was not built because there were no changes.

Build ID: cdc0d886c7d43b186a560123
Build Logs: View logs

@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 21, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@conwuegb conwuegb requested a review from a team as a code owner November 21, 2025 23:27
@briannafugate408
Copy link
Contributor

@tninesling - we are wondering why the CI/CD test is asking for a more updated version of Node?

@conwuegb conwuegb changed the title Update form-data dependency Update form-data dependency Nov 22, 2025
@duckki
Copy link
Contributor

duckki commented Nov 22, 2025

Our dependencies must have started requiring Node.js 18+. And 14, 16, 18 are already deprecated. Maybe it's time to deprecate those old node version tests?

@conwuegb
Copy link
Author

@tninesling - we are wondering why the CI/CD test is asking for a more updated version of Node?

For reference, we ran these commands in this order

$ npm install
$ npm update form-data
$ npm audit fix
$ npm run test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants