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

Catch all error events by using empty 'errorCode' does not work #12837

Closed
ralfpuchert opened this issue May 23, 2023 · 2 comments · Fixed by #12840
Closed

Catch all error events by using empty 'errorCode' does not work #12837

ralfpuchert opened this issue May 23, 2023 · 2 comments · Fixed by #12840
Assignees
Labels
kind/bug Categorizes an issue or PR as a bug version:8.2.6 Marks an issue as being completely or in parts released in 8.2.6 version:8.3.0-alpha3 Marks an issue as being completely or in parts released in 8.3.0-alpha3 version:8.3.0 Marks an issue as being completely or in parts released in 8.3.0

Comments

@ralfpuchert
Copy link
Contributor

ralfpuchert commented May 23, 2023

Describe the bug
Using an error catch event (boundary or start event in Subprocess) with an empty errorCode results in incident with errorType=UNHANDLED_ERROR_EVENT.
Also only error events with a non-empty errorCode will be reported like this: Available error events are [boundary].

To Reproduce

  1. Create a BPMN process with error catch event as boundary event or as start event in subprocess.
  2. Deploy, start the process. In a job client throw an error command with an unhandled errorCode.
  3. Zeebe creates an incident instead of catching the error

Here an example process errorProcessCatchAll.bpmn.txt

Slack thread

Expected behavior

The error is caught and no incident is created.

Log/Stacktrace

Example error message from Zeebe:

errorType=UNHANDLED_ERROR_EVENT, errorMessage='Expected to throw an error event with the code 'unknown' with message 'Process error', but it was not caught. Available error events are [boundary]

Environment:

  • OS: MacOS
  • Zeebe Version: 8.2.0-alpha3 and later
  • Configuration: ElasticsearchExporter
@ralfpuchert ralfpuchert added the kind/bug Categorizes an issue or PR as a bug label May 23, 2023
@lzgabel
Copy link
Contributor

lzgabel commented May 23, 2023

Hi @korthout. I think I found the problem, please assign this task to me, I will submit a PR to fix this problem soon. Thanks ❤️

@lzgabel
Copy link
Contributor

lzgabel commented May 25, 2023

BTW. As a workaround, user can catch all error events by removing the errorEventDefinition.

zeebe-bors-camunda bot added a commit that referenced this issue Jun 6, 2023
12958: [Backport stable/8.2] fix(engine): catch-all error events with empty errorCode r=koevskinikola a=backport-action

# Description
Backport of #12840 to `stable/8.2`.

relates to #12837

Co-authored-by: lzgabel <lz19960321lz@gmail.com>
@lenaschoenburg lenaschoenburg added the version:8.2.6 Marks an issue as being completely or in parts released in 8.2.6 label Jun 7, 2023
@Zelldon Zelldon added the version:8.3.0-alpha3 Marks an issue as being completely or in parts released in 8.3.0-alpha3 label Jul 6, 2023
@megglos megglos added the version:8.3.0 Marks an issue as being completely or in parts released in 8.3.0 label Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes an issue or PR as a bug version:8.2.6 Marks an issue as being completely or in parts released in 8.2.6 version:8.3.0-alpha3 Marks an issue as being completely or in parts released in 8.3.0-alpha3 version:8.3.0 Marks an issue as being completely or in parts released in 8.3.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants