Content-Length: 308281 | pFad | http://github.com/angular/angular/pull/61528

55 fix(common): cancel reader when app is destroyed by arturovt · Pull Request #61528 · angular/angular · GitHub
Skip to content

fix(common): cancel reader when app is destroyed #61528

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

Closed
wants to merge 1 commit into from

Conversation

arturovt
Copy link
Contributor

Streams left in a pending state (due to break without cancel) may continue consuming or holding onto data behind the scenes. Calling reader.cancel() allows the browser or the underlying system to release any network or memory resources associated with the stream.

@pullapprove pullapprove bot requested a review from AndrewKushnir May 20, 2025 16:59
@angular-robot angular-robot bot added the area: common Issues related to APIs in the @angular/common package label May 20, 2025
@ngbot ngbot bot added this to the Backlog milestone May 20, 2025
@AndrewKushnir AndrewKushnir requested a review from JeanMeche May 20, 2025 17:50
@AndrewKushnir AndrewKushnir added target: patch This PR is targeted for the next patch release action: review The PR is still awaiting reviews from at least one requested reviewer labels May 20, 2025
@arturovt arturovt force-pushed the fix/common-cancel-reader branch from 29aa693 to 17fa299 Compare May 20, 2025 20:52
Streams left in a pending state (due to `break` without cancel) may continue consuming or holding onto data behind the scenes. Calling `reader.cancel()` allows the browser or the underlying system to release any network or memory resources associated with the stream.
@thePunderWoman thePunderWoman force-pushed the fix/common-cancel-reader branch from 17fa299 to 4a644d1 Compare May 21, 2025 14:22
@thePunderWoman thePunderWoman removed the request for review from AndrewKushnir May 21, 2025 14:27
@thePunderWoman thePunderWoman added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels May 21, 2025
thePunderWoman pushed a commit that referenced this pull request May 21, 2025
Streams left in a pending state (due to `break` without cancel) may continue consuming or holding onto data behind the scenes. Calling `reader.cancel()` allows the browser or the underlying system to release any network or memory resources associated with the stream.

PR Close #61528
thePunderWoman pushed a commit that referenced this pull request May 21, 2025
Streams left in a pending state (due to `break` without cancel) may continue consuming or holding onto data behind the scenes. Calling `reader.cancel()` allows the browser or the underlying system to release any network or memory resources associated with the stream.

PR Close #61528
@thePunderWoman
Copy link
Contributor

This PR was merged into the repository by commit 17fe731.

The changes were merged into the following branches: main, 19.2.x, 20.0.x

@arturovt arturovt deleted the fix/common-cancel-reader branch May 21, 2025 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: common Issues related to APIs in the @angular/common package target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/angular/angular/pull/61528

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy