Skip to content

Comments

Implement gateway listener with add worklist item action handler#9

Merged
steventux merged 10 commits intomainfrom
feat/gateway-listener
Jan 20, 2026
Merged

Implement gateway listener with add worklist item action handler#9
steventux merged 10 commits intomainfrom
feat/gateway-listener

Conversation

@steventux
Copy link
Contributor

@steventux steventux commented Jan 12, 2026

Description

  • Add relay listener which connects to Azure Relay resource and enables communication from Azure hosted services (Manage Breast Screening) to remote (NHS Trust networks) Modality Worklist storage.
  • Adds a command class to handle the creation of worklist items.
image

Jira link

https://nhsd-jira.digital.nhs.uk/browse/DTOSS-11852

Review notes

Review checklist

  • Check database queries are correctly scoped to current_provider

@steventux steventux changed the title Feat/gateway listener Implement gateway listener with add worklist item action handler Jan 12, 2026
@steventux steventux force-pushed the feat/gateway-listener branch 2 times, most recently from 88a62fa to 1952e2e Compare January 13, 2026 13:22
@steventux steventux marked this pull request as ready for review January 13, 2026 14:12
@steventux steventux requested a review from a team January 13, 2026 14:13
@steventux steventux force-pushed the feat/gateway-listener branch from eab34ca to f676bde Compare January 14, 2026 09:52
Calls store_worklist_item on the storage instance using the values from the given payload.
Adds .env.development template for local development.
Adds listener compose service definition.
@steventux steventux force-pushed the feat/gateway-listener branch from f676bde to e7db97a Compare January 19, 2026 11:02

## Firewall Compatibility

Both connections work through firewalls because:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Both connections work through firewalls because:
The connection works through firewalls because:

Comment on lines 72 to 84
### 3. Running the Services

#### Gateway

```bash
docker compose up --build
```

This starts:

- Modality Worklist server
- Relay listener which listens on `AZURE_RELAY_HYBRID_CONNECTION`

Copy link
Contributor

Choose a reason for hiding this comment

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

maybe not necessary here, since it's the same as for the rest of the gateway

Copy link
Contributor

@carlosmartinez carlosmartinez left a comment

Choose a reason for hiding this comment

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

@steventux steventux force-pushed the feat/gateway-listener branch 2 times, most recently from 743047d to f7a4440 Compare January 19, 2026 12:28
We share the database between docker containers at /var/lib/pacs and this is a shared volume mounted on the docker host.
Make it consistent and add the relevant env vars consistently.
@steventux steventux force-pushed the feat/gateway-listener branch from f7a4440 to 0a141a3 Compare January 20, 2026 09:50
@steventux steventux merged commit 3cdcd6b into main Jan 20, 2026
4 checks passed
@steventux steventux deleted the feat/gateway-listener branch January 20, 2026 14:12
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