Skip to content
This repository was archived by the owner on Oct 15, 2025. It is now read-only.

Conversation

@cyrus2281
Copy link

What is the purpose of this change?

Improve error handling in the SolaceMessaging component to ensure that the original exception is properly raised when creating a persistent receiver fails, making debugging and error handling more effective.

How is this accomplished?

  • fix: improve error handling in SolaceMessaging by raising the original exception
    • Capture the PubSubPlusClientError as variable 'e' instead of letting it pass uncaptured
    • Raise the actual exception 'e' instead of an undefined 'exception' variable

Anything reviews should focus on/be aware of?

The fix addresses a bug where an undefined 'exception' variable was being raised, which would have caused a NameError rather than showing the actual underlying issue.

@cyrus2281 cyrus2281 self-assigned this Oct 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant