Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

fix: add a separate DEFAULT_CLIENT_INFO for rest clients#988

Merged
busunkim96 merged 1 commit into
googleapis:masterfrom
jameslynnwu:default-client-info
Sep 10, 2021
Merged

fix: add a separate DEFAULT_CLIENT_INFO for rest clients#988
busunkim96 merged 1 commit into
googleapis:masterfrom
jameslynnwu:default-client-info

Conversation

@jameslynnwu

Copy link
Copy Markdown
Contributor

Previously, DEFAULT_CLIENT_INFO would be used for rest transports. However, if you generated both rest and grpc transports, rest version information would appear in both transport's client_infos. Now, we create our own DEFAULT_CLIENT_INFO for rest clients and copy the common information from the base DEFAULT_CLIENT_INFO.

@jameslynnwu jameslynnwu requested a review from a team September 9, 2021 19:50
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 9, 2021
@jameslynnwu jameslynnwu marked this pull request as draft September 9, 2021 19:53
@tseaver tseaver changed the title Add a separate DEFAULT_CLIENT_INFO for rest clients fix: add a separate DEFAULT_CLIENT_INFO for rest clients Sep 9, 2021
@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 9, 2021
@jameslynnwu jameslynnwu marked this pull request as ready for review September 9, 2021 21:28

@vam-google vam-google left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@busunkim96 busunkim96 merged commit 22ac400 into googleapis:master Sep 10, 2021
@jameslynnwu jameslynnwu deleted the default-client-info branch October 20, 2021 14:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement. kokoro:force-run Add this label to force Kokoro to re-run the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants