Merge pull request #239 from NHSDigital/CCM-7908_VarsFromGithubUppercase

CCM-7908 case fixing tfvars

Author: aidenvaines-cgi

infrastructure/terraform/components/app/ssm_parameter_amplify_password.tf

@@ -3,11 +3,11 @@ resource "aws_ssm_parameter" "amplify_password" {
 
   name  = "/${local.csi}/amplify_password"
   type  = "String"
-  value = var.amplify_basic_auth_secret != "unset" ?  var.amplify_basic_auth_secret : random_password.password[0].result
+  value = var.AMPLIFY_BASIC_AUTH_SECRET != "unset" ?  var.AMPLIFY_BASIC_AUTH_SECRET : random_password.password[0].result
 }
 
 resource "random_password" "password" {
-  count = var.enable_amplify_basic_auth && var.amplify_basic_auth_secret == "unset" ? 1 : 0
+  count = var.enable_amplify_basic_auth && var.AMPLIFY_BASIC_AUTH_SECRET == "unset" ? 1 : 0
 
   length  = 16
   special = true

infrastructure/terraform/components/app/variables.tf

@@ -93,7 +93,8 @@ variable "enable_amplify_basic_auth" {
   default     = true
 }
 
-variable "amplify_basic_auth_secret" {
+# Github Environments only handles uppercase envvars
+variable "AMPLIFY_BASIC_AUTH_SECRET" {
   type        = string
   description = "Secret key/password to use for Amplify Basic Auth - This is entended to be read from CI variables and not commited to any codebase"
   sensitive   = true