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

Adding additional logging to express why an exception is happening. #26

Merged
merged 2 commits into from
Feb 9, 2024

Conversation

sferich888
Copy link
Collaborator

It is possible for the cloud event from_http method to throw a variety of error messages (which you can't debug because the exception is swallowed by this try/catch or method).

This should extend a error to the user on why exactly 'No CloudEvent available' is happening.

It is possible for the cloud event `from_http` method to throw a variety of error messages (which you can't debug because the exception is swallowed by this try/catch or method). 

This should extend a error to the user on why exactly 'No CloudEvent available' is happening.
@matejvasek
Copy link
Contributor

/approve
/lgtm

@matejvasek matejvasek requested a review from lance February 9, 2024 18:57
Copy link
Member

@lance lance left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

sferich888 added a commit to sferich888/parliament that referenced this pull request Feb 9, 2024
matejvasek pushed a commit that referenced this pull request Feb 9, 2024
@matejvasek matejvasek merged commit fd0503b into boson-project:main Feb 9, 2024
5 checks passed
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