Skip to content

Fix crashes during unsubscribe. Fix pubnub_fetch_history with crypto module.#214

Merged
KGronek-Pubnub merged 3 commits intomasterfrom
fix/unsubscribe_crashes
May 26, 2025
Merged

Fix crashes during unsubscribe. Fix pubnub_fetch_history with crypto module.#214
KGronek-Pubnub merged 3 commits intomasterfrom
fix/unsubscribe_crashes

Conversation

@KGronek-Pubnub
Copy link
Copy Markdown
Contributor

@KGronek-Pubnub KGronek-Pubnub commented May 22, 2025

fix(unsubscribe): fix crashes that could sometimes happen during unsubscribe

Fix crashes that could sometimes happen during unsubscribe due to calling strlen on null pointers.

fix(fetch_history): fix pubnub_fetch_history function not sending all url parameters when used with crypto api.

Fix pubnub_fetch_history function when used with crypto api. Change PUBNUB_MAX_URL_PARAMS to 12.

fix(auto-heartbeat): fix user ID copy crash

Fix the issue that was causing memory fragmentation fault at the moment of the next automated heartbeat call.

@KGronek-Pubnub KGronek-Pubnub self-assigned this May 22, 2025
@KGronek-Pubnub KGronek-Pubnub added status: done This issue is considered resolved. type: fix This PR contains fixes to existing features. labels May 22, 2025
Fix the issue that was causing memory fragmentation fault at the moment of the next automated
heartbeat call.
Copy link
Copy Markdown
Contributor

@parfeon parfeon left a comment

Choose a reason for hiding this comment

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

LGTM!

@KGronek-Pubnub
Copy link
Copy Markdown
Contributor Author

@pubnub-release-bot release

@KGronek-Pubnub KGronek-Pubnub merged commit 24b3601 into master May 26, 2025
5 checks passed
@KGronek-Pubnub KGronek-Pubnub deleted the fix/unsubscribe_crashes branch May 26, 2025 14:04
@pubnub-release-bot
Copy link
Copy Markdown
Contributor

🚀 Release successfully completed 🚀

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

Labels

status: done This issue is considered resolved. type: fix This PR contains fixes to existing features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants