Skip to content

Commit 6c55508

Browse files
Fix/JS (#1016)
* changed shallow copy to deep copy * removed redundant type checking Co-authored-by: hickeydh-aws <hickeydh@amazon.com> Co-authored-by: Brian969 <56414362+Brian969@users.noreply.github.com>
1 parent 5c3122e commit 6c55508

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/deployments/cdk/src/apps/phase-1.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,7 @@ export async function deploy({ acceleratorConfig, accountStacks, accounts, conte
510510
accountIamConfigs[accountKey].users = iamConfig.users;
511511
}
512512
} else {
513-
accountIamConfigs[accountKey] = iamConfig;
513+
accountIamConfigs[accountKey] = IamConfigType.encode(JSON.parse(JSON.stringify(iamConfig)));
514514
}
515515
}
516516

src/deployments/cdk/src/common/iam-assets.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -244,14 +244,14 @@ export class IamAssets extends cdk.Construct {
244244
return iamSSMLogArchiveReadOnlyAccessPolicy;
245245
};
246246

247-
if (!IamConfigType.is(iamConfig)) {
247+
if (!iamConfig) {
248248
console.log(
249249
`IAM config is not defined for account with key - ${accountKey}. Skipping Policies/Users/Roles creation.`,
250250
);
251251
} else {
252252
const iamPolicies = iamConfig.policies;
253253
for (const iamPolicy of iamPolicies!) {
254-
if (!IamPolicyConfigType.is(iamPolicy)) {
254+
if (!iamPolicy) {
255255
console.log(
256256
`IAM config - policies is not defined for account with key - ${accountKey}. Skipping Policies creation.`,
257257
);
@@ -262,7 +262,7 @@ export class IamAssets extends cdk.Construct {
262262

263263
const iamUsers = iamConfig.users;
264264
for (const iamUser of iamUsers!) {
265-
if (!IamUserConfigType.is(iamUser)) {
265+
if (!iamUser) {
266266
console.log(
267267
`IAM config - users is not defined for account with key - ${accountKey}. Skipping Users creation.`,
268268
);
@@ -287,7 +287,7 @@ export class IamAssets extends cdk.Construct {
287287
: undefined;
288288

289289
for (const iamRole of iamRoles) {
290-
if (!IamRoleConfigType.is(iamRole)) {
290+
if (!iamRole) {
291291
console.log(
292292
`IAM config - roles is not defined for account with key - ${accountKey}. Skipping Roles creation.`,
293293
);

0 commit comments

Comments
 (0)