-
Notifications
You must be signed in to change notification settings - Fork 26.4k
feat(http): add keepalive support for fetch requests in httpResource #61833
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@alxhub Is any additional validation needed to be able to merge it? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
reviewed-for: fw-general, public-api
This commit adds support for the Fetch API's keepalive option when using httpResource with the withFetch provider
68629eb
to
30290b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed-for: public-api
This PR was merged into the repository by commit 73269cf. The changes were merged into the following branches: main |
This commit adds support for the Fetch API's keepalive option when using httpResource with the withFetch provider.
The keepalive option is particularly useful for:
Beacon API-like functionality (logging analytics on page unload)
Better control over connection management in long-lived applications
Example new usage :