Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Sep 27, 2023. It is now read-only.

Conversation

blakeli0
Copy link
Contributor

@blakeli0 blakeli0 commented Feb 4, 2022

There are some PathTemplate patterns that need to be supported for explicit dynamic routing header feature. After testing thoroughly, all the patterns defined in routing.proto are already supported, however, there are a few test cases missing.
This PR is to add those missing test cases.

@blakeli0 blakeli0 requested a review from a team as a code owner February 4, 2022 21:29
Copy link
Contributor

@vam-google vam-google left a comment

Choose a reason for hiding this comment

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

LGTM

@blakeli0 blakeli0 merged commit fb048ca into main Feb 5, 2022
@blakeli0 blakeli0 deleted the add-unit-tests-dynamic-routing-headers branch February 5, 2022 02:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants