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

[release/8.0-preview5] [Blazor] Fix WebView JavaScript boot bug #48594

Merged
merged 2 commits into from
Jun 3, 2023

Conversation

MackinnonBuck
Copy link
Member

@MackinnonBuck MackinnonBuck commented Jun 2, 2023

[Blazor] Fix WebView JavaScript boot bug

Fixes an issue where the WebView IPC receiver starts before the JavaScript dispatcher object gets initialized, preventing all Blazor Hybrid apps from starting.

Description

Due to some recent major changes in this area, a WebView-specific initialization order bug was introduced that prevents Blazor Hybrid apps from booting correctly.

Unblocks dotnet/maui#15415

Customer Impact

Very large. All Blazor Hybrid apps will fail to start without a workaround.

Regression?

  • Yes
  • No

Regressed from .NET 8.0 Preview 4

Risk

  • High
  • Medium
  • Low

The fix is very isolated and unlikely to introduce another bug.

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

@MackinnonBuck MackinnonBuck requested a review from a team as a code owner June 2, 2023 20:55
@ghost ghost added the area-blazor Includes: Blazor, Razor Components label Jun 2, 2023
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-blazor Includes: Blazor, Razor Components label Jun 2, 2023
@MackinnonBuck MackinnonBuck added the Servicing-consider Shiproom approval is required for the issue label Jun 2, 2023
@ghost
Copy link

ghost commented Jun 2, 2023

Hi @MackinnonBuck. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.

To learn more about how to prepare a servicing PR click here.

@mkArtakMSFT mkArtakMSFT added * NO MERGE * Do not merge this PR as long as this label is present. Servicing-approved Shiproom has approved the issue and removed Servicing-consider Shiproom approval is required for the issue labels Jun 2, 2023
@ghost
Copy link

ghost commented Jun 2, 2023

Hi @MackinnonBuck. This PR was just approved to be included in the upcoming servicing release. Somebody from the @dotnet/aspnet-build team will get it merged when the branches are open. Until then, please make sure all the CI checks pass and the PR is reviewed.

@mkArtakMSFT
Copy link
Member

IMPORTANT: we're still waiting for validation of this fix. @MackinnonBuck when you confirm that this is the right fix, please remove the NO MERGE label and merge then.

@MackinnonBuck MackinnonBuck removed the * NO MERGE * Do not merge this PR as long as this label is present. label Jun 2, 2023
@MackinnonBuck
Copy link
Member Author

Completed manual verification.

@mkArtakMSFT mkArtakMSFT enabled auto-merge (squash) June 2, 2023 23:10
@mkArtakMSFT mkArtakMSFT merged commit fe4f0d0 into release/8.0-preview5 Jun 3, 2023
25 checks passed
@mkArtakMSFT mkArtakMSFT deleted the mbuck/fix-webview-boot-preview5 branch June 3, 2023 00:01
@ghost ghost added this to the 8.0-preview5 milestone Jun 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components Servicing-approved Shiproom has approved the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants