Skip to content

common: fix auto-detected projectId assignment#1878

Merged
stephenplusplus merged 2 commits intogoogleapis:masterfrom
stephenplusplus:spp--1877
Dec 12, 2016
Merged

common: fix auto-detected projectId assignment#1878
stephenplusplus merged 2 commits intogoogleapis:masterfrom
stephenplusplus:spp--1877

Conversation

@stephenplusplus
Copy link
Copy Markdown
Contributor

I caught this while testing out a bug. Our code mistakenly considered our projectId placeholder value ({{projectId}}) as a user-set projectId. Additionally, we were not swapping the placeholder for the actual value inside of API requests with nested arrays.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 12, 2016
@stephenplusplus stephenplusplus added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. don't merge and removed cla: yes This human has signed the Contributor License Agreement. labels Dec 12, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 12, 2016
Copy link
Copy Markdown
Contributor

@callmehiphop callmehiphop left a comment

Choose a reason for hiding this comment

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

LGTM

@stephenplusplus stephenplusplus merged commit b1c6b21 into googleapis:master Dec 12, 2016
@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 1988fcd on stephenplusplus:spp--1877 into ** on GoogleCloudPlatform:master**.

GautamSharda pushed a commit that referenced this pull request Mar 11, 2026
…#1878)

* feat: Trusted Private Cloud support, use the universeDomain parameter
feat: auto populate UUID fields where needed
fix: revert changes to streaming retries

Use gapic-generator-typescript v4.4.0.

PiperOrigin-RevId: 603757799

Source-Link: googleapis/googleapis@1a45bf7

Source-Link: googleapis/googleapis-gen@19ca4b4
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMTljYTRiNDVhNTNkMDBjYjdiZGQ5NGI0NDJiNjBiZDIzN2RmZTEyMyJ9

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
sofisl pushed a commit that referenced this pull request Mar 17, 2026
* fix: Disable Universe Domain Check

* chore: compodoc nonsense
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. core type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants