Skip to content

CCM-24047: added mTLS instructions for client callbacks - #1099

Merged
cgitim merged 12 commits into
releasefrom
feature/CCM-24047
Nov 4, 2026
Merged

cgitim merged 12 commits into
releasefrom
feature/CCM-24047

Conversation

@cgitim

@cgitim cgitim commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Added mTLS instructions for client callbacks

https://nhsd-confluence.digital.nhs.uk/spaces/RIS/pages/1213034622/REFCOM-2025-085+%F0%9F%9F%A2+Improved+Client+Callbacks
https://nhsd-confluence.digital.nhs.uk/spaces/RIS/pages/1373767319/REFCOM-2026-039+%F0%9F%9F%A2+Callbacks+mTLS+Certificate+Approach

Reviews Required

  • Dev
  • Test
  • Tech Author
  • Product Owner

Checklist

  • Brief description of work completed, and any technical decisions made as part of the PR
  • PR link added as a comment to the relevant JIRA ticket
  • PR link shared on Slack and/or Teams
  • 2 reviews received
  • Tester approval

@github-actions

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@github-actions

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@github-actions

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

Error in browser when running `make serve`:
```
Uncaught SyntaxError: Cannot use 'import.meta' outside a module
hot.js:35 [hot] hot reloading connected
(index):35 Uncaught TypeError: Cannot read properties of undefined (reading 'init')
    at (index):35:35
```

Cause:
 make serve  ran  redocly preview-docs , whose generated HTML loaded
 cdn.redocly.com/redoc/latest/...  as a classic script. The current  latest
bundle uses  import.meta , which only parses inside a module — hence the
SyntaxError,  window.Redoc  undefined,  .init  failure, blank page.

Fix options considered:
1. patch-package — pin preview server to  redoc/v2.5.0 . Durable (survives
   reinstall), keeps live-reload for  serve  +  serve-watch . Cost: a committed
   patch file +  postinstall  hook.
2. Switch  serve  to  build-docs  —  redocly build-docs  already pins the
   working  redoc/v2.5.0 . Generate a static HTML and serve it. No node_modules
   changes at all. Cost: loses redocly's websocket live-reload (browser needs
   manual refresh;  serve-watch  would just rebuild the file).
3. Upgrade  @redocly/cli  1.34 → 2.x — as is fixed upstream, but it's a major
   bump. Cost: also loses redocly's websocket live-reload. But, it enables us
   to pin a working version, so can avoid this happenning again.

Fix (option 3):
- Bump @redocly/cli to 2.72.1. The `preview-docs` command was removed in
  2.x, so the serving approach had to change regardless.
- Add a `build-docs` script using `redocly build-docs --inlineBundle`,
  producing a self-contained build/index.html with redoc and the spec
  inlined (loaded as a module), removing the broken-CDN dependency.
- Point `serve` at the static build/ directory via python http.server.
- Add browser-sync (serve-live) and rewire `make serve-watch` to rebuild
  on spec edits (nodemon) and auto-reload the browser, restoring true
  live reload that preview-docs previously provided.
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@cgitim
cgitim requested a review from mjewildnhs November 3, 2026 15:28
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

This branch is work on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket:

CCM-24047

@cgitim
cgitim merged commit 20a6d63 into release Nov 4, 2026
6 checks passed
@cgitim
cgitim deleted the feature/CCM-24047 branch November 4, 2026 16:49
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.

4 participants