storage: Add Boto3 List Objects in GCS #2041
Conversation
googlebot
added
the
cla: yes
frankyn
changed the title
frankyn
added
the
do not merge
frankyn
added
kokoro:force-run
storage/s3-sdk/list_gcs_objects.py
Outdated
|
|
||
| # Print object names | ||
| print("Objects:") | ||
| for bucket in response["Contents"]: |
There was a problem hiding this comment.
Since this is listing objects in a bucket, it would be clearer for this to say for object in ... instead of for bucket in .... (Or another word for object, just not bucket.)
There was a problem hiding this comment.
Or "blobs". That's a word I like.
| canonical_headers = '' | ||
| ordered_headers = collections.OrderedDict(sorted(headers.items())) | ||
| for k, v in ordered_headers.items(): | ||
| lower_k = str(k).lower() |
There was a problem hiding this comment.
lower_k and strip_v? name and value would be better variable name.
| strip_v = str(v).lower() | ||
| strip_v = v | ||
| canonical_headers += '{}:{}\n'.format(lower_k, strip_v) | ||
| canonical_headers = "host:storage.googleapis.com\nx-goog-encryption-algorithm:AES56\n" # "x-goog-encryption-key:AAryxNglNkXQY0Wa+h9+7BLSFMhCzPo22MtXUWjOBbI=\nx-goog-encryption-key-sha256:QlCdVONb17U1aCTAjrFvMbnxW/Oul8VAvnG1875WJ3k=\n" |
There was a problem hiding this comment.
Shouldn't this be += instead of =? Otherwise earlier values are lost.
| # [START storage_signed_url_signed_headers] | ||
| signed_headers = '' | ||
| for k, _ in ordered_headers.items(): | ||
| lower_k = str(k).lower() |
There was a problem hiding this comment.
Similar issue with names. Also, these aren't signed_headers, they're names of the headers that need to be signed, so a different name would be more clear. And no need to use items() since you just want the keys, no values.
| lower_k = str(k).lower() | ||
| signed_headers += '{};'.format(lower_k) | ||
| signed_headers = signed_headers[:-1] # remove trailing ';' | ||
| signed_headers = 'host;x-goog-encryption-algorithm' #;x-goog-encryption-key;x-goog-encryption-key-sha256' |
There was a problem hiding this comment.
Again, why the comment? If not used, shouldn't they be removed?
| canonical_query_string = '' | ||
| ordered_query_parameters = collections.OrderedDict( | ||
| sorted(query_parameters.items())) | ||
| for k, v in ordered_query_parameters.items(): |
There was a problem hiding this comment.
Variable names. Something like safe_name, safe_value would be clearer than encoded_k and encoded_v
| parser.add_argument('expiration', help='Expiration Time.') | ||
|
|
||
| args = parser.parse_args() | ||
| headers = {'X-Goog-Encryption-Key': 'AAryxNglNkXQY0Wa+h9+7BLSFMhCzPo22MtXUWjOBbI=', |
There was a problem hiding this comment.
We don't usually include secrets in samples, instead using a placeholder or getting them from command line arguments or environment variables.
frankyn
removed
the
do not merge
3 participants
Hi @engelke,
Sample to list objects in GCS using S3 SDK.