Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Record the encoding in a metric if there is an encrypted payload #166

Merged
merged 1 commit into from
Jun 29, 2020

Conversation

AzureMarker
Copy link
Contributor

@AzureMarker AzureMarker commented Jun 23, 2020

This improves parity with Python autoendpoint.

jrconlin
jrconlin previously approved these changes Jun 23, 2020
Base automatically changed from feat/autoendpoint-subscription-auth to master June 24, 2020 18:57
@pjenvey pjenvey dismissed jrconlin’s stale review June 24, 2020 18:57

The base branch was changed.

@AzureMarker AzureMarker force-pushed the feat/autoendpoint-encoding-metric branch from e7690ae to 962383a Compare June 24, 2020 20:39
@AzureMarker AzureMarker marked this pull request as ready for review June 24, 2020 20:39
@AzureMarker AzureMarker requested a review from jrconlin June 24, 2020 20:39
jrconlin
jrconlin previously approved these changes Jun 24, 2020
@AzureMarker
Copy link
Contributor Author

Going to let #154 merge first before rebasing this PR.

This improves parity with Python autoendpoint.
@AzureMarker AzureMarker force-pushed the feat/autoendpoint-encoding-metric branch from 962383a to e9d4ea6 Compare June 26, 2020 13:36
@AzureMarker AzureMarker requested a review from jrconlin June 26, 2020 13:36
@AzureMarker AzureMarker merged commit 8451d3f into master Jun 29, 2020
@AzureMarker AzureMarker deleted the feat/autoendpoint-encoding-metric branch June 29, 2020 15:03
This was referenced Oct 16, 2020
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