-
Notifications
You must be signed in to change notification settings - Fork 1k
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: custom SSL CA support for SQL data sources #7802
Conversation
...ld-a-backend/data/connect-to-existing-data-sources/connect-postgres-mysql-database/index.mdx
Outdated
Show resolved
Hide resolved
...ld-a-backend/data/connect-to-existing-data-sources/connect-postgres-mysql-database/index.mdx
Outdated
Show resolved
Hide resolved
``` | ||
|
||
The Lambda function will then use the specified root CA to validate connections to the database. | ||
|
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.
Should we callout that the size of the file must be <= 4 KB?
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.
If you merge in latest from main, it includes a fix for the accessibility violations on this page which will unblock the checks.
…data-sources/connect-postgres-mysql-database/index.mdx Co-authored-by: josef <[email protected]>
…data-sources/connect-postgres-mysql-database/index.mdx Co-authored-by: josef <[email protected]>
0d2e308
to
a5bf1eb
Compare
Force-pushed a rebase from |
Description of changes:
Add documentation for specifying custom SSL CA certificate bundle for SQL Lambda data sources.
Related GitHub issue #, if available:
Instructions
If this PR should not be merged upon approval for any reason, please submit as a DRAFT
This should not be submitted until aws-amplify/amplify-backend#1696 is released.
Which product(s) are affected by this PR (if applicable)?
Which platform(s) are affected by this PR (if applicable)?
Please add the product(s)/platform(s) affected to the PR title
Checks
When this PR is ready to merge, please check the box below
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.