Skip to content

[ServiceBus] remove network trace params from extra in pyamqp logging#34365

Closed
swathipil wants to merge 4 commits intoAzure:mainfrom
swathipil:swathipil/sb/amqp-conn-info-log
Closed

[ServiceBus] remove network trace params from extra in pyamqp logging#34365
swathipil wants to merge 4 commits intoAzure:mainfrom
swathipil:swathipil/sb/amqp-conn-info-log

Conversation

@swathipil
Copy link
Copy Markdown
Member

@swathipil swathipil commented Feb 18, 2024

related to #32190
following #32215

TODO:

  • sync changes with EH pyamqp + README

@kashifkhan
Copy link
Copy Markdown
Member

can we add the same changes in to EH as well please 🙂

Comment thread sdk/eventhub/azure-eventhub/azure/eventhub/_pyamqp/aio/_client_async.py Outdated
@azure-sdk
Copy link
Copy Markdown
Collaborator

API change check

API changes are not detected in this pull request.

@swathipil swathipil force-pushed the swathipil/sb/amqp-conn-info-log branch from c334fae to 8294a6f Compare February 21, 2024 23:50
@swathipil
Copy link
Copy Markdown
Member Author

/azp run python - servicebus - tests

@swathipil
Copy link
Copy Markdown
Member Author

/azp run python - eventhub - tests

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@swathipil
Copy link
Copy Markdown
Member Author

closing in favor of #34458

@swathipil swathipil closed this Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants