Skip to content

Commit 50934da

Browse files
authored
fix: add required fields to mobile config (#670)
* fix: add required fields to mobile config * more fields * more fields * cs
1 parent 5cc2522 commit 50934da

File tree

4 files changed

+80
-3
lines changed

4 files changed

+80
-3
lines changed

.changeset/spicy-carrots-shave.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@aws-amplify/client-config': patch
3+
---
4+
5+
include additional fields in mobile config generation

packages/client-config/src/client-config-types/mobile/client_config_mobile_types.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,15 @@ export type ClientConfigMobileAuth = {
4343
Auth: {
4444
Default: {
4545
authenticationFlowType: 'USER_SRP_AUTH';
46+
mfaConfiguration: string | undefined;
47+
mfaTypes: Array<string> | undefined;
48+
passwordProtectionSettings: {
49+
passwordPolicyMinLength: number | undefined;
50+
passwordPolicyCharacters: Array<string>;
51+
};
52+
signupAttributes: Array<string>;
53+
usernameAttributes: Array<string>;
54+
verificationMechanisms: Array<string>;
4655
};
4756
};
4857
AppSync?: {

packages/client-config/src/client-config-writer/client_config_converter.test.ts

Lines changed: 49 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,25 @@ void describe('client config converter', () => {
1818
aws_cognito_region: 'test_cognito_region',
1919
aws_user_pools_id: 'test_user_pool_id',
2020
aws_user_pools_web_client_id: 'test_user_pool_app_client_id',
21+
aws_cognito_identity_pool_id: 'test_identity_pool_id',
22+
aws_cognito_signup_attributes: [
23+
'test_signup_attribute_1',
24+
'test_signup_attribute_2',
25+
],
26+
aws_cognito_username_attributes: [
27+
'test_username_attribute_1',
28+
'test_username_attribute_2',
29+
],
30+
aws_cognito_password_protection_settings: {
31+
passwordPolicyMinLength: 1234,
32+
passwordPolicyCharacters: ['a', 'b', 'c'],
33+
},
34+
aws_cognito_verification_mechanisms: [
35+
'test_verification_mechanism_1',
36+
'test_verification_mechanism_2',
37+
],
38+
aws_cognito_mfa_configuration: 'test_mfa_configuration',
39+
aws_cognito_mfa_types: ['test_mfa_type_1', 'test_mfa_type_2'],
2140
};
2241
const expectedMobileConfig: ClientConfigMobile = {
2342
UserAgent: expectedUserAgent,
@@ -37,14 +56,32 @@ void describe('client config converter', () => {
3756
CredentialsProvider: {
3857
CognitoIdentity: {
3958
Default: {
40-
PoolId: 'test_user_pool_id',
59+
PoolId: 'test_identity_pool_id',
4160
Region: 'test_cognito_region',
4261
},
4362
},
4463
},
4564
Auth: {
4665
Default: {
4766
authenticationFlowType: 'USER_SRP_AUTH',
67+
mfaConfiguration: 'test_mfa_configuration',
68+
mfaTypes: ['test_mfa_type_1', 'test_mfa_type_2'],
69+
signupAttributes: [
70+
'test_signup_attribute_1',
71+
'test_signup_attribute_2',
72+
],
73+
usernameAttributes: [
74+
'test_username_attribute_1',
75+
'test_username_attribute_2',
76+
],
77+
passwordProtectionSettings: {
78+
passwordPolicyMinLength: 1234,
79+
passwordPolicyCharacters: ['a', 'b', 'c'],
80+
},
81+
verificationMechanisms: [
82+
'test_verification_mechanism_1',
83+
'test_verification_mechanism_2',
84+
],
4885
},
4986
},
5087
},
@@ -91,6 +128,7 @@ void describe('client config converter', () => {
91128
const clientConfig: ClientConfig = {
92129
aws_cognito_region: 'test_cognito_region',
93130
aws_user_pools_id: 'test_user_pool_id',
131+
aws_cognito_identity_pool_id: 'test_identity_pool_id',
94132
aws_user_pools_web_client_id: 'test_user_pool_app_client_id',
95133
aws_appsync_region: 'test_app_sync_region',
96134
aws_appsync_graphqlEndpoint: 'https://test_api_endpoint.amazon.com',
@@ -115,14 +153,23 @@ void describe('client config converter', () => {
115153
CredentialsProvider: {
116154
CognitoIdentity: {
117155
Default: {
118-
PoolId: 'test_user_pool_id',
156+
PoolId: 'test_identity_pool_id',
119157
Region: 'test_cognito_region',
120158
},
121159
},
122160
},
123161
Auth: {
124162
Default: {
125163
authenticationFlowType: 'USER_SRP_AUTH',
164+
mfaConfiguration: undefined,
165+
mfaTypes: undefined,
166+
signupAttributes: [],
167+
usernameAttributes: [],
168+
passwordProtectionSettings: {
169+
passwordPolicyCharacters: [],
170+
passwordPolicyMinLength: undefined,
171+
},
172+
verificationMechanisms: [],
126173
},
127174
},
128175
AppSync: {

packages/client-config/src/client-config-writer/client_config_converter.ts

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,14 +42,30 @@ export class ClientConfigConverter {
4242
CredentialsProvider: {
4343
CognitoIdentity: {
4444
Default: {
45-
PoolId: clientConfig.aws_user_pools_id,
45+
PoolId: clientConfig.aws_cognito_identity_pool_id,
4646
Region: clientConfig.aws_cognito_region,
4747
},
4848
},
4949
},
5050
Auth: {
5151
Default: {
5252
authenticationFlowType: 'USER_SRP_AUTH',
53+
mfaConfiguration: clientConfig.aws_cognito_mfa_configuration,
54+
mfaTypes: clientConfig.aws_cognito_mfa_types,
55+
passwordProtectionSettings: {
56+
passwordPolicyMinLength:
57+
clientConfig.aws_cognito_password_protection_settings
58+
?.passwordPolicyMinLength,
59+
passwordPolicyCharacters:
60+
clientConfig.aws_cognito_password_protection_settings
61+
?.passwordPolicyCharacters ?? [],
62+
},
63+
signupAttributes:
64+
clientConfig.aws_cognito_signup_attributes ?? [],
65+
usernameAttributes:
66+
clientConfig.aws_cognito_username_attributes ?? [],
67+
verificationMechanisms:
68+
clientConfig.aws_cognito_verification_mechanisms ?? [],
5369
},
5470
},
5571
},

0 commit comments

Comments
 (0)