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.

Sign up for GitHub

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

Jump to bottom

ErrorMessage metadata - only save the message and not the java exception #3677

Closed
gabriel-farache opened this issue Sep 20, 2024 · 0 comments · Fixed by #3679
Closed

ErrorMessage metadata - only save the message and not the java exception #3677

gabriel-farache opened this issue Sep 20, 2024 · 0 comments · Fixed by #3679
Assignees
@fjtirado

Comments

@gabriel-farache
Copy link
Contributor

Description

Currently, when an errorMessage metadata is set, in the error message saved in the DI and logged in the logs, the java exception is also there which is adding useless noise
image

It would be great and user friendly if only the error message was saved

Implementation ideas

No response
@fjtirado fjtirado self-assigned this Sep 23, 2024
fjtirado added a commit to fjtirado/kogito-runtimes that referenced this issue Sep 23, 2024
@fjtirado
[Fix apache#3677] Removing exception type from error message
ca13f7d
fjtirado added a commit to fjtirado/kogito-runtimes that referenced this issue Sep 23, 2024
@fjtirado
[Fix apache#3677] Removing exception type from error message
f8de3b8
@fjtirado fjtirado mentioned this issue Sep 23, 2024
Merged
fjtirado added a commit to fjtirado/kogito-runtimes that referenced this issue Sep 24, 2024
@fjtirado
[Fix apache#3677] Alternative approach
ff54f84
rgdoliveira pushed a commit to rgdoliveira/kogito-runtimes that referenced this issue Oct 3, 2024
@fjtirado @gmunozfe @rgdoliveira
[Fix apache#3677] Removing exception type from error message (apache#…
d921c51 
…3679)

* [Fix apache#3677] Removing exception type from error message

* Update jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/WorkflowProcessInstanceImpl.java

Co-authored-by: Gonzalo Muñoz <[email protected]>

* [Fix apache#3677] Alternative approach

* [Fix apache#281] Adding specific exception handler

---------

Co-authored-by: Gonzalo Muñoz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fjtirado fjtirado

Labels
None yet
Projects
🦉 KIE Podling Board
Status: 🎯 Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Fix #3677] Removing exception type from error message fjtirado/kogito-runtimes
2 participants
@gabriel-farache @fjtirado