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

fix: populate etag / generation / metageneration properties during download - #488

Merged
tseaver merged 3 commits into
googleapis:masterfrom
daniellehanks:populate-headers
Jul 6, 2021
Merged

fix: populate etag / generation / metageneration properties during download#488
tseaver merged 3 commits into
googleapis:masterfrom
daniellehanks:populate-headers

Conversation

@daniellehanks

@daniellehanksdaniellehanks commented Jul 6, 2021

Copy link
Copy Markdown
Contributor

Populates additional properties (etag, generation, metageneration) from the corresponding headers (ETag, x-goog-generation, and x-goog-metageneration) during download functions via Blob._extract_headers_from_download.

Additionally, update documentation for running system tests to use Python 3.8 instead of Python 3.7 to align with nox.

Closes#490.

@daniellehanks
daniellehanks requested review from a teamJuly 6, 2021 00:30
@product-auto-labelproduct-auto-labelBot added the api: storage Issues related to the googleapis/python-storage API. label Jul 6, 2021
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Jul 6, 2021
@tseavertseaver changed the title Populate headers on downloadfix: populate etag / generation / metageneration properties during downloadJul 6, 2021

@tseavertseaver 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.

Thanks very much for the patch!

Comment threadCONTRIBUTING.rst
@tseavertseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 6, 2021
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 6, 2021
@tseaver
tseaver merged commit 49ba14c into googleapis:masterJul 6, 2021
gcf-merge-on-greenBot pushed a commit that referenced this pull request Jul 8, 2021
Support conditional requests based on ETag for read operations (`reload`, `exists`, `download_*`). My own testing seems to indicate that the JSON API does not support ETag If-Match/If-None-Match headers on modify requests (`patch`, `delete`, etc.), please correct me if I am mistaken.
This part two of #451. Part one in #488.
Fixes#451 🦕
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
…wnload (googleapis#488)
Populate properties during download functions via Blob._extract_headers_from_download.
Additionally, update documentation for running system tests to use Python 3.8 instead of Python 3.7 to align with nox.
Closesgoogleapis#490.
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Support conditional requests based on ETag for read operations (`reload`, `exists`, `download_*`). My own testing seems to indicate that the JSON API does not support ETag If-Match/If-None-Match headers on modify requests (`patch`, `delete`, etc.), please correct me if I am mistaken.
This part two of googleapis#451. Part one in googleapis#488.
Fixesgoogleapis#451 🦕
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
…wnload (googleapis#488)
Populate properties during download functions via Blob._extract_headers_from_download.
Additionally, update documentation for running system tests to use Python 3.8 instead of Python 3.7 to align with nox.
Closesgoogleapis#490.
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Support conditional requests based on ETag for read operations (`reload`, `exists`, `download_*`). My own testing seems to indicate that the JSON API does not support ETag If-Match/If-None-Match headers on modify requests (`patch`, `delete`, etc.), please correct me if I am mistaken.
This part two of googleapis#451. Part one in googleapis#488.
Fixesgoogleapis#451 🦕
@release-pleaserelease-pleaseBot mentioned this pull request Jan 12, 2022
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Mar 9, 2026
Support conditional requests based on ETag for read operations (`reload`, `exists`, `download_*`). My own testing seems to indicate that the JSON API does not support ETag If-Match/If-None-Match headers on modify requests (`patch`, `delete`, etc.), please correct me if I am mistaken.
This part two of #451. Part one in googleapis/python-storage#488.
Fixes#451 🦕
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storageIssues related to the googleapis/python-storage API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Downloads do not populate properties from all headers

3 participants

@daniellehanks@tseaver@yoshi-kokoro