Problem/Motivation

Currently the eu-central-1 region is hard-coded, this could/should be made configurable.

Proposed resolution

Patch supplied.

CommentFileSizeAuthor
#4 aws-region-bucket-prefix-combined.patch2.63 KBthoward216
#2 prefix-unrequired.patch621 bytes8ballsteve
aws-s3-region.patch2.68 KB8ballsteve

Issue fork tome_aws-3274677

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

8ballsteve created an issue. See original summary.

8ballsteve’s picture

8ballsteve commented
StatusFileSize
new prefix-unrequired.patch621 bytes

The prefix probably shouldn't be required as it's a valid usecase to upload directly to an S3 bucket.

thoward216 made their first commit to this issue’s fork.

thoward216’s picture

thoward216 commented
StatusFileSize
new aws-region-bucket-prefix-combined.patch2.63 KB

Combined patch as they both touch the same line.

GuillaumeG made their first commit to this issue’s fork.

GuillaumeG changed the visibility of the branch 3274677-support-multiple-aws-regions to hidden.

guillaumeg’s picture

guillaumeg commented

Thanks both.

Patch applied and merged https://git.drupalcode.org/project/tome_aws/-/merge_requests/2.

guillaumeg’s picture

guillaumeg commented
Status: Active » Fixed
taggartj’s picture

taggartj commented
Status: Fixed » Closed (fixed)

In next release 1.0.9