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

AbortController.timeout max value #36044

Closed
3dyuval opened this issue Sep 25, 2024 · 3 comments · Fixed by #36054
Closed

AbortController.timeout max value #36044

3dyuval opened this issue Sep 25, 2024 · 3 comments · Fixed by #36054
Assignees
Labels
Content:WebAPI Web API docs good first issue A good issue for newcomers to get started with.

Comments

@3dyuval
Copy link

3dyuval commented Sep 25, 2024

MDN URL

https://developer.mozilla.org/en-US/docs/Web/API/AbortSignal/timeout_static

What specific section or headline is this issue about?

Parameters / time

What information was incorrect, unhelpful, or incomplete?

The max value in ms for AbortController.timeout is unknown.

What did you expect to see?

For comparison, setTimeout() has Maximum delay value explicitly specifing the max value is 2,147,483,647ms

Do you have any supporting links, references, or citations?

No,I would like to inquire

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@3dyuval 3dyuval added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Sep 25, 2024
@github-actions github-actions bot added the Content:WebAPI Web API docs label Sep 25, 2024
@Josh-Cena
Copy link
Member

Any value that does not cause it to throw a TypeError behaves as expected. We could clarify that it must be a valid non-negative safe integer.

@Josh-Cena Josh-Cena added good first issue A good issue for newcomers to get started with. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Sep 25, 2024
@BartoszKlonowski
Copy link
Contributor

I will work on this!
@Josh-Cena please assign me if possible, so I can see it on my list.

@Josh-Cena
Copy link
Member

Sure, if it helps you I'm happy to assign.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs good first issue A good issue for newcomers to get started with.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants