Skip to content

Add parsed errors to default OutputError message#36

Merged
petergebala merged 1 commit into
masterfrom
update-output-error-message
Feb 19, 2026
Merged

Add parsed errors to default OutputError message#36
petergebala merged 1 commit into
masterfrom
update-output-error-message

Conversation

@msx2

@msx2 msx2 commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

Update default Opera::Operation::Result::OutputError message to include parsed errors as part of the message.

In ProFinda's current stack it's not always easy to see and find specific exceptions, returned as just Cannot retrieve output from a Failure.. DataDog properly logs error params, but we're still lacking visibility in Rollbar, Sidekiq UI and development/test modes.

Change previously discussed with @cintrzyk to improve our error handling workflow.

New examples of the errors output:

Operation failed — output cannot be retrieved.
- example: Example error
Operation failed — output cannot be retrieved.
- example: Example error; Unexpected behavior
- other_example: Hello darkness, my old friend

@msx2 msx2 requested review from cintrzyk and petergebala February 18, 2026 15:38
@petergebala petergebala merged commit bdace2f into master Feb 19, 2026
4 checks passed
@msx2 msx2 deleted the update-output-error-message branch April 1, 2026 12:50
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.

3 participants