I have discovered while working with this module that it uploads the file without considering file_managed uris, which may have already been created. It results in a sql error, so to mitigate I wrote a patch to copy the file to w/ file_create_filename to ensure the uri is always unique before calling _amazons3_cors_file_save if the file uri already exists in file_managed.

I was able to reproduce this by aborting an upload and then editing the node to re-attempt the upload again.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arosboro created an issue. See original summary.

arosboro’s picture

Status: Active » Needs review
FileSize
925 bytes

Here is the patch