Code review comment for lp:~dave-cheney/goamz/s3-lower-case-bucket-names

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

LGTM if the us-west situation is sorted out.

https://codereview.appspot.com/6343080/diff/7001/aws/aws.go
File aws/aws.go (right):

https://codereview.appspot.com/6343080/diff/7001/aws/aws.go#newcode50
aws/aws.go:50: false,
I believe both of these are wrong. us-west seems to have the same
software deployment as the other regions.

https://codereview.appspot.com/6343080/

« Back to merge proposal