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

feat: add 'response_callback' to 'consume' / 'consume_next_chunk'#114

Closed
tseaver wants to merge 2 commits into
googleapis:masterfrom
tseaver:113-add-response_callback-for-downloads
Closed

feat: add 'response_callback' to 'consume' / 'consume_next_chunk'#114
tseaver wants to merge 2 commits into
googleapis:masterfrom
tseaver:113-add-response_callback-for-downloads

Conversation

@tseaver
Copy link
Copy Markdown
Contributor

@tseaver tseaver commented Nov 13, 2019

Allows callers access to the response objects, e.g., to use HTTP headers.

Closes #113.

Allows callers access to the response objects, e.g., to use HTTP headers.

Closes #113.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 13, 2019
@tseaver
Copy link
Copy Markdown
Contributor Author

tseaver commented Jul 23, 2020

@william-silversmith has submitted a PR to python-storage which makes the general-purpose callback strategy unneeded.

@tseaver tseaver closed this Jul 23, 2020
@tseaver tseaver deleted the 113-add-response_callback-for-downloads branch July 23, 2020 18:23
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add 'response_callback' argument to 'consume' / 'consume_next_chunk' methods.

2 participants