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

Implement fetchParams #34741

Open
arthmis opened this issue Dec 22, 2024 · 0 comments
Open

Implement fetchParams #34741

arthmis opened this issue Dec 22, 2024 · 0 comments
Labels
A-network B-feature-tracking This issue tracks a particular high-level feature

Comments

@arthmis
Copy link
Contributor

arthmis commented Dec 22, 2024

fetchParams, spec here, is used as an input to several http fetch functions, specifically

Those functions take a plain request as input, but it would be good to replace that with a minimal implementation of fetchParams only because, the full implementation is a bit more complex. The minimal implementation could look like

struct FetchParams {
    pub request: Request
    // Todo: implement rest of FetchParams with a link to the spec and link to this issue
}

I looked into maybe merging FetchContext with FetchParams but they don't share anything in common except for potentially the timing information and cancellation.

This would help with getting #33161 completed and would bring the fetch implementations a little more in line with the spec.

I would like to take on this task.

@arthmis arthmis added B-feature-tracking This issue tracks a particular high-level feature C-untriaged New issues that haven't been triaged yet labels Dec 22, 2024
@jdm jdm added A-network and removed C-untriaged New issues that haven't been triaged yet labels Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-network B-feature-tracking This issue tracks a particular high-level feature
Projects
None yet
Development

No branches or pull requests

2 participants