Fix #164: Map HTTP error responses for storage onto specific exception classes. by tseaver · Pull Request #355 · googleapis/google-cloud-python

tseaver · 2014-11-07T18:00:25Z

Fixes #164.

coveralls · 2014-11-07T18:03:14Z

Changes Unknown when pulling 89788fb on tseaver:164-add_http_specific_exceptions into * on GoogleCloudPlatform:master*.

gcloud/storage/exceptions.py

tseaver · 2014-11-07T21:37:45Z

@dhermes reworked to recurse over __subclasses__.

coveralls · 2014-11-07T21:39:53Z

Changes Unknown when pulling e322609 on tseaver:164-add_http_specific_exceptions into * on GoogleCloudPlatform:master*.

dhermes · 2014-11-07T22:01:12Z

LGTM

tseaver · 2014-11-07T22:07:50Z

It would be good if we could document the specific exception types better, but the storage API docs for the various endpoints don't normally detail the error responses possible (we've inferred a few in the past).

Fix #164: Map HTTP error responses for storage onto specific exception classes.

@summer-ji-eng

* fix: use full link in comment to fix JSDoc broken link Committer: @summer-ji-eng PiperOrigin-RevId: 440481666 Source-Link: googleapis/googleapis@6a21110 Source-Link: googleapis/googleapis-gen@e6e39af Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZTZlMzlhZmYzM2I5MGE2M2ExOGU5MDgxZTQ4MGIwOGE0MWM1NDhhYyJ9 * 🦉 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>

@summer-ji-eng

* fix: use full link in comment to fix JSDoc broken link Committer: @summer-ji-eng PiperOrigin-RevId: 440481666 Source-Link: googleapis/googleapis@6a21110 Source-Link: googleapis/googleapis-gen@e6e39af Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZTZlMzlhZmYzM2I5MGE2M2ExOGU5MDgxZTQ4MGIwOGE0MWM1NDhhYyJ9 * 🦉 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>

@summer-ji-eng

* fix: use full link in comment to fix JSDoc broken link Committer: @summer-ji-eng PiperOrigin-RevId: 440481666 Source-Link: googleapis/googleapis@6a21110 Source-Link: googleapis/googleapis-gen@e6e39af Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZTZlMzlhZmYzM2I5MGE2M2ExOGU5MDgxZTQ4MGIwOGE0MWM1NDhhYyJ9 * 🦉 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>

@summer-ji-eng

* fix: use full link in comment to fix JSDoc broken link Committer: @summer-ji-eng PiperOrigin-RevId: 440481666 Source-Link: googleapis/googleapis@6a21110 Source-Link: googleapis/googleapis-gen@e6e39af Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZTZlMzlhZmYzM2I5MGE2M2ExOGU5MDgxZTQ4MGIwOGE0MWM1NDhhYyJ9 * 🦉 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>

* chore(deps): update all dependencies * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * revert Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Anthonios Partheniou <partheniou@google.com>

Source-Link: googleapis/synthtool@1b9ad76 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:9db98b055a7f8bd82351238ccaacfd3cda58cdf73012ab58b8da146368330021

* chore(deps): update all dependencies * revert Co-authored-by: Anthonios Partheniou <partheniou@google.com>

)

* feat: new Bytes and File types: POWERPOINT and EXCEL PiperOrigin-RevId: 437260831 Source-Link: googleapis/googleapis@3c34a40 Source-Link: googleapis/googleapis-gen@a403521 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYTQwMzUyMTNlNjg2Yjk2YWZjM2VkNjZkMGQwNDMyOGU0ZmZiZDRkMSJ9 * 🦉 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>

* docs: Add documentation for enums fix: Add context manager return types chore: Update gapic-generator-python to v1.8.1 PiperOrigin-RevId: 503210727 Source-Link: googleapis/googleapis@a391fd1 Source-Link: googleapis/googleapis-gen@0080f83 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMDA4MGY4MzBkZWMzN2MzMzg0MTU3MDgyYmNlMjc5ZTM3MDc5ZWE1OCJ9 * 🦉 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>

…[autoapprove] (#355) Source-Link: googleapis/synthtool@1f37ce7 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:8e84e0e0d71a0d681668461bba02c9e1394c785f31a10ae3470660235b673086 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* feat: Update Compute Engine API to revision 20221011 (#736) Source-Link: googleapis/googleapis@cb8c914 Source-Link: googleapis/googleapis-gen@1174767 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMTE3NDc2NzQ0NmM2NTAyNzFmMzk3ZWM2ZWY1NDEwMWJlNDRjZGNiNCJ9 * 🦉 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>

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

…355)

Source-Link: googleapis/synthtool@453a5d9 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:81ed5ecdfc7cac5b699ba4537376f3563f6f04122c4ec9e735d3b3dc1d43dd32

* fix: Add async context manager return types chore: Mock return_value should not populate oneof message fields chore: Support snippet generation for services that only support REST transport chore: Update gapic-generator-python to v1.11.0 PiperOrigin-RevId: 545430278 Source-Link: googleapis/googleapis@601b532 Source-Link: googleapis/googleapis-gen@b3f18d0 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYjNmMThkMGY2NTYwYTg1NTAyMmZkMDU4ODY1ZTc2MjA0NzlkN2FmOSJ9 * 🦉 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>

Source-Link: https://togithub.com/googleapis/synthtool/commit/30bd01b4ab78bf1b2a425816e15b3e7e090993dd Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:9bc5fa3b62b091f60614c08a7fb4fd1d3e1678e326f34dd66ce1eefb5dc3267b

…pprove] (#355) Source-Link: https://togithub.com/googleapis/synthtool/commit/e3a1277ac35fc88c09db1930533e24292b132ced Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:452901c74a22f9b9a3bd02bce780b8e8805c97270d424684bff809ce5be8c2a2

@summer-ji-eng

* fix: use full link in comment to fix JSDoc broken link Committer: @summer-ji-eng PiperOrigin-RevId: 440481666 Source-Link: googleapis/googleapis@6a21110 Source-Link: googleapis/googleapis-gen@e6e39af Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZTZlMzlhZmYzM2I5MGE2M2ExOGU5MDgxZTQ4MGIwOGE0MWM1NDhhYyJ9 * 🦉 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>

* FIX: remove references to deprecated client.dataset method * fix lint * fix constraints file

Extracted from PR #325. Closes #324.

* build: configure owlbot to copy release configuration files * style * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * delete github action to publish packages to PyPI --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Source-Link: googleapis/synthtool@703554a Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:94961fdc5c9ca6d13530a6a414a49d2f607203168215d074cdb0a1df9ec31c0b

* test: remove python 3.7 and 3.8 from noxfile unit tests * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * undo owlbot change * add owlbot string replacement * correct regex * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * correct regex * modify noxfile --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

…355)

gcr.io/repo-automation-bots/owlbot-python:latest@sha256:4c981a6b6f2b8914a448d7b3a01688365be03e3ed26dfee399a6aa77fb112eaa

Co-authored-by: Carlos de la Guardia <cguardia@yahoo.com>

Map HTTP error responses for storage onto specific exception classes.

89788fb

Fixes #164.

tseaver added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. api: storage Issues related to the Cloud Storage API. labels Nov 7, 2014

dhermes reviewed Nov 7, 2014
View reviewed changes

gcloud/storage/exceptions.py Outdated

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

Recursively walk subclass tree to build code->exception class mapping.

e322609

tseaver added a commit that referenced this pull request Nov 7, 2014

Merge pull request #355 from tseaver/164-add_http_specific_exceptions

efae0ef

Fix #164: Map HTTP error responses for storage onto specific exception classes.

tseaver merged commit efae0ef into googleapis:master Nov 7, 2014

tseaver deleted the 164-add_http_specific_exceptions branch November 7, 2014 22:08

tseaver mentioned this pull request Nov 13, 2014

DISCUSSION: At what level of granularity should we have error classes matching API errors? #214

Closed

parthea added a commit that referenced this pull request Jun 4, 2023

chore(deps): update dependency pytest to v7.1.3 (#355)

866fb0e

* chore(deps): update all dependencies * revert Co-authored-by: Anthonios Partheniou <partheniou@google.com>

parthea pushed a commit that referenced this pull request Jul 6, 2023

chore(deps): update dependency google-api-python-client to v2.58.0 (#355

13a1538

)

parthea pushed a commit that referenced this pull request Sep 20, 2023

chore(deps): update dependency google-cloud-retail to v1.10.2 (#355)

afd2efd

parthea pushed a commit that referenced this pull request Sep 22, 2023

chore(deps): update dependency pytest to v7.2.2 (#355)

a82875f

parthea pushed a commit that referenced this pull request Oct 21, 2023

chore(main): release 2.11.0 (#355)

48c7659

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

parthea pushed a commit that referenced this pull request Oct 21, 2023

chore(deps): update dependency google-cloud-texttospeech to v2.11.0 (#…

3a94034

…355)

parthea pushed a commit that referenced this pull request Sep 18, 2025

FIX: remove references to deprecated client.dataset method (#355)

97ae7c7

* FIX: remove references to deprecated client.dataset method * fix lint * fix constraints file

parthea pushed a commit that referenced this pull request Nov 22, 2025

chore: use templated 'noxfile.py' (#355)

fa9a4dc

Extracted from PR #325. Closes #324.

parthea added a commit that referenced this pull request Nov 24, 2025

fix: allow grpc without grpcio-status (#355)

0652d52

parthea pushed a commit that referenced this pull request Nov 24, 2025

chore: Update setup.py to accept Document AI client library >=3.0.0 (#…

563ba62

…355)

parthea pushed a commit that referenced this pull request Nov 24, 2025

chore: Update blunderbuss (#355)

31344cb

parthea pushed a commit that referenced this pull request Nov 24, 2025

chore: new owl bot post processor docker image (#355)

c345cc5

gcr.io/repo-automation-bots/owlbot-python:latest@sha256:4c981a6b6f2b8914a448d7b3a01688365be03e3ed26dfee399a6aa77fb112eaa

parthea pushed a commit that referenced this pull request Nov 24, 2025

chore(deps): update dependency google-cloud-bigquery to v2.22.0 (#355)

4ce8a12

parthea pushed a commit that referenced this pull request Nov 24, 2025

fix: LocalStructuredProperty keep_keys (#355)

42093bd

Co-authored-by: Carlos de la Guardia <cguardia@yahoo.com>

parthea pushed a commit that referenced this pull request Nov 25, 2025

chore(deps): update dependency google-cloud-bigquery to v2.22.0 (#355)

6fdf232

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix #164: Map HTTP error responses for storage onto specific exception classes.#355

Fix #164: Map HTTP error responses for storage onto specific exception classes.#355
tseaver merged 2 commits intogoogleapis:masterfrom
tseaver:164-add_http_specific_exceptions

tseaver commented Nov 7, 2014

Uh oh!

coveralls commented Nov 7, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

tseaver commented Nov 7, 2014

Uh oh!

coveralls commented Nov 7, 2014

Uh oh!

dhermes commented Nov 7, 2014

Uh oh!

tseaver commented Nov 7, 2014

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

tseaver commented Nov 7, 2014

Uh oh!

coveralls commented Nov 7, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

tseaver commented Nov 7, 2014

Uh oh!

coveralls commented Nov 7, 2014

Uh oh!

dhermes commented Nov 7, 2014

Uh oh!

tseaver commented Nov 7, 2014

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants