Skip to content
New issue

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

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

Already on GitHub? # to your account

advancedTLS: Add in deprecated name for transitionary period #7221

Merged
merged 1 commit into from
May 9, 2024

Conversation

gtcooke94
Copy link
Contributor

@gtcooke94 gtcooke94 commented May 9, 2024

I changed RevocationConfig to RevocationOptions in #7151 and missed leaving the old name as a deprecated option so that migrating is easier. This PR adds it back so that we shouldn't be actively breaking code.

RELEASE NOTES: none

@gtcooke94 gtcooke94 added the Type: Internal Cleanup Refactors, etc label May 9, 2024
@gtcooke94 gtcooke94 added this to the 1.64 Release milestone May 9, 2024
@gtcooke94 gtcooke94 requested a review from arvindbr8 May 9, 2024 20:27
Copy link
Member

@arvindbr8 arvindbr8 left a comment

Choose a reason for hiding this comment

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

Thanks. LGTM!

@dfawley dfawley merged commit 3bf7e9a into grpc:master May 9, 2024
11 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 6, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants