-
Notifications
You must be signed in to change notification settings - Fork 3.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(elasticloadbalancing): rename 'sslCertificateId' property of LB listener to 'sslCertificateArn'; deprecate sslCertificateId property #13766
Conversation
…cateId to README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution! I made a few minor suggestions for improvements.
packages/@aws-cdk/aws-elasticloadbalancing/lib/load-balancer.ts
Outdated
Show resolved
Hide resolved
packages/@aws-cdk/aws-elasticloadbalancing/lib/load-balancer.ts
Outdated
Show resolved
Hide resolved
packages/@aws-cdk/aws-elasticloadbalancing/lib/load-balancer.ts
Outdated
Show resolved
Hide resolved
…to using sslCertificateArn
…n when both sslCertificateId and sslCertificateArn are used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks!
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
@njlynch, looks like the build has failed - is there something that I need to do to resolve this issue? Thanks |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
Our |
…listener to 'sslCertificateArn'; deprecate sslCertificateId property (aws#13766) The property `sslCertificateId` of the LoadBalancer listener actually means sslCertificateArn. So as suggested in aws#9303, I have deprecated `sslCertificateId` and replaced it by `sslCertificateArn` to better reflect its actual meaning. fixes aws#9303
The property
sslCertificateId
of the LoadBalancer listener actually means sslCertificateArn. So as suggested in #9303, I have deprecatedsslCertificateId
and replaced it bysslCertificateArn
to better reflect its actual meaning.fixes #9303