Skip to content

Fix: rendering in worker error bug [INS-5482] - #8695

Merged
cwangsmv merged 2 commits into
developfrom
fix/worker-templating-render-error-issue
May 13, 2025
Merged

Fix: rendering in worker error bug [INS-5482]#8695
cwangsmv merged 2 commits into
developfrom
fix/worker-templating-render-error-issue

Conversation

@cwangsmv

@cwangsmv cwangsmv commented May 8, 2025

Copy link
Copy Markdown
Contributor

Problem
Using a environment variable with a invalid nunjck tag in OAuth:
Screenshot 2025-05-08 at 14 21 46
Expected error message that shoud displayed
Screenshot 2025-05-08 at 14 21 34
Actually result when click Fetch Tokens
Screenshot 2025-05-08 at 14 32 05

Changes
The templating-handler is initialized RenderError with the error object, which should be the string type of error message content.
So components using the err.message to render will throw unexpected React child error.
Change to initialize RenderError with err.message instead. Also add path and location attribute if exists

@cwangsmv
cwangsmv requested a review from jackkav May 8, 2025 06:44
@cwangsmv
cwangsmv force-pushed the fix/worker-templating-render-error-issue branch from a15eeaa to 31ef809 Compare May 13, 2025 09:42
@cwangsmv
cwangsmv enabled auto-merge (squash) May 13, 2025 09:42
@cwangsmv
cwangsmv merged commit 8f91dda into develop May 13, 2025
@cwangsmv
cwangsmv deleted the fix/worker-templating-render-error-issue branch May 13, 2025 09:55
cwangsmv added a commit that referenced this pull request Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants