feat: fix signed url mismatch in BlobWriteChannel by JesseLovelace · Pull Request #915 · googleapis/java-storage · GitHub
Skip to content

feat: fix signed url mismatch in BlobWriteChannel#915

Merged
JesseLovelace merged 2 commits into
masterfrom
issue823
Jul 12, 2021
Merged

feat: fix signed url mismatch in BlobWriteChannel#915
JesseLovelace merged 2 commits into
masterfrom
issue823

Conversation

@JesseLovelace

Copy link
Copy Markdown
Contributor

Use of the "x-goog-api-client" header when created a BlobWriteChannel for a signed url causes a mismatch with URLs generated outside that specific instance of the java client library, removing it makes sense for the intended usage of signed URLs.

Fixes #823

@JesseLovelace JesseLovelace requested review from a team and BenWhitehead July 8, 2021 22:17
@product-auto-label product-auto-label Bot added the api: storage Issues related to the googleapis/java-storage API. label Jul 8, 2021
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Jul 8, 2021

@BenWhitehead BenWhitehead left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ready to merge after cleaning up logging.propeties

Comment thread logging.properties Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Leftover file?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yup! Oops

@JesseLovelace JesseLovelace merged commit 8b05867 into master Jul 12, 2021
@JesseLovelace JesseLovelace deleted the issue823 branch July 12, 2021 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Client Library should stop sending x-goog-api-client, which otherwise needs to be included in a signed URL

2 participants