Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Suggestion: Combine common parts of SetBlobPropertiesOptions and BlobProperties #78

Closed
jcookems opened this Issue · 1 comment

2 participants

@jcookems

As noted in #76, you will clobber server blob properties when using setBlobProperties unless you first call getBlobProperties and pass that info to setBlobProperties. However, you must copy 5 separate values by hand. That makes is easy for people to miss some, which would lead to data loss in some situations.

I recommend that the common properties be refactored out into a separate class, which can easily be passed from the get to the set. The common properties are:

  • CacheControl
  • ContentEncoding
  • ContentLanguage
  • ContentMD5
  • ContentType
@gcheng

After design discussion, close.

@gcheng gcheng closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.