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

Fix the "aborted flag" reference #175

Closed
nsatragno opened this issue Nov 10, 2021 · 0 comments · Fixed by #176
Closed

Fix the "aborted flag" reference #175

nsatragno opened this issue Nov 10, 2021 · 0 comments · Fixed by #176
Assignees
@nsatragno

Comments

@nsatragno
Copy link
Member

nsatragno commented Nov 10, 2021 •
edited by equalsJeffH
Loading

On PR #155 I attempted to fix the "aborted flag" reference that was removed by whatwg/dom#1027 to make the CI happy. However, the way I did it was wrong and I should have followed the guidance instead.

See also: whatwg/dom#1030 "AbortSignal's abort reason downstream integration"
nsatragno added a commit to nsatragno/webappsec-credential-management that referenced this issue Nov 10, 2021
@nsatragno
Fix w3c#175
18a0146 
Reject aborted operations with the "abort reason" instead of AbortError.
@nsatragno nsatragno mentioned this issue Nov 10, 2021
Merged
@nsatragno nsatragno self-assigned this Nov 10, 2021
equalsJeffH pushed a commit that referenced this issue Nov 12, 2021
@nsatragno
Fix #175 (#176)
68a9134 
Reject aborted operations with the "abort reason" instead of AbortError.
github-actions bot added a commit that referenced this issue Nov 12, 2021
@nsatragno @github-actions
Fix #175 (#176)
e9828fd 
SHA: 68a9134
Reason: push, by @equalsJeffH

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nsatragno nsatragno

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Reject aborted operations with the abort reason nsatragno/webappsec-credential-management
1 participant
@nsatragno