Skip to content

Commit

Permalink
Merge pull request #84 from nekochans/feature/issue76/access-token-env
Browse files Browse the repository at this point in the history
lgtm-cat-api ECS にアクセストークンの検証に必要となる環境変数を追加
  • Loading branch information
kobayashi-m42 authored Feb 8, 2023
2 parents 483f54c + 4c9b74d commit 8cb31ab
Show file tree
Hide file tree
Showing 8 changed files with 24 additions and 0 deletions.
4 changes: 4 additions & 0 deletions modules/aws/cognito/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,7 @@ output "idp_endpoint" {
output "lgtm_cat_bff_client_id" {
value = aws_cognito_user_pool_client.lgtm_cat_bff_client.id
}

output "cognito_user_pool_id" {
value = aws_cognito_user_pool.user_pool.id
}
1 change: 1 addition & 0 deletions modules/aws/ecs/ecs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ resource "aws_ecs_task_definition" "api" {
db_name_arn = aws_ssm_parameter.db_name.arn
upload_images_bucket_name = var.upload_images_bucket_name
lgtm_images_cdn_domain = var.lgtm_images_cdn_domain
cognito_user_pool_id = var.cognito_user_pool_id
})

cpu = var.ecs_task_cpu
Expand Down
4 changes: 4 additions & 0 deletions modules/aws/ecs/task/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,10 @@
{
"name": "LGTM_IMAGES_CDN_DOMAIN",
"value": "${lgtm_images_cdn_domain}"
},
{
"name": "COGNITO_USER_POOL_ID",
"value": "${cognito_user_pool_id}"
}
],
"logConfiguration": {
Expand Down
3 changes: 3 additions & 0 deletions modules/aws/ecs/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -55,5 +55,8 @@ variable "lgtm_images_cdn_domain" {
variable "sentry_dsn" {
type = string
}
variable "cognito_user_pool_id" {
type = string
}

data "aws_region" "current" {}
5 changes: 5 additions & 0 deletions providers/aws/environments/prod/17-cognito/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,8 @@ output "lgtm_cat_bff_client_id" {
value = module.cognito.lgtm_cat_bff_client_id
sensitive = true
}

output "cognito_user_pool_id" {
value = module.cognito.cognito_user_pool_id
sensitive = true
}
1 change: 1 addition & 0 deletions providers/aws/environments/prod/20-api/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -57,4 +57,5 @@ module "ecs" {
db_username = local.db_username
lgtm_images_cdn_domain = data.terraform_remote_state.images.outputs.lgtm_images_cdn_domain
sentry_dsn = local.sentry_dsn
cognito_user_pool_id = data.terraform_remote_state.cognito.outputs.cognito_user_pool_id
}
5 changes: 5 additions & 0 deletions providers/aws/environments/stg/17-cognito/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,8 @@ output "lgtm_cat_bff_client_id" {
value = module.cognito.lgtm_cat_bff_client_id
sensitive = true
}

output "cognito_user_pool_id" {
value = module.cognito.cognito_user_pool_id
sensitive = true
}
1 change: 1 addition & 0 deletions providers/aws/environments/stg/20-api/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -57,4 +57,5 @@ module "ecs" {
db_username = local.db_username
lgtm_images_cdn_domain = data.terraform_remote_state.images.outputs.lgtm_images_cdn_domain
sentry_dsn = local.sentry_dsn
cognito_user_pool_id = data.terraform_remote_state.cognito.outputs.cognito_user_pool_id
}

0 comments on commit 8cb31ab

Please sign in to comment.