-
Notifications
You must be signed in to change notification settings - Fork 80
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
feat: fix signed url mismatch in BlobWriteChannel #915
Conversation
There was a problem hiding this comment.
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
logging.properties
Outdated
java.util.logging.ConsoleHandler.level = CONFIG | ||
|
||
# Set up logging of HTTP requests and responses (uncomment "level" to show) | ||
com.google.api.client.http.level = CONFIG |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leftover file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup! Oops
🤖 I have created a release \*beep\* \*boop\* --- ## [1.118.0](https://github.com/googleapis/java-storage/compare/v1.117.1...v1.118.0) (2021-07-13) ### Features * fix signed url mismatch in BlobWriteChannel ([#915](https://github.com/googleapis/java-storage/issues/915)) ([8b05867](https://github.com/googleapis/java-storage/commit/8b0586757523cfc550c62ff264eea3eebbd7f32e)) ### Bug Fixes * correct lastChunk retry logic in BlobWriteChannel ([#918](https://github.com/googleapis/java-storage/issues/918)) ([ab0228c](https://github.com/googleapis/java-storage/commit/ab0228c95df831d79f4a9c993908e5700dab5aa7)) ### Dependencies * update dependency com.google.apis:google-api-services-storage to v1-rev20210127-1.32.1 ([#910](https://github.com/googleapis/java-storage/issues/910)) ([2c54acc](https://github.com/googleapis/java-storage/commit/2c54acca0653a96773ab3606a8d97299e9fdf045)) * update kms.version to v0.90.0 ([#911](https://github.com/googleapis/java-storage/issues/911)) ([1050725](https://github.com/googleapis/java-storage/commit/1050725c91b4375340ba113568ba04538c7f52fc)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
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