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

Skip to content

fix(nuxt): call app:error in ssr#28139

Closed
huang-julien wants to merge 1 commit into
mainfrom
fix/error_hook
Closed

fix(nuxt): call app:error in ssr#28139
huang-julien wants to merge 1 commit into
mainfrom
fix/error_hook

Conversation

@huang-julien

@huang-julien huang-julien commented Jul 12, 2024

Copy link
Copy Markdown
Member

πŸ”— Linked issue

πŸ“š Description

This PR reverts partially #20585 .
Currently throwing an error in a vue component does call app:error but not when calling showError . The issue mentionned does not happen anymore (app:error being called twice)

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@github-actions github-actions Bot added 3.x πŸ› bug Something isn't working as expected labels Jul 12, 2024
@huang-julien huang-julien marked this pull request as ready for review July 12, 2024 21:42
@huang-julien huang-julien requested a review from danielroe as a code owner July 12, 2024 21:42
@danielroe

danielroe commented Jul 15, 2024

Copy link
Copy Markdown
Member

Could you share a reproduction of the issue? πŸ™

We should be calling app:error here on SSR:

await ssrContext.nuxt?.hooks.callHook('app:error', _err)

If this isn't being called (because it's not throwing an error in the render function) then we should probably call in the renderer.ts file rather than in showError.

@huang-julien

Copy link
Copy Markdown
Member Author

@huang-julien huang-julien marked this pull request as draft July 15, 2024 20:06
@huang-julien

Copy link
Copy Markdown
Member Author

I think error handling are swallowed due to suspense ....

@huang-julien

Copy link
Copy Markdown
Member Author

closeing in favor of #28162

@danielroe danielroe deleted the fix/error_hook branch July 15, 2024 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.x πŸ› bug Something isn't working as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants