feat: sort order of auth modes

aws-amplify · Oct 18, 2024 · 72ad88b · 72ad88b
1 parent 42526fa
commit 72ad88b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/packages/amplify-graphql-api-construct/src/__tests__/__functional__/metadata.test.ts b/packages/amplify-graphql-api-construct/src/__tests__/__functional__/metadata.test.ts
@@ -332,6 +332,12 @@ describe('metrics metadata', () => {
  `),
  authorizationModes: {
  defaultAuthorizationMode: 'API_KEY',
+ // assert output authorizationModes is sorted correctly
+ // aws_lambda should be last in list
+ lambdaConfig: {
+ function: new NodejsFunction(stack, 'TestAuthorizer', { entry: path.join(__dirname, 'authorizer.ts') }),
+ ttl: Duration.days(7),
+ },
  apiKeyConfig: { expires: Duration.days(7) },
  iamConfig: {
  identityPoolId: 'abc',
@@ -342,10 +348,6 @@ describe('metrics metadata', () => {
  assumedBy: new ArnPrincipal('aws:iam::1234:root'),
  }),
  },
- lambdaConfig: {
- function: new NodejsFunction(stack, 'TestAuthorizer', { entry: path.join(__dirname, 'authorizer.ts') }),
- ttl: Duration.days(7),
- },
  },
  });
  const template = Template.fromStack(stack);

diff --git a/packages/amplify-graphql-api-construct/src/internal/metadata.ts b/packages/amplify-graphql-api-construct/src/internal/metadata.ts
@@ -19,7 +19,8 @@ export const getMetadataAuthorizationModes = (authorizationModes: AuthorizationM
  const authModes = Object.keys(authorizationModes)
  .map((mode) => configKeyToAuthMode[mode])
  // remove values not found in mapping
- .filter((mode) => !!mode);
+ .filter((mode) => !!mode)
+ .sort();
  return authModes.join(',');
 };