Skip to content

Deduplicate channels and groups in request building#173

Merged
parfeon merged 4 commits intomasterfrom
feat/deduplicate-subscribe-heartbeat-requests
Mar 3, 2026
Merged

Deduplicate channels and groups in request building#173
parfeon merged 4 commits intomasterfrom
feat/deduplicate-subscribe-heartbeat-requests

Conversation

@parfeon
Copy link
Contributor

@parfeon parfeon commented Mar 3, 2026

refactor(subscribe): deduplicate channels and groups in request building

Add .uniq calls when building the URL path and channel-group query parameter to prevent duplicate channels and channel groups from leaking into HTTP requests.

refactor(heartbeat): deduplicate channels and groups in request building

Add .uniq calls when building the URL path and channel-group query parameter to prevent duplicate channels and channel groups from leaking into HTTP requests.

Add `.uniq` calls when building the URL path and channel-group query parameter to prevent duplicate
channels and channel groups from leaking into HTTP requests.

feat(heartbeat): deduplicate channels and groups in request building

Add `.uniq` calls when building the URL path and channel-group query parameter to prevent duplicate
channels and channel groups from leaking into HTTP requests.
@parfeon parfeon self-assigned this Mar 3, 2026
@parfeon parfeon added status: done This issue is considered resolved. priority: medium This PR should be reviewed after all high priority PRs. type: refactor This PR contains refactored existing features. labels Mar 3, 2026
@pubnub-ops-terraform
Copy link

pubnub-ops-terraform commented Mar 3, 2026

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.

Update recorded URIs in presence cassettes to reflect that duplicate channels are now removed before
building the request URL.
@parfeon
Copy link
Contributor Author

parfeon commented Mar 3, 2026

@pubnub-release-bot release

@parfeon parfeon merged commit e45d2cc into master Mar 3, 2026
10 checks passed
@parfeon parfeon deleted the feat/deduplicate-subscribe-heartbeat-requests branch March 3, 2026 09:17
@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: refactor This PR contains refactored existing features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants