Skip to content
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

CDD-2443 Create cognito and api gateway modules using terraform #1201

Merged
merged 38 commits into from
Feb 19, 2025

Conversation

ChristianAMartin
Copy link
Contributor

@ChristianAMartin ChristianAMartin commented Jan 22, 2025

Define and implement the infrastructure using Infrastructure as Code (Terraform) to provision Amazon Cognito and API Gateway resources, including required dependencies such as stubbed SAML and OIDC identity providers, and pre-signup and post-authentication Lambda functions for logging. The infrastructure also involves role and policy definitions for required permissions. The project intends to use the same database for both public and private data access.

Copy link

github-actions bot commented Jan 22, 2025

unit test coverage report

Title Lines Statements Branches Functions
lambda-producer-handler Coverage: 100%
100% (41/41) 25% (2/8) 100% (8/8)
lambda-db-password-rotation Coverage: 100%
100% (26/26) 40% (4/10) 100% (4/4)
lambda-alarm-notification Coverage: 100%
100% (23/23) 100% (5/5)
legacy-dashboard-redirect-viewer-request Coverage: 100%
100% (16/16) 100% (4/4) 100% (3/3)
public-api-cloud-front-viewer-request Coverage: 100%
100% (18/18) 100% (12/12) 100% (3/3)
Title Tests Skipped Failures Errors Time
lambda-producer-handler 16 0 💤 0 ❌ 0 🔥 0.842s ⏱️
lambda-db-password-rotation 5 0 💤 0 ❌ 0 🔥 0.641s ⏱️
lambda-alarm-notification 5 0 💤 0 ❌ 0 🔥 0.644s ⏱️
legacy-dashboard-redirect-viewer-request 10 0 💤 0 ❌ 0 🔥 0.495s ⏱️
public-api-cloud-front-viewer-request 11 0 💤 0 ❌ 0 🔥 0.495s ⏱️

@ChristianAMartin ChristianAMartin force-pushed the task/add-infra-cognito-db/CDD-2443 branch from e8e629f to d358ba4 Compare January 22, 2025 16:45
terraform/20-app/main.tf Outdated Show resolved Hide resolved
terraform/20-app/main.tf Outdated Show resolved Hide resolved
terraform/20-app/main.tf Outdated Show resolved Hide resolved
terraform/20-app/main.tf Outdated Show resolved Hide resolved
terraform/20-app/main.tf Outdated Show resolved Hide resolved
terraform/20-app/main.tf Outdated Show resolved Hide resolved
terraform/20-app/main.tf Outdated Show resolved Hide resolved
@ChristianAMartin ChristianAMartin force-pushed the task/add-infra-cognito-db/CDD-2443 branch from 5b64ff2 to 1c1bcfe Compare January 24, 2025 15:53
@ChristianAMartin ChristianAMartin force-pushed the task/add-infra-cognito-db/CDD-2443 branch from 8b40e4e to 1fc0e12 Compare January 29, 2025 15:04
@ChristianAMartin ChristianAMartin changed the title CDD-2443 Create cognito module using terraform CDD-2443 Create cognito and api gateway modules using terraform Jan 29, 2025
@ChristianAMartin ChristianAMartin force-pushed the task/add-infra-cognito-db/CDD-2443 branch 8 times, most recently from 2efae3c to 4881138 Compare February 7, 2025 09:33
@ChristianAMartin ChristianAMartin force-pushed the task/add-infra-cognito-db/CDD-2443 branch 2 times, most recently from b4ce97d to 8930dfc Compare February 17, 2025 09:18
@ChristianAMartin ChristianAMartin marked this pull request as ready for review February 17, 2025 15:49
@ChristianAMartin ChristianAMartin requested a review from a team as a code owner February 17, 2025 15:49
@ChristianAMartin ChristianAMartin force-pushed the task/add-infra-cognito-db/CDD-2443 branch from 8930dfc to 19c3848 Compare February 17, 2025 15:50
@ChristianAMartin ChristianAMartin force-pushed the task/add-infra-cognito-db/CDD-2443 branch from a1e51aa to 03694f1 Compare February 19, 2025 09:35
@ChristianAMartin ChristianAMartin merged commit ae44589 into main Feb 19, 2025
6 checks passed
@ChristianAMartin ChristianAMartin deleted the task/add-infra-cognito-db/CDD-2443 branch February 19, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants