Closed bretambrose closed 1 year ago
Patch coverage: 75.00%
and project coverage change: +0.11%
:tada:
Comparison is base (
b7a323d
) 82.12% compared to head (51368c0
) 82.24%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
$share/<share-name>
to the iot core validation utility functions. Adds testing for the additional functionality.Leaves the validation utility functions intact. We have determined these were not necessary in MQTT5 (and unnecessary client-side validation should always be avoided for future-compatibility reasons) as IoT Core now returns sensible reason codes that help us diagnose these issues (when using MQTT5; MQTT311 continues to be impacted due to the limitations of the spec). We may revisit and use this functionality as an opt-in setting on the 311 client to help prevent/diagnose particular problems.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.