-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Upgrade Axios (#632) #633
base: master
Are you sure you want to change the base?
Upgrade Axios (#632) #633
Conversation
Ping @TheBoroer Would you be able to look into this any time soon? |
@strangelookingnerd oh man I hope @TheBoroer will merge this. Nobody wants any issues with localtunnel. |
@TheBoroer please merge this! |
Since he's not responding, I'm just going to throw this out there: If you're worried about the security in your app and you're using localtunnel, you can upgrade it manually. Just run: npm install axios --save And you should be good to go. |
I gave up waiting so I forked, patched, and published my own version to npm (https://www.npmjs.com/package/@security-patched/localtunnel)
|
+1 |
The reason this sadly isn't being merged is because @TheBoroer doesn't have edit perms on this repo, neither to npm |
Still waiting for the PR to be approved so we can migrate to Angular 17.. |
Fixes #632
Axios has recently patched a critical vulnerability affecting version >= 0.8.1, < 1.6.0