Skip to content

Remove axios#147

Open
dkoss wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
dkoss:remove-axios
Open

Remove axios#147
dkoss wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
dkoss:remove-axios

Conversation

@dkoss
Copy link
Collaborator

@dkoss dkoss commented Jun 4, 2025

Removes axios dependency, replaces it with node native fetch to fix #142 and #141 and prevent future issues with the dependency.

@dkoss dkoss force-pushed the remove-axios branch 11 times, most recently from 032e48b to db0d12b Compare June 4, 2025 03:11
@@ -11,7 +10,7 @@ jobs:
node: [18, 20]
include:
# use latest npm by default
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: remove comment

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.

Axios vulnerability

2 participants