Skip to content

Commit

Permalink
Merge pull request #364 from GemeenteNijmegen/acceptance
Browse files Browse the repository at this point in the history
feat: use temporary IAM credentials instead of IAM user
  • Loading branch information
marnixdessing authored May 14, 2024
2 parents 9b1d9db + 6eb7971 commit 7190f08
Show file tree
Hide file tree
Showing 3 changed files with 182 additions and 185 deletions.
4 changes: 2 additions & 2 deletions package.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 2 additions & 3 deletions src/Configuration.ts
Original file line number Diff line number Diff line change
Expand Up @@ -82,8 +82,7 @@ export const configurations: { [key: string]: Configuration } = {
deployFromEnvironment: Statics.deploymentEnvironment,
deployToEnvironment: Statics.acceptanceEnvironment,
sessionEndpointAllowList: [
'arn:aws:iam::699363516011:role/yivi-issue-api-api-stack-yiviissueissuefunctionlam-1AAD4D76XTRK5', // webapp new lz
'arn:aws:iam::315037222840:role/yivi-issue-api-api-stack-yiviissueissuefunctionlam-13P8QZIPPJXI1', // webapp old lz
'arn:aws:iam::699363516011:role/yivi-issue-api-api-stack-yiviissueissuefunctionlam-1AAD4D76XTRK5',
],
sessionEndpointIamUser: true,
useSpotInstances: true,
Expand All @@ -98,7 +97,7 @@ export const configurations: { [key: string]: Configuration } = {
deployFromEnvironment: Statics.deploymentEnvironment,
deployToEnvironment: Statics.productionEnvironment,
sessionEndpointAllowList: [
// This cannot be used currently (webapp does not offer support yet)
'arn:aws:iam::185512167111:role/yivi-issue-api-api-stack-yiviissueissuefunctionlam-TFTCG5EPAX36',
],
sessionEndpointIamUser: true,
criticality: new Criticality('high'),
Expand Down
Loading

0 comments on commit 7190f08

Please sign in to comment.