CCM-21858: APIM Auth module - #237
Open
simonlabarere wants to merge 28 commits into
Open
Conversation
gareth-allan
reviewed
Aug 7, 2026
gareth-allan
left a comment
gareth-allan
left a comment
Contributor
There was a problem hiding this comment.
I've not been over the source or TF files yet, but some comments on the other bits.
Contributor
There was a problem hiding this comment.
Not something to do in this PR, but I wonder if we could create an npm package containing this default ESLint config and publish it for other repos to consume? It would make it easier to keep everything aligned and push out any future updates, and you could still override specific settings on a per-repo basis if you wanted.
gareth-allan
previously approved these changes
Aug 10, 2026
Ian-Hodges
reviewed
Aug 11, 2026
Ian-Hodges
reviewed
Aug 11, 2026
simonlabarere
force-pushed
the
feature/CCM-21858_apim_auth_module
branch
2 times, most recently
from
August 11, 2026 12:28
47c4a16 to
471bbd9
Compare
Ian-Hodges
previously approved these changes
Aug 11, 2026
aidenvaines-cgi
requested changes
Aug 11, 2026
aidenvaines-cgi
left a comment
aidenvaines-cgi
left a comment
Contributor
There was a problem hiding this comment.
you can slap a tag on this branch like tmp-ccm-21858 or something to test this out
gareth-allan
approved these changes
Aug 13, 2026
aidenvaines-cgi
approved these changes
Aug 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR is to create an APIM Authentication shared module contains all the components required for APIM authentication:
The output of the module is the SSM parameter containing the APIM token that will be automatically refreshed every 9 minutes.
This PR also contains changes in order to:
The 2 lambdas and utils library have been copied directly from the Digital Letters BC.
Check https://github.com/NHSDigital/nhs-notify-deep-ping/pull/40 for test evidence.
Type of changes
Checklist
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.