Skip to content

Commit

Permalink
docs: revise docstrings for generate_signed_url (#408)
Browse files Browse the repository at this point in the history
* Adjusting comment for accuracy

The assumed property is `signer_email` and not `service_account_email`.
In nowhere a property called `service_account_email` is read from `credentials`, but `signer_email` is.

* fix: adjusting comment for accuracy

The assumed property is `signer_email` and not `service_account_email`.
In nowhere a property called `service_account_email` is read from `credentials`, but `signer_email` is.

Co-authored-by: cojenco <59401799+cojenco@users.noreply.github.com>
  • Loading branch information
DiegoQueiroz and cojenco committed Apr 19, 2021
1 parent 412ce0b commit f090548
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions google/cloud/storage/_signing.py
Expand Up @@ -277,7 +277,7 @@ def generate_signed_url_v2(
Assumes ``credentials`` implements the
:class:`google.auth.credentials.Signing` interface. Also assumes
``credentials`` has a ``service_account_email`` property which
``credentials`` has a ``signer_email`` property which
identifies the credentials.
.. note::
Expand Down Expand Up @@ -441,7 +441,7 @@ def generate_signed_url_v4(
Assumes ``credentials`` implements the
:class:`google.auth.credentials.Signing` interface. Also assumes
``credentials`` has a ``service_account_email`` property which
``credentials`` has a ``signer_email`` property which
identifies the credentials.
.. note::
Expand Down

0 comments on commit f090548

Please sign in to comment.