Skip to content

lib: add aborted() utility function #46494

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

Merged
nodejs-github-bot merged 26 commits into from
Feb 7, 2023
Merged

lib: add aborted() utility function #46494

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
6e797f5
lib: add aborted() utility function
debadree25 Feb 4, 2023
3834e9f
fixup! update param validation
debadree25 Feb 4, 2023
1cab0ca
fixup! make async func
debadree25 Feb 4, 2023
2e41a14
fixup! make resource compulsory
debadree25 Feb 4, 2023
0a58ffc
fixup! add doc
debadree25 Feb 4, 2023
f3621fd
fixup! doc fixup
debadree25 Feb 4, 2023
3bfeec0
fixup! doc fixup
debadree25 Feb 4, 2023
a2f2c56
fixup! doc
debadree25 Feb 4, 2023
f076729
Update doc/api/util.md
debadree25 Feb 5, 2023
6334317
Update doc/api/util.md
debadree25 Feb 5, 2023
7ff30c8
Update test/parallel/test-aborted-util.js
debadree25 Feb 5, 2023
f1ca1f7
Update test/parallel/test-aborted-util.js
debadree25 Feb 5, 2023
812c723
Update test/parallel/test-aborted-util.js
debadree25 Feb 5, 2023
b209fe1
fixup
debadree25 Feb 5, 2023
2f633c6
fixup
debadree25 Feb 5, 2023
b03ab4e
fixup! doc
debadree25 Feb 6, 2023
bae2765
fixup! doc
debadree25 Feb 6, 2023
b1a93a5
fixup! ordering
debadree25 Feb 6, 2023
8dfe5b7
fixup! ordering
debadree25 Feb 6, 2023
a5b43fe
fixup! ordering
debadree25 Feb 6, 2023
2401897
fixup! ordering
debadree25 Feb 6, 2023
347c252
fixup! ordering
debadree25 Feb 6, 2023
2446138
fixup! refactor function
debadree25 Feb 6, 2023
2b6929c
Update test/parallel/test-aborted-util.js
debadree25 Feb 6, 2023
a4a7bef
Update test/parallel/test-aborted-util.js
debadree25 Feb 6, 2023
f106f87
fixup! add undefined check
debadree25 Feb 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fixup! add undefined check
  • Loading branch information
@debadree25
debadree25 committed Feb 6, 2023
commit f106f87bfe5b64f15c37e44d1161b0fab688a3d4
3 changes: 3 additions & 0 deletions lib/internal/abort_controller.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -368,6 +368,9 @@ function transferableAbortController() {
* @returns {Promise<void>}
*/
async function aborted(signal, resource) {
if (signal === undefined) {
throw new ERR_INVALID_ARG_TYPE('signal', 'AbortSignal', signal);

@debadree25 debadree25 Feb 6, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have added this check since turns out changing the condition in validateAbortSignal() immediately results in the build breaking, so i assume its probably not a bug but would like a review here

@debadree25 debadree25 Feb 6, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like a common pattern across to the codebase to check for undefined and then validate the signal so i think its probably fine?

}
validateAbortSignal(signal, 'signal');
validateObject(resource, 'resource', { nullable: false, allowFunction: true, allowArray: true });
Comment thread
debadree25 marked this conversation as resolved.
Resolved
if (signal.aborted)
Expand Down