Skip to content

Commit 04424cf

Browse files
edwardfoyledanielleadams
authored andcommitted
fix: set NoEcho flag on OAuth creds CFN param
1 parent 7727a3f commit 04424cf

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

packages/amplify-category-auth/src/__tests__/provider-utils/awscloudformation/auth-stack-builder/__snapshots__/auth-stack-transform.test.ts.snap

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -788,6 +788,7 @@ Object {
788788
"Type": "String",
789789
},
790790
"hostedUIProviderCreds": Object {
791+
"NoEcho": true,
791792
"Type": "String",
792793
},
793794
"hostedUIProviderMeta": Object {

packages/amplify-category-auth/src/provider-utils/awscloudformation/auth-stack-builder/auth-stack-transform.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -485,6 +485,16 @@ export class AmplifyAuthTransform extends AmplifyCategoryTransform {
485485
}
486486

487487
for (const [key, value] of Object.entries(props)) {
488+
if (key === 'hostedUIProviderCreds') {
489+
this._authTemplateObj.addCfnParameter(
490+
{
491+
type: 'String',
492+
noEcho: true,
493+
},
494+
key,
495+
);
496+
continue;
497+
}
488498
if (typeof value === 'string' || (typeof value === 'object' && !Array.isArray(value))) {
489499
this._authTemplateObj.addCfnParameter(
490500
{

0 commit comments

Comments
 (0)