Skip to content

Comments

feat(ses): auto email validation for configuration sets#36679

Open
badmintoncryer wants to merge 13 commits intoaws:mainfrom
badmintoncryer:ses-config
Open

feat(ses): auto email validation for configuration sets#36679
badmintoncryer wants to merge 13 commits intoaws:mainfrom
badmintoncryer:ses-config

Conversation

@badmintoncryer
Copy link
Contributor

Issue # (if applicable)

None

Reason for this change

AWS SES configuration sets supports for auto email validation.
https://docs.aws.amazon.com/ses/latest/dg/email-validation-auto.html

Description of changes

  • Introduced confidenceVerdictThreshold property in ConfigurationSet to manage suppression list validation.
  • Added support for thresholds: DISABLED, MEDIUM, HIGH, and MANAGED.

This validation function is enabled by ConditionThresholdEnabled property. I decided to add DISABLED option for disable this feature explicitly.

function renderValidationOptions(threshold?: ConfidenceVerdictThreshold): CfnConfigurationSet.ValidationOptionsProperty | undefined {
  if (!threshold) {
    return undefined;
  }

  if (threshold === ConfidenceVerdictThreshold.DISABLED) {
    return {
      conditionThreshold: {
        conditionThresholdEnabled: 'DISABLED',
      },
    };
  }

  return {
    conditionThreshold: {
      conditionThresholdEnabled: 'ENABLED',
      overallConfidenceThreshold: {
        confidenceVerdictThreshold: threshold,
      },
    },
  };
}

Describe any new or updated permissions being added

None

Description of how you validated changes

Add both unit and integ tests.

Checklist


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

- Introduced `confidenceVerdictThreshold` property in `ConfigurationSet` to manage suppression list validation.
- Added support for thresholds: DISABLED, MEDIUM, HIGH, and MANAGED.
- Updated README with examples for configuring confidence verdict thresholds.
- Implemented integration tests for various configurations of confidence verdict thresholds.
- Enhanced unit tests to validate behavior of `ConfigurationSet` with different threshold settings.
@aws-cdk-automation aws-cdk-automation requested a review from a team January 13, 2026 15:33
@github-actions github-actions bot added p2 distinguished-contributor [Pilot] contributed 50+ PRs to the CDK labels Jan 13, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results48 ran48 passed
TestResult
No test annotations available

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results with resolved templates48 ran48 passed
TestResult
No test annotations available

@badmintoncryer badmintoncryer marked this pull request as ready for review January 18, 2026 00:27
@aws-cdk-automation aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Jan 18, 2026
Copy link
Contributor

@lpizzinidev lpizzinidev left a comment

Choose a reason for hiding this comment

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

Ty!.

@aws-cdk-automation aws-cdk-automation added pr/needs-maintainer-review This PR needs a review from a Core Team Member and removed pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. labels Feb 11, 2026
badmintoncryer and others added 2 commits February 12, 2026 12:42
Co-authored-by: Luca Pizzini <lpizzini7@gmail.com>
Co-authored-by: Luca Pizzini <lpizzini7@gmail.com>
@badmintoncryer
Copy link
Contributor Author

@lpizzinidev Thanks always!!

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

Labels

distinguished-contributor [Pilot] contributed 50+ PRs to the CDK p2 pr/needs-maintainer-review This PR needs a review from a Core Team Member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants