Skip to content

Replace axios library with built-in fetch#711

Merged
angelozerr merged 1 commit intoredhat-developer:masterfrom
datho7561:use-fetch-instead-of-axios
Aug 21, 2024
Merged

Replace axios library with built-in fetch#711
angelozerr merged 1 commit intoredhat-developer:masterfrom
datho7561:use-fetch-instead-of-axios

Conversation

@datho7561
Copy link
Copy Markdown
Contributor

Signed-off-by: David Thompson davthomp@redhat.com

@datho7561 datho7561 force-pushed the use-fetch-instead-of-axios branch 3 times, most recently from c0b2d99 to fb6bee4 Compare April 11, 2024 16:20
@datho7561
Copy link
Copy Markdown
Contributor Author

axios will still be pulled in as a transitive dependency from @redhat-developer/vscode-redhat-telemetry, but I've also opened a PR there to address this.

Signed-off-by: David Thompson <davthomp@redhat.com>
@datho7561 datho7561 force-pushed the use-fetch-instead-of-axios branch from fb6bee4 to 3f4240f Compare August 9, 2024 15:11
@datho7561 datho7561 requested review from angelozerr and removed request for angelozerr August 9, 2024 15:18
@datho7561
Copy link
Copy Markdown
Contributor Author

@angelozerr this might fix the issue with the proxy described in #680 (comment) (This is not the bug that the person filing the issue encountered, but it should be addressed).

@angelozerr
Copy link
Copy Markdown
Contributor

@datho7561 I will test your PR without proxy.

If it is working can I merge it?

@datho7561
Copy link
Copy Markdown
Contributor Author

Okay sounds good

@angelozerr angelozerr added this to the 1.18.0 milestone Aug 21, 2024
@angelozerr
Copy link
Copy Markdown
Contributor

It works like a charm (tested without proxy), thanks @datho7561 !

@angelozerr angelozerr merged commit 0fa1c0e into redhat-developer:master Aug 21, 2024
@datho7561 datho7561 deleted the use-fetch-instead-of-axios branch August 22, 2024 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants