Skip to content

Add functions to opt-out "smart heartbeat" #218

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

parfeon
Copy link
Contributor

@parfeon parfeon commented Jun 13, 2025

feat(auto-heartbeat): add functions to opt-out "smart heartbeat"

Add functions which allow to enable and disable "smart heartbeat" behavior for automatic heartbeat: pubnub_enable_smart_heartbeat and pubnub_disable_smart_heartbeat.

fix(auto-heartbeat): fix dead lock on secondary thread

Fix issue which caused deadlock on secondary thread and as result heartbeat stop.

fix(windows): fix function signature and call

Fix signature of pbntf_init_callback and its usage in Windows dedicated code.

Add functions which allow to enable and disable "smart heartbeat" behavior for automatic heartbeat:
`pubnub_enable_smart_heartbeat` and `pubnub_disable_smart_heartbeat`.

fix(auto-heartbeat): fix dead lock on secondary thread

Fix issue which caused deadlock on secondary thread and as result heartbeat stop.

fix(windows): fix function signature and call

Fix signature of `pbntf_init_callback` and its usage in Windows dedicated code.
@parfeon parfeon self-assigned this Jun 13, 2025
@parfeon parfeon added status: done This issue is considered resolved. priority: medium This PR should be reviewed after all high priority PRs. type: feature This PR contains new feature. labels Jun 13, 2025
Copy link
Contributor

@KGronek-Pubnub KGronek-Pubnub left a comment

Choose a reason for hiding this comment

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

Everything compiles fine on Windows, all Unreal Engine tests passed with those changes, so Windows build works.

@KGronek-Pubnub
Copy link
Contributor

@parfeon before merging please check changes in pubnub_pubsubapi.h and pubnub_pubsubapi.c

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: feature This PR contains new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants