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.

Sign up for GitHub

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

Jump to bottom

Custom fetch implementation #125

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Custom fetch implementation #125

wants to merge 2 commits into from

Conversation

ugo-studio
Copy link

@ugo-studio ugo-studio commented Nov 17, 2024 •
edited
Loading

This PR introduces a custom fetch implementation to enhance flexibility and compatibility in handling HTTP requests.

  • Added support for custom fetch function.
  • Converts Web Standard Response objects to Undici.Response for compatibility.
  • Leverages Undici.request when no custom fetch function is provided.

Why This Change?
This custom implementation provides the following benefits:

  • Flexibility: Developers can use their preferred fetch implementation (e.g., node-fetch, isomorphic-fetch, etc.).
  • Improved Developer Experience: Simplifies HTTP request and response handling while maintaining robust error handling.

@ugo-studio ugo-studio marked this pull request as ready for review November 17, 2024 03:36
@skick1234 skick1234 force-pushed the master branch 2 times, most recently from 01e40c9 to 037acb6 Compare December 17, 2024 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ugo-studio