Skip to content

Commit 6cb2a0c

Browse files
committed
Elm: Change S3 wildcard away from *
Per Globus' documentation[0], the `s3:ListAllMyBuckets` Action must be allowed for Resource `*`. But in the MinIO version that Elm is using, that throws an error. So instead, switch to the fully-qualified wildcard of `arn:aws:s3:::*`. The root cause is a lack of MinIO support for the `*` wildcard. This was fixed upstream in [1], but Elm has not yet moved to one of the releases that includes the commit. [0]: https://docs.globus.org/premium-storage-connectors/v5.4/aws-s3/#permissions-anchor [1]: minio/pkg@f99581f,
1 parent 66bf104 commit 6cb2a0c

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

assets/js/connect.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ $(document).ready(function() {
146146
let autoEntry = {};
147147
autoEntry["Effect"] = "Allow";
148148
autoEntry["Action"] = "s3:ListAllMyBuckets";
149-
autoEntry["Resource"] = "*";
149+
autoEntry["Resource"] = "arn:aws:s3:::*";
150150
statements.push(autoEntry);
151151
//loop through fields
152152
for (var i = 1; i <= fieldCounter; i++) {
@@ -440,4 +440,4 @@ $(document).ready(function() {
440440
iconTag.removeClass().addClass(iconClass);
441441
}
442442

443-
});
443+
});

0 commit comments

Comments
 (0)