Skip to content

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

Merged
anjalitrace2-nhs merged 13 commits intodevelopfrom
NRL-2015-option-4
Mar 27, 2026
Merged

NRL-2015 option 4 - choose v2 if v2 permissions file in lambda layer#1172
anjalitrace2-nhs merged 13 commits intodevelopfrom
NRL-2015-option-4

Conversation

@anjalitrace2-nhs
Copy link
Copy Markdown
Contributor

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

  • use v2 permissions if a v2 permissions file is present in the lambda layer (rather than by if both v1 headers are supplied)
image
  • redid unit tests to match this new behaviour
  • all integration tests now include both v1 & v2 headers to best reflect what will be happening in prod going forward. Got rid of v1/v2-specific integration test steps

for even more scenarios see: https://miro.com/app/board/uXjVICmXH9E=/?moveToWidget=3458764664560834826&cot=14

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
Copy Markdown

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

@github-actions
Copy link
Copy Markdown

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

@github-actions
Copy link
Copy Markdown

💥 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
Copy Markdown

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

@github-actions
Copy link
Copy Markdown

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

@github-actions
Copy link
Copy Markdown

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

@github-actions
Copy link
Copy Markdown

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

mattdean3-nhs
mattdean3-nhs previously approved these changes Mar 26, 2026
Copy link
Copy Markdown
Contributor

@mattdean3-nhs mattdean3-nhs left a comment

Choose a reason for hiding this comment

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

Looks/works great 💪

Once comment on belt-and-braces error handling but not a blocker for me.

@github-actions
Copy link
Copy Markdown

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

@github-actions
Copy link
Copy Markdown

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

@github-actions
Copy link
Copy Markdown

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

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

🚀 PR environment successfully deployed.
Commit Hash: 644137f7dd98fffe6f26da063364f6cd459e6adc
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.

2 participants