Skip to content

Address security scan reported issues. - #63

Merged
KGronek-Pubnub merged 7 commits into
masterfrom
fix/CLEN-3507_security_fixes
Jun 22, 2026
Merged

KGronek-Pubnub merged 7 commits into
masterfrom
fix/CLEN-3507_security_fixes

Conversation

@KGronek-Pubnub

Copy link
Copy Markdown
Contributor

refactor: Mask SecretKey in logs

refactor: Remove SecretKey from global config (plugin settings), so it's not exposed in .ini files.

refactor: Make SetAuthToken safer, add Async option to queue SetAuthToken with other functions.

fix: Fix C-Core crash during timer-list assert in callback API on Windows when closing or reconnecting during async DnsQueryEx DNS resolution.

@codacy-production

codacy-production Bot commented Jun 22, 2026 •

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 1 minor

Results:
1 new issue

Category Results
Documentation 1 minor

View in Codacy

🟢 Metrics 34 complexity · 2 duplication

Metric Results
Complexity 34
Duplication 2

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@jakub-grzesiowski jakub-grzesiowski left a comment

Copy link
Copy Markdown

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 as 2.1.0

@KGronek-Pubnub
KGronek-Pubnub merged commit fd16f06 into master Jun 22, 2026
3 checks passed
@KGronek-Pubnub
KGronek-Pubnub deleted the fix/CLEN-3507_security_fixes branch June 22, 2026 14:29
@pubnub-release-bot

Copy link
Copy Markdown

🚀 Release successfully completed 🚀

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants