Skip to content

NRL-2015 option 4 - choose v2 if v2 permissions file in lambda layer#1172

Draft
anjalitrace2-nhs wants to merge 7 commits intodevelopfrom
NRL-2015-option-4
Draft

NRL-2015 option 4 - choose v2 if v2 permissions file in lambda layer#1172
anjalitrace2-nhs wants to merge 7 commits intodevelopfrom
NRL-2015-option-4

Conversation

@anjalitrace2-nhs
Copy link
Contributor

@anjalitrace2-nhs anjalitrace2-nhs commented Mar 17, 2026

image

TODO:

  • Remove all references to v1 vs v2 in integration tests now that v1 & v2 requests have the same headers
  • Write an extra integration test that falls back to v1 when a v2 file isn't present (but a neighbouring ODS is :| )

@github-actions
Copy link

💥 Something went wrong while building the pull request environment.
Check Output Logs

@github-actions
Copy link

💥 Something went wrong while building the pull request environment.
Check Output Logs

@github-actions
Copy link

💥 Something went wrong while building the pull request environment.
Check Output Logs

…v2 permissions file. No longer decide based on missing v1 headers. In order to lookup v2 permissions file we need v2 headers though
@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: e49fd0e33858a6868ae428fdd99900ec2538c5c8
URL: https://nrl2015-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: ed6254d30512a70a46fc53ca7460c2fcd39f1967
URL: https://nrl2015-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: d7cf9543eb520fb07303dbfbf62eb6379a0323bc
URL: https://nrl2015-01ba47.api.record-locator.dev.national.nhs.uk/

@anjalitrace2-nhs anjalitrace2-nhs changed the title NRL-2015 option 4 propsed changes NRL-2015 option 4 - choose v2 if v2 permissions file in lambda layer Mar 20, 2026
@anjalitrace2-nhs anjalitrace2-nhs marked this pull request as ready for review March 20, 2026 11:40
@anjalitrace2-nhs anjalitrace2-nhs requested a review from a team as a code owner March 20, 2026 11:40
… or v2 determined by v2 file present rahter than request headers
@sonarqubecloud
Copy link

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 29e931520ba96a98f73efddfe538a585b932d2c7
URL: https://nrl2015-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: d66af6a5a03fb5470bc99c939156761a0819c2aa
URL: https://nrl2015-01ba47.api.record-locator.dev.national.nhs.uk/

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.

1 participant