If there is a bucket in the S3 account specified on admin/settings/video/filesystem that has a very large number of objects (I have some with tens of thousands) the page will timeout or a memory limit error will occur.
This is happening because of the object count function implemented in the plugins/video_s3/filesystem/video_s3.inc file when you are generating a table of all of the available buckets in the the Amazon S3 Information fieldset. I'm not sure why there needs to be a count of all of the objects here, as it has nothing to do with letting someone know what buckets are available, which appears to be the reason for this table.
I will submit a patch that removes the count and the related table header and data.

CommentFileSizeAuthor
#1 object_count-1340464-1.patch1.08 KBcarsonblack

Comments

carsonblack’s picture

carsonblack commented
Status: Active » Needs review
StatusFileSize
new object_count-1340464-1.patch1.08 KB

Here's the patch.

Jorrit’s picture

Jorrit commented
Assigned: Unassigned » Jorrit

I did not know that that count caused problems. I also don't see a reason for it to be there. If I can't find a memory / time efficient way of counting the objects, I will apply the patch you provided. There is one problem with your patch, you should also modify the rows[] statement at line 191.

Jorrit’s picture

Jorrit commented
Status: Needs review » Fixed

I have applied the patch to 6.x-4.x.

carsonblack’s picture

carsonblack commented

Thanks! Are you talking about the 'Unsupported Bucket Name' thing? If so, do you want me to roll another patch removing that?

Jorrit’s picture

Jorrit commented

Yes, that's what I meant. But there is no need for you to create another patch, I have fixed this myself. It should be in todays -dev release.

Status: Fixed » Closed (fixed)

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