Skip to content

Drop operation identifier from request handler debug logs#234

Merged
jguz-pubnub merged 3 commits into
masterfrom
python-http-ver-logging
Jun 15, 2026
Merged

Drop operation identifier from request handler debug logs#234
jguz-pubnub merged 3 commits into
masterfrom
python-http-ver-logging

Conversation

@jguz-pubnub

@jguz-pubnub jguz-pubnub commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

fix: drop operation identifier from request handler debug logs

@jguz-pubnub jguz-pubnub requested a review from parfeon as a code owner June 12, 2026 09:40
@pubnub-ops-terraform

pubnub-ops-terraform commented Jun 12, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine 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.

@parfeon parfeon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

But do we really need to print it? Python probably will be the only SDK which does it.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jguz-pubnub jguz-pubnub changed the title Log endpoint name instead of numeric operation type Log HTTP protocol version in request handler debug logs Jun 15, 2026
@jguz-pubnub jguz-pubnub changed the title Log HTTP protocol version in request handler debug logs Drop operation identifier from request handler debug logs Jun 15, 2026
@parfeon parfeon self-requested a review June 15, 2026 08:46

@parfeon parfeon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@jguz-pubnub

Copy link
Copy Markdown
Contributor Author

@pubnub-release-bot release

@jguz-pubnub jguz-pubnub merged commit 7ac96cd into master Jun 15, 2026
15 of 20 checks passed
@jguz-pubnub jguz-pubnub deleted the python-http-ver-logging branch June 15, 2026 11:24
@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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants