Skip to content

Commit

Permalink
docs(elasticloadbalancingv2): add comment to recommended policy. (#32379
Browse files Browse the repository at this point in the history
)

### Issue # (if applicable)

Closes #32307 .

### Reason for this change
Enum SslPolicy doesn't have a TLS13_12.
When I try to add, it was failed caused by the value conflicts with the
exiting value `RECOMMENDED_TLS`.
But User will be confused. So I add a comment.

#32377 (comment)

### Description of changes

add comment to doc. 

### Description of how you validated changes

<!--Have you added any unit tests and/or integration tests?-->

### Checklist
- [x] My code adheres to the [CONTRIBUTING
GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and
[DESIGN
GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

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

---------

Co-authored-by: MasakiYamanaka <nes-m.yamanaka@nec.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
  • Loading branch information
3 people authored Dec 6, 2024
1 parent c78ef1b commit 7d1dd0d
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,8 @@ export enum SslPolicy {
/**
* The recommended security policy for TLS listeners.
* This is the default policy for listeners created using the AWS Management Console
*
* This policy includes TLS 1.3, and is backwards compatible with TLS 1.2
*/
RECOMMENDED_TLS = 'ELBSecurityPolicy-TLS13-1-2-2021-06',

Expand Down

0 comments on commit 7d1dd0d

Please sign in to comment.