feat:(typescript/postgres-lambda) #1194
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Fix security group rule direction (allowDefaultPortFrom instead of allowDefaultPortTo) - Improve Lambda to PostgreSQL connection with increased timeout and better logging - Replace custom CloudFormation response handling with cfn-response library - Update PostgreSQL setup function to use standard CloudFormation response pattern - Add proper error handling in setup function
- Update SQL functions to use JSONB casting instead of text for proper JSON handling - Change Lambda invocation type from 'Event' to 'RequestResponse' for synchronous execution - Add parameter group to RDS cluster for custom configuration - Update Aurora PostgreSQL version to 17.4 - Fix code formatting and whitespace in Lambda functions
- Add detailed logging throughout the setup process - Include environment variable logging for better debugging - Add verification step to confirm function creation - Improve error handling with stack traces - Add comments to SQL script for better readability - Include more detailed CloudFormation response data - Add connection timeout for better error handling
- Fix security group rule direction for setup function (allowDefaultPortFrom instead of allowDefaultPortTo) - Add test property to custom resource to force update on deployment
- Remove setup-postgres-lambda.sql as PostgreSQL setup is now automated via Lambda - Remove test-lambda.sh as testing can be done through the AWS Console or CLI directly - These files are no longer needed since the setup process is fully automated through CDK
- Update README.md to remove references to deleted files (setup-postgres-lambda.sql and test-lambda.sh) - Add information about synchronous Lambda invocation in PostgreSQL to Lambda flow - Remove unused parameter group from RDS cluster configuration - Remove test property from custom resource as it's not needed - Remove outdated SUMMARY.md file that's no longer relevant
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Shows how to use lambda with Postgres.
Lambda postgres via custom resource
Lambda call postgres
Postgres call lambda
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.