Skip to content

Conversation

@mohitpubnub
Copy link
Contributor

fix: subscription not breaking when user provided listener callback throws exception.

Fixes issue of subscription loop breaking when a listener callback throws exception.

@pubnub-ops-terraform
Copy link

pubnub-ops-terraform commented Nov 10, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@mohitpubnub mohitpubnub changed the title improved test coverage fix for subscription to continue after exception from listener callback Nov 10, 2025
@mohitpubnub mohitpubnub self-assigned this Nov 12, 2025
@mohitpubnub mohitpubnub added status: done This issue is considered resolved. priority: medium This PR should be reviewed after all high priority PRs. type: fix This PR contains fixes to existing features. type: test This PR contains new tests for existing functionality or fixes to existing tests. labels Nov 12, 2025
Copy link
Contributor

@jakub-grzesiowski jakub-grzesiowski left a comment

Choose a reason for hiding this comment

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

LGTM

@mohitpubnub
Copy link
Contributor Author

@pubnub-release-bot release

@mohitpubnub mohitpubnub merged commit ece74f6 into master Nov 18, 2025
7 of 8 checks passed
@mohitpubnub mohitpubnub deleted the CLEN-2983 branch November 18, 2025 06:51
@pubnub-release-bot
Copy link
Contributor

🚀 Release successfully completed 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: medium This PR should be reviewed after all high priority PRs. status: done This issue is considered resolved. type: fix This PR contains fixes to existing features. type: test This PR contains new tests for existing functionality or fixes to existing tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants