Skip to content

feat(cognito): add issuerType property to UserPool#38290

Draft
emi111y wants to merge 2 commits into
aws:mainfrom
emi111y:feat/cognito-user-pool-issuer-type
Draft

feat(cognito): add issuerType property to UserPool#38290
emi111y wants to merge 2 commits into
aws:mainfrom
emi111y:feat/cognito-user-pool-issuer-type

Conversation

@emi111y

@emi111y emi111y commented Jul 13, 2026

Copy link
Copy Markdown

Summary

Add support for configuring the token issuer URL type on the L2 UserPool construct via
the new issuerType prop. This enables users to opt in to the multi-Region issuer URL
format (UPDATED) without resorting to L1 escape hatches.

Changes

  • Add UserPoolIssuerType enum (ORIGINAL | UPDATED)
  • Add optional issuerType prop to UserPoolProps
  • Wire to L1 issuerConfiguration property on CfnUserPool
  • Add unit tests for both values and absence
  • Add integration test with snapshot
  • Update README with usage example

Usage

new cognito.UserPool(this, 'UserPool', {
  issuerType: cognito.UserPoolIssuerType.UPDATED,
});  

Notes

  • No breaking changes — new optional prop with no default behavior change
  • No feature flag needed — purely additive functionality
  • No IAM or security implications

By submitting this pull request, I confirm that my contribution is made under the terms
of the Apache-2.0 license.


Add support for configuring the token issuer URL type on the L2 UserPool
construct. This enables users to opt in to the multi-Region issuer URL
format without resorting to L1 escape hatches.

Uses addPropertyOverride until the L1 code generation includes the
IssuerConfiguration property on CfnUserPool.

closes aws#38286
@github-actions github-actions Bot added the p2 label Jul 13, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team July 13, 2026 02:18
@github-actions github-actions Bot added the beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK label Jul 13, 2026
@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

⚠️ This pull request description does not follow the correct template structure.

PRs without a linked issue will receive lower priority for review and merging. Please update the description to follow the PR template and include a line like Closes #123 in the Issue section. If no existing issue matches your change, create one first.

@aws-cdk-automation aws-cdk-automation left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This review is outdated)

@aws-cdk-automation aws-cdk-automation added the pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. label Jul 13, 2026
Replace addPropertyOverride with direct issuerConfiguration property
on CfnUserPool now that L1 code generation includes it. Add integration
test with snapshot.
@aws-cdk-automation aws-cdk-automation dismissed their stale review July 13, 2026 05:01

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK p2 pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants