Skip to content

CCM-21851 missing pack specification#645

Open
stevebux wants to merge 4 commits into
mainfrom
feature/CCM-21851-Missing-Pack-Specification-1.3
Open

CCM-21851 missing pack specification#645
stevebux wants to merge 4 commits into
mainfrom
feature/CCM-21851-Missing-Pack-Specification-1.3

Conversation

@stevebux

@stevebux stevebux commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Description

addresses two unrelated build failures:

  • extra validation catching an error in the dev supplier config
  • A failure to apply the TLS version when the API gateway is deployed

Context

Type of changes

  • Refactoring (non-breaking change)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would change existing functionality)
  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I am familiar with the contributing guidelines
  • I have followed the code style of the project
  • I have added tests to cover my changes
  • I have updated the documentation accordingly
  • This PR is a result of pair or mob programming

DT3-Specific Checklist

  • If I have added a new resource (SQS, Lambda, Gateway, DDB table, etc), I have created the appropriate alarms

Sensitive Information Declaration

To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.

  • I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

@stevebux stevebux requested review from a team as code owners July 10, 2026 10:34
@stevebux stevebux force-pushed the feature/CCM-21851-Missing-Pack-Specification-1.3 branch from 5036c50 to e247583 Compare July 10, 2026 14:11
@sonarqubecloud

Copy link
Copy Markdown

@stevebux

Copy link
Copy Markdown
Contributor Author

I went back and looked at the commit that set the Gateway Rest API to TLS1.2, and I had forgotten that the change was more complex than it needed to be, as Terraform didn't support the security_policy and endpoint_access_mode properties on aws_api_gateway_rest_api resources. A fix for that was released a couple of weeks ago, which may explain the change in behaviour. I have therefore reverted the change done under CCM-14114 and reimplemented it in a simple, straightforward manner (i.e. by adding the two properties to aws_api_gateway_rest_api.main

@stevebux stevebux changed the title Feature/ccm 21851 missing pack specification 1.3 CCM-21851 missing pack specification 1.3 Jul 10, 2026
@stevebux stevebux changed the title CCM-21851 missing pack specification 1.3 CCM-21851 missing pack specification Jul 10, 2026

@francisco-videira-nhs francisco-videira-nhs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK to me

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.

2 participants