Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

storage.bucket.Bucket: add direct support for additional resource fields #314

Copy link
Copy link
Closed
@tseaver

Description

@tseaver
Issue body actions

The Bucket API resource representation defines a number of fields as "writable", but which we don't already map:

  • cors[]
  • lifecycle{}
  • location
  • logging{}
  • versioning{}

The support for the website{} field is slightly odd (Bucket.configure_website / Bucket.disable_website), and should likely be normalized with support for the other fields.

Bucket.from_dict currently captures them all in the key's metadata attribute, but that name creates confusion with its metadata key (which stores application-defined key-value pairs).

I think it would be cleaner to map each the non-user-defined fields onto separate attributes, and propagate them to the server via the insert, and patch or update endpoints.

Bucket should also expose read-only properties for the non-writable resource fields not already mapped, and which aren't constants:

  • etag
  • id
  • metageneration
  • name
  • owner{}
  • projectNumber
  • selfLink
  • storageClass
  • timeCreated

Metadata

Metadata

Assignees

Labels

api: storageIssues related to the Cloud Storage API.Issues related to the Cloud Storage API.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions

    Morty Proxy This is a proxified and sanitized view of the page, visit original site.