Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix s3 csharp doc #7027

Merged
merged 5 commits into from Jan 29, 2016
Merged

Fix s3 csharp doc #7027

merged 5 commits into from Jan 29, 2016

Conversation

mathslinux
Copy link
Contributor

small fixes for c# s3 example:

  • some APIs are obsolete and updated by amazon.
  • there are some errors in examples, e.g. missing semicolon, variable name ...

Signed-off-by: Dunrong Huang riegamaths@gmail.com

From http://blogs.aws.amazon.com/net/post/Tx2JT7AYCL2F6CB/ , Amazon.AWSClientFactory was removed

Signed-off-by: Dunrong Huang <riegamaths@gmail.com>
Since v2 and later version, SetACLRequest() and SetACL() were removed,
and replaced by PutACLRequest() and PutACL()

Reference:
http://docs.aws.amazon.com/sdkfornet/latest/apidocs/items/MS3_S3PutACL_PutACLRequestNET4_5.html
http://docs.aws.amazon.com/sdkfornet/v3/apidocs/items/S3/MS3_S3PutACL_PutACLRequest.html

Signed-off-by: Dunrong Huang <riegamaths@gmail.com>
from amazon s3 documentaion, the name shoule be BucketName rather than Bucket.

Signed-off-by: Dunrong Huang <riegamaths@gmail.com>
Signed-off-by: Dunrong Huang <riegamaths@gmail.com>
@mathslinux
Copy link
Contributor Author

@dachary @yehudasa Could you please review/comment?

liewegas added a commit that referenced this pull request Jan 29, 2016
@liewegas liewegas merged commit 1d2948f into ceph:master Jan 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants