New serverless pattern APIGW-APIKey-tenantid-mapping by Lavanya0513 · Pull Request #3124 · aws-samples/serverless-patterns

Lavanya0513 · 2026-06-02T03:14:44Z

Issue #, if available:

Description of changes:
API Gateway usage plan tenant-id mapping

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

… to tenant" This reverts commit 004c876.

…Ikeyauthorization apigw tenantid pattern

parikhudit · 2026-06-07T17:07:55Z

+    // API Gateway
+    const api = new apigateway.RestApi(this, "ApiGateway", {
+      restApiName: "DynamoDB API Key Protected Service",
+      description: "API protected with DynamoDB-based API key authorization",
+    });
+
+    // Token authorizer using Authorization header (Cognito JWT)
+    const lambdaAuthorizer = new apigateway.TokenAuthorizer(this, "TokenAuthorizer", {
+      handler: authorizerFn,
+      identitySource: "method.request.header.Authorization",
+    });
+
+    // Protected endpoint with mock integration
+    const protectedResource = api.root.addResource("protected");
+
+    protectedResource.addMethod(
+      "GET",
+      new apigateway.MockIntegration({
+        integrationResponses: [
+          {
+            statusCode: "200",
+            responseTemplates: {
+              "application/json": '{ "message": "Access granted" }',
+            },
+          },
+        ],
+        passthroughBehavior: apigateway.PassthroughBehavior.NEVER,
+        requestTemplates: {
+          "application/json": '{ "statusCode": 200 }',
+        },
+      }),
+      {
+        authorizer: lambdaAuthorizer,
+        methodResponses: [{ statusCode: "200" }],


The whole point of the pattern as per the README and title is mapping a tenant to an API Gateway usage plan via an API key so throttling is enforced. But the stack creates no apigateway.UsagePlan, no apigateway.ApiKey, no usage-plan to key association, and never sets the REST API's apiKeySource to AUTHORIZER. Returning usageIdentifierKey: apiKey from the authorizer has no effect unless
(a) apiKeySource = AUTHORIZER on the API, and
(b) the returned key value matches an API key that is attached to a usage plan applied to the stage/method.
Without those, the authorizer's API-key lookup is dead code and the pattern does not actually throttle anyone. The security/operational guarantee it advertises is absent.

I have added steps to create and use usage plans while testing. The pattern is not a full solution and I have specified some pre requisites in readme

Added apiKeySource = AUTHORIZER

parikhudit · 2026-06-07T17:08:03Z

+
+    if (!tenantId) {
+      throw new Error("Unauthorized: No tenant ID in claims");
+    }


The authorizer extracts the tenantId by base64url-decoding the middle segment of the JWT and reading custom:tenantId it never validates the token's signature, issuer (iss), audience (aud/client_id), expiry (exp), or token_use. The inline comment ("Cognito token is already validated by API Gateway if needed") is incorrect: a Lambda TOKEN authorizer receives the raw header value and is itself responsible for authenticating the caller (per the Lambda authorizer workflow, step 3 which says "The Lambda function authenticates the caller"). API Gateway does not pre-validate a JWT for a custom authorizer. As written, an attacker can craft an unsigned JWT (header.{"custom:tenantId":"victim-tenant"}.) with any tenant ID, if that tenant exists in the table, the request is authorized and billed/throttled against the victim tenant. This is a full authorization bypass and undermines the pattern's stated security purpose.

Recommended fix: Verify the token against the Cognito User Pool JWKS before trusting any claim. Use aws-jwt-verify (the AWS-published library) and create the verifier outside the handler so the JWKS is cached across invocations.

This would require passing USER_POOL_ID and CLIENT_ID into the function's environment in the stack, and adding aws-jwt-verify as a bundled dependency (do not add it to externalModules)

Added this check

parikhudit · 2026-06-07T17:09:31Z

+    const userPoolClient = userPool.addClient("TenantUserPoolClient", {
+      authFlows: { userPassword: true },
+    });


authFlows: { userPassword: true } enables USER_PASSWORD_AUTH, which sends the raw password to Cognito. AWS recommends USER_SRP_AUTH (Secure Remote Password) so the password is never transmitted. For a demo USER_PASSWORD_AUTH keeps the get-token.js helper simple, but the README/pattern should at least call out that SRP is preferred for production, and ideally default to it, or you may want to use SRP if possible.
Recommended Fix: Prefer authFlows: { userSrp: true } and use an SRP-capable client in the helper, or document the trade-off explicitly in the README,

added instructions in README

parikhudit · 2026-06-07T17:12:02Z

+    const userPool = new cognito.UserPool(this, "TenantUserPool", {
+      selfSignUpEnabled: false,
+      signInAliases: { email: true },
+      customAttributes: {
+        tenantId: new cognito.StringAttribute({ mutable: false }),
+      },
+      removalPolicy: cdk.RemovalPolicy.DESTROY,
+    });


The user pool sets no explicit passwordPolicy, no MFA, and no advanced security mode. Defaults are reasonable for a demo, but as this pattern is for "secure tenant-based", it's recommended to either set a password policy (preferred) / MFA or note these as production hardening steps in the README.
Recommended Fix: Add a passwordPolicy and consider mfa: cognito.Mfa.OPTIONAL

set Password policy

parikhudit · 2026-06-07T17:33:23Z

@@ -0,0 +1,74 @@
+{


The repo's schema-validation workflow expects the metadata file to be named exactly example-pattern.json at the pattern root.

renamed the file

parikhudit · 2026-06-07T17:34:27Z

+    {
+      "name": "Lavanya Tangutur",     
+      "bio": "Lavanya Tangutur serves as a Senior Technical Account Manager at AWS ocused on helping customers build, deploy, and run secure, resilient, and cost-effective workloads on AWS.",
+      "linkedin": "www.linkedin.com/in/lavanyatangutur"


Only LinkedIn handle should be added

removed bio

Bio is good to have. I meant just linkedId handle is needed, not full URL

parikhudit · 2026-06-07T17:35:12Z

+  "authors": [
+    {
+      "name": "Lavanya Tangutur",     
+      "bio": "Lavanya Tangutur serves as a Senior Technical Account Manager at AWS ocused on helping customers build, deploy, and run secure, resilient, and cost-effective workloads on AWS.",


Please add an image URL if you'd like that to appear with name & bio on serverlessland.com

image is not required

parikhudit · 2026-06-07T17:36:01Z

+1. Install dependencies:
+    ```
+    npm install
+    ```


Missing cdk bootstrap step, important for first time users

added cdk bootstrap

parikhudit · 2026-06-07T17:37:27Z

+
+Note the outputs from the CDK deployment process. The output will include the API Gateway URL, DynamoDB table name, Cognito User Pool ID, and User Pool Client ID.
+
+## How it works


The section says the API key is "returned in the authorization context via usageIdentifierKey," but the deployed stack never creates a usage plan or API key, and the test steps put a "apiKey": "my-api-key-123" item into DynamoDB that is shorter than the 20-character minimum for a real API key and is never associated with any plan. So even after following the README, throttling is not clearly demonstrated.

I have added instructions for usage plan and APIkey in the testing section

parikhudit · 2026-06-07T17:40:10Z

Minor naming items: the feature is "Lambda authorizer" (lowercase "authorizer") per AWS docs; first references should be "AWS Lambda", "Amazon API Gateway", "Amazon Cognito", "Amazon DynamoDB" (the README mostly does this well). The pattern folder is apigw-APIKey-tenantid-cdk with mixed-case APIKey repo convention is all-lowercase hyphenated slugs (e.g., apigw-apikey-tenantid-cdk).
Heads up that the example-pattern.json repoURL/projectFolder already use the mixed-case folder, so renaming the folder requires updating those too.

parikhudit · 2026-06-07T17:41:37Z

The script name implies it deploys DynamoDB, but it actually runs cdk deploy for the whole stack with a hardcoded --app override duplicating cdk.json. The README never references it, so it's an undocumented second deploy path that can drift from cdk.json. It also runs npm install redundantly.

This script is not required. so removed it

parikhudit · 2026-06-07T17:44:20Z

package.json > bin points at bin/apigw-dynamodb-apikey-cdk.js (no such file, sources live under src/bin and src/lib). cdk.json app uses npx ts-node ... src/bin/apigw-dynamodb-apikey-cdk.ts (correct). The example-pattern.json templateFile says src/lib/apigw-dynamodb-apikey-stack.ts. The deploy_dynamodb.sh re-specifies the app on the CLI. This scattered/contradictory wiring is confusing and the package.json bin entry is simply wrong.

Recommended Fix: Standardize on the src/-based layout, fix the package.json bin path (or remove it), and ensure cdk.json is the single source of truth for the app command. Remove deploy_dynamodb.sh so there's one documented deploy path.

standardized the file paths

parikhudit · 2026-06-07T17:50:15Z

+import { DynamoDBClient, GetItemCommand } from "@aws-sdk/client-dynamodb";
+const client = new DynamoDBClient();
+
+const TABLE_NAME = process.env.TABLE_NAME;
+
+exports.handler = async (event) => {


The authorizer uses import { DynamoDBClient } ... ECMAScript Modules at the top but exports with exports.handler = ... (CommonJS). Mixing ESM import with CommonJS exports in the same .js file is invalid and may fail at runtime/bundling depending on how esbuild resolves it. NodejsFunction bundles with esbuild, but the handler contract must be consistent i.e. either full ESM (export const handler) or full CommonJS (const { DynamoDBClient } = require(...) + exports.handler).

Current Code/Configuration:

import { DynamoDBClient, GetItemCommand } from "@aws-sdk/client-dynamodb"; // ... exports.handler = async (event) => { ... };

Recommended Fix: Pick one module system. For ESM: export const handler = async (event) => { ... }. For CJS: const { DynamoDBClient, GetItemCommand } = require("@aws-sdk/client-dynamodb");.

implemented

Lavanya0513 and others added 10 commits March 30, 2026 15:11

added new pattern for using Lambda Authorizer to map API Keys to tenant

004c876

Revert "added new pattern for using Lambda Authorizer to map API Keys…

9397873

… to tenant" This reverts commit 004c876.

added new pattern for tenant to apikey

0c7a68f

changes to add cognito

5e69d50

Added jpg file and updated readme

1739bc0

Updated readme

c6e8226

changed link

e78d895

apigw tenantid pattern

6bf4bc5

Merge pull request #1 from Lavanya0513/lavatang-feature-tenantbasedAP…

b048c4e

…Ikeyauthorization apigw tenantid pattern

removed old directory

431e3ee

julianwood assigned julianwood and parikhudit Jun 2, 2026

added bio

c17ba2c

Lavanya0513 mentioned this pull request Jun 2, 2026

API Gateway usage plan tenant-id mapping #3125

Open

parikhudit reviewed Jun 7, 2026

View reviewed changes

parikhudit added requested-changes invalid-example-pattern-file labels Jun 7, 2026

Addressed comments on pull request

455247a

Lavanya0513 added 6 commits June 8, 2026 18:56

added usage plan

717db4c

removed bio

5d50c5b

added proper API key name

caff571

changed folder name

fc92614

Added bio

9d027a4

changed bio

3030a71


		Note the outputs from the CDK deployment process. The output will include the API Gateway URL, DynamoDB table name, Cognito User Pool ID, and User Pool Client ID.

		## How it works

Conversation

Lavanya0513 commented Jun 2, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

parikhudit Jun 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

parikhudit Jun 7, 2026 •

edited

Loading