Skip to content
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

fix(http): remove brackets from hostname #5731

Open
wants to merge 13 commits into
base: v1.x
Choose a base branch
from

Conversation

Rishi556
Copy link

@Rishi556 Rishi556 commented Jun 7, 2023

@Rishi556
Copy link
Author

@DigitalBrainJS Hey, any chance you could take a look at this please? Axios@1.0.0 up to current broke support IPv6 literals. I've had to resort to using 0.27.2 due to this for certain codebases.

@ssbarnea
Copy link

@jasonsaayman @DigitalBrainJS Any chance you could please look at this one, at least approve it to run on CI to see its outcome. Thanks.

@Rishi556
Copy link
Author

Rishi556 commented Mar 20, 2024

@ssbarnea I've published this under six-axios(https://www.npmjs.com/package/six-axios) for now as I need the support while waiting for someone from the team to be able to look at this PR. Hopefully others can find use of it. You can do "axios": "npm:six-axios@^1.6.8" in your package.json to avoid having to rename everything in your code. Hope this helps!

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 attempts to resolve IPv6 addresses as hostnames
2 participants