Skip to content

Commit 4598829

Browse files
fix: give Get/Put/DeleteObject* perms
1 parent 9ed8ef3 commit 4598829

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

lib/package/s3/compileIamRoleToS3.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ module.exports = {
3636
const permissions = bucketActions.map(({ bucket, action }) => {
3737
return {
3838
Effect: 'Allow',
39-
Action: `s3:${action}`,
39+
Action: `s3:${action}*`, // e.g. PutObject*, GetObject*, DeleteObject*
4040
Resource: {
4141
'Fn::Sub': [
4242
'${bucket}/*',

lib/package/s3/compileIamRoleToS3.test.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ describe('#compileIamRoleToS3()', () => {
9797
},
9898
{
9999
Effect: 'Allow',
100-
Action: 's3:PutObject',
100+
Action: 's3:PutObject*',
101101
Resource: {
102102
'Fn::Sub': [
103103
'${bucket}/*',
@@ -109,7 +109,7 @@ describe('#compileIamRoleToS3()', () => {
109109
},
110110
{
111111
Effect: 'Allow',
112-
Action: 's3:GetObject',
112+
Action: 's3:GetObject*',
113113
Resource: {
114114
'Fn::Sub': [
115115
'${bucket}/*',
@@ -121,7 +121,7 @@ describe('#compileIamRoleToS3()', () => {
121121
},
122122
{
123123
Effect: 'Allow',
124-
Action: 's3:DeleteObject',
124+
Action: 's3:DeleteObject*',
125125
Resource: {
126126
'Fn::Sub': [
127127
'${bucket}/*',
@@ -135,7 +135,7 @@ describe('#compileIamRoleToS3()', () => {
135135
},
136136
{
137137
Effect: 'Allow',
138-
Action: 's3:PutObject',
138+
Action: 's3:PutObject*',
139139
Resource: {
140140
'Fn::Sub': [
141141
'${bucket}/*',

0 commit comments

Comments
 (0)