I am curious why the Amazon S3 portion of the video module says Amazon S3 bucket names cannot contain numbers, but the Amazon S3 console allows me to create buckets with numbers?? On who's end is this error??

(I am going to say Amazon knows what they do/don't allow.....)

Comments

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Active » Closed (won't fix)

its not possible with the S3 class that we are using.

magnusk’s picture

magnusk commented
Title: AmazonS3 Bucket name should allow numbers » Amazon S3 Bucket name should allow numbers
Priority: Major » Normal
Status: Closed (won't fix) » Needs review

In file plugins/video_s3/filesystem/video_s3.inc, function admin_settings_validate
I successfully changed the following code:

      // S3 buckets must contain only lower case alphanumeric characters, dots and dashes.
      if (!preg_match("/^[a-z.-]+$/", $bucket)) {
        form_set_error('amazon_s3_bucket', t('S3 buckets must contain only lower case alphanumeric characters, dots and dashes.'));
      } else {

with a regex that accepts digits

      if (!preg_match("/^[a-z0-9.-]+$/", $bucket)) {
Jorrit’s picture

Jorrit commented
Assigned: Unassigned » Jorrit
Status: Needs review » Reviewed & tested by the community

You're right. This will be fixed in 4.4.

Jorrit’s picture

Jorrit commented
Component: Code » Amazon S3
Jorrit’s picture

Jorrit commented
Status: Reviewed & tested by the community » Fixed

Fixed in 6.x-4.x.

Status: Fixed » Closed (fixed)
Issue tags: -Amazon S3, -video s3

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Amazon S3, +video s3

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.