Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(core): check whether application is destroyed before printing hydration stats #59716

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

Conversation

arturovt
Copy link
Contributor

In this commit, we check whether the application is destroyed before printing hydration stats. The application may be destroyed before it becomes stable, so when the whenStableWithTimeout resolves, the injector might already be in a destroyed state. As a result, calling injector.get would throw an error indicating that the injector has already been destroyed.

@pullapprove pullapprove bot requested a review from kirjs January 25, 2025 20:28
@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Jan 25, 2025
@ngbot ngbot bot added this to the Backlog milestone Jan 25, 2025
@arturovt arturovt force-pushed the fix/core-check-appRef-destroyed-before-hydration-stats branch 2 times, most recently from b8acb29 to 58b0820 Compare January 26, 2025 15:43
@AndrewKushnir AndrewKushnir added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Jan 27, 2025
@AndrewKushnir AndrewKushnir removed the request for review from kirjs January 27, 2025 04:53
…ration stats

In this commit, we check whether the application is destroyed before printing hydration stats. The application may be destroyed before it becomes stable, so when the `whenStableWithTimeout` resolves, the injector might already be in a destroyed state. As a result, calling `injector.get` would throw an error indicating that the injector has already been destroyed.
@arturovt arturovt force-pushed the fix/core-check-appRef-destroyed-before-hydration-stats branch from 58b0820 to 8a32c84 Compare January 27, 2025 12:36
@AndrewKushnir
Copy link
Contributor

Presubmit.

@AndrewKushnir AndrewKushnir added action: presubmit The PR is in need of a google3 presubmit 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 action: presubmit The PR is in need of a google3 presubmit labels Jan 28, 2025
@alxhub
Copy link
Member

alxhub commented Jan 29, 2025

This PR was merged into the repository by commit cf90542.

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

@alxhub alxhub closed this in cf90542 Jan 29, 2025
alxhub pushed a commit that referenced this pull request Jan 29, 2025
…ration stats (#59716)

In this commit, we check whether the application is destroyed before printing hydration stats. The application may be destroyed before it becomes stable, so when the `whenStableWithTimeout` resolves, the injector might already be in a destroyed state. As a result, calling `injector.get` would throw an error indicating that the injector has already been destroyed.

PR Close #59716
@arturovt arturovt deleted the fix/core-check-appRef-destroyed-before-hydration-stats branch January 29, 2025 19:59
PrajaktaB27 pushed a commit to PrajaktaB27/angular that referenced this pull request Feb 7, 2025
…ration stats (angular#59716)

In this commit, we check whether the application is destroyed before printing hydration stats. The application may be destroyed before it becomes stable, so when the `whenStableWithTimeout` resolves, the injector might already be in a destroyed state. As a result, calling `injector.get` would throw an error indicating that the injector has already been destroyed.

PR Close angular#59716
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Mar 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants