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

Fix presence leave call on used channels #425

Merged
merged 3 commits into from
Dec 12, 2024
Merged

Fix presence leave call on used channels #425

merged 3 commits into from
Dec 12, 2024

Conversation

parfeon
Copy link
Contributor

@parfeon parfeon commented Dec 12, 2024

fix(presence): fix presence leave call on used channels

Fix issue with Subscription and SubscriptionSet when one can unsubscribe channel / group which is still in use by another.

fix(network): fix fetch resource error report as bad request

Fix particular TypeError emitted when browser forcefully closes long-poll connection before its timeout and reported as bad request. This type of error will be reported as a network error.

fix(node-fetch): keepAlive for Node.js 19+

Fix issue because of which node-fetch used default agent, which after Node.js 19+ has keepAlive enabled by default.

Fix issue with `Subscription` and `SubscriptionSet` when one can unsubscribe channel / group which is still in use by another.

fix(network): fix fetch resource error report as bad request

Fix particular `TypeError` emitted when browser forcefully close long-poll connection before its timeout and reported as bad request. This type of error will be reported as network error.
@parfeon parfeon 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. labels Dec 12, 2024
@parfeon parfeon self-assigned this Dec 12, 2024
@@ -68,9 +69,18 @@ export abstract class SubscribeCapable {
* Stop real-time events processing.
*/
unsubscribe() {
this.pubnub.unregisterSubscribeCapable(this);
Copy link
Contributor

Choose a reason for hiding this comment

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

👍🏻

*
* @internal
*/
public unregisterSubscribeCapable(subscribeCapable: SubscribeCapable) {
Copy link
Contributor

Choose a reason for hiding this comment

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

👍🏻

@parfeon
Copy link
Contributor Author

parfeon commented Dec 12, 2024

@pubnub-release-bot release

@parfeon parfeon merged commit d9af3ec into master Dec 12, 2024
7 checks passed
@parfeon parfeon deleted the CLEN-2433 branch December 12, 2024 09:13
@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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants