Skip to content

feat(iot-dev): Allow users to opt-out of treating routine disconnects as errors#1740

Merged
timtay-microsoft merged 5 commits into
mainfrom
timtay/logging
Sep 18, 2023
Merged

feat(iot-dev): Allow users to opt-out of treating routine disconnects as errors#1740
timtay-microsoft merged 5 commits into
mainfrom
timtay/logging

Conversation

@timtay-microsoft
Copy link
Copy Markdown
Member

SAS authorized MQTT connections have routine disconnects that some users don't want treated as errors or warnings in logs. This adds a flag they can set in client options to opt out of the current default.

Also defer creating a multiplexing retry policy until a multiplexing client is created to avoid unnecessary logs

Like #1739 but requiring opting in to logging these routine disconnects opt-in

… as errors

SAS authorized MQTT connections have routine disconnects that some users don't want treated as errors or warnings in logs. This adds a flag they can set in client options to opt out of the current default.

Also defer creating a multiplexing retry policy until a multiplexing client is created to avoid unnecessary logs
@timtay-microsoft
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@Azure Azure deleted a comment from azure-pipelines Bot Sep 12, 2023
@timtay-microsoft
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@timtay-microsoft
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@timtay-microsoft timtay-microsoft merged commit d0bbb65 into main Sep 18, 2023
@timtay-microsoft timtay-microsoft deleted the timtay/logging branch September 18, 2023 14:12
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.

2 participants