derrickchoi / s3fs

Automatically exported from code.google.com/p/s3fs
GNU General Public License v2.0
0 stars 0 forks source link

Patch to add bucket name check (for invalid characters) #163

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
I was caught up recently when I tried to mount my bucket as follows, using the 
s3cmd syntax:

  s3fs s3://mybucket /mnt/mybucket

http://s3tools.org/s3cmd

S3FS was throwing the error that the bucket was not found, which is strictly 
correct, but it could have told me that I was using invalid characters (like it 
does with uppercase characters).

The attached patch checks for '/' and ':' characters in the bucket name and 
bails if it finds them with a helpful error message. Hopefully this saves some 
other user a bit of time trying to figure out why their buckets are not found. 
:)

Original issue reported on code.google.com by mark.the...@gmail.com on 21 Feb 2011 at 4:56

Attachments:

GoogleCodeExporter commented 8 years ago
http://docs.amazonwebservices.com/AmazonS3/latest/dev/ outlines restrictions on 
bucket names  -- rather than just checking for '/' and ':', let's check for 
other illegal characters as well

Original comment by dmoore4...@gmail.com on 26 Feb 2011 at 2:47

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r330.

Original comment by dmoore4...@gmail.com on 26 Feb 2011 at 2:48