Update rails 7.1.2

[alexdesi]

Jira link

https://transformuk.atlassian.net/browse/HOTT-4624

What?

Update Rails from 7.0 to 7.1.2

[jebw]

Looks good - couple of minor points to address and I think you need clarity on why force_ssl got turned off originally

[jebw]

@alexdesi This is the same as frontend? If you're using this option, you'll need to enable assume_ssl I think ?

@willfish @amberstarlight I'm not familiar on why this wasn't working before but it does appear to be working without assume_ssl????

[alexdesi]

This caused issues with the apply-terraform workflow, which uses the health check endpoint.
By the way, it is passing now, I'll check with @amberstarlight, in the FE I guess we need to coordinate the update and deployment.

[willfish]

CDN and load balancer terminate SSL and reverse proxy onto the apps, yeah.

The deployment will fail with force_ssl = true and we should instead be using assume_ssl = true

https://github.com/trade-tariff/trade-tariff-platform-aws-terraform/blob/main/environments/common/application-load-balancer/alb.tf#L26-L36

The load balancer checks the health of the apps via these endpoints over http:

locals {
  services = {
    admin = {
      host             = ["admin.*"]
      healthcheck_path = "/healthcheckz"
    }

    signon = {
      host             = ["signon.*"]
      healthcheck_path = "/healthcheck/live"
    }

    backend_uk = {
      paths            = ["/uk/api/beta/*"]
      healthcheck_path = "/healthcheckz"
    }

    backend_xi = {
      paths            = ["/xi/api/beta/*"]
      healthcheck_path = "/healthcheckz"
    }

    duty_calculator = {
      paths            = ["/duty-calculator/*"]
      healthcheck_path = "/healthcheckz"
    }

    search_query_parser = {
      paths            = ["/api/search/*"]
      healthcheck_path = "/healthcheckz"
    }

    frontend = {
      paths            = ["/*"]
      healthcheck_path = "/healthcheckz"
    }
  }
}

[willfish]

rails/rails@17ee01f

[alexdesi]

I've also set
config.assume_ssl = true
for the TT Admin

[alexdesi]

This is just a tiny refactor to make the row shorter and readable.

feeeexed