Skip to content

Commit 8d297c4

Browse files
authored
CCM-13475: Update readme (#176)
* Fix PDF Analysed documentation event name * Correct S3 & event names: Letters -> Files * Added link between UnscannedFiles bucket and MoveFiles lambda, and tidied up lines * Remove unused elements from diagram * Updated README to include example events
1 parent fc4bbd8 commit 8d297c4

7 files changed

Lines changed: 287 additions & 243 deletions

File tree

README.md

Lines changed: 94 additions & 89 deletions
Large diffs are not rendered by default.

docs/collections/_diagrams/c4code-file-scanner.md

Lines changed: 24 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ architecture-beta
1010
service itemDequeued(aws:res-amazon-eventbridge-event)[ItemDequeued Event]
1111
service scannerQueue(logos:aws-sqs)[Scanner Queue] in fileScanner
1212
service scannerLambda(logos:aws-lambda)[File Scanner] in fileScanner
13+
service moveQueue(logos:aws-sqs)[MoveFiles Queue] in fileScanner
1314
service moveLambda(logos:aws-lambda)[Move Scanned Files] in fileScanner
1415
service docRefBucket(logos:aws-s3)[DocumentReference] in fileScanner
1516
service unscannedBucket(logos:aws-s3)[UnscannedFiles] in fileScanner
@@ -22,20 +23,35 @@ architecture-beta
2223
junction j1 in fileScanner
2324
junction j2 in fileScanner
2425
junction j3 in fileScanner
26+
junction j4 in fileScanner
27+
junction j5 in fileScanner
28+
junction j6 in fileScanner
29+
junction j7 in fileScanner
30+
junction j8 in fileScanner
31+
junction j9 in fileScanner
2532
2633
itemDequeued:R --> L:scannerQueue
2734
docRefBucket:B --> T:scannerLambda
2835
scannerQueue:R --> L:scannerLambda
2936
scannerLambda:B --> T:unscannedBucket
3037
unscannedBucket:R --> L:guardDuty
38+
j1:T -- B:unscannedBucket
3139
guardDuty:R --> L:scanComplete
32-
scanComplete:R --> L:moveLambda
33-
moveLambda:R -- L:j1
34-
j1:T -- B:j2
35-
j1:B -- T:j3
36-
j2:R --> L:quarantineBucket
37-
j3:R --> L:safeBucket
38-
quarantineBucket:R --> L:quarantinedFile
39-
safeBucket:R --> L:safeFile
40+
scanComplete:R --> L:moveQueue
41+
moveQueue:R --> L:moveLambda
42+
j2:T --> B:moveLambda
43+
j1:R -- R:j2
4044
45+
moveLambda:R -- L:j3
46+
j3:T -- B:j4
47+
j4:R -- L:j5
48+
j5:R --> L:quarantineBucket
49+
j5:T -- L:j6
50+
j6:R --> L:quarantinedFile
51+
52+
j3:B -- T:j7
53+
j7:R -- L:j8
54+
j8:R --> L:safeBucket
55+
j8:B -- L:j9
56+
j9:R --> L:safeFile
4157
```

docs/collections/_diagrams/c4code-printer-analyser.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ architecture-beta
99
service fileSafe(aws:res-amazon-eventbridge-event)[FileSafe Event]
1010
service analyseQueue(logos:aws-sqs)[Analyse Queue] in printAnalyser
1111
service analyseLambda(logos:aws-lambda)[PrintAnalyser] in printAnalyser
12-
service safeBucket(logos:aws-s3)[SafeLetters] in printAnalyser
12+
service safeBucket(logos:aws-s3)[SafeFiles] in printAnalyser
1313
service pdfAnalysed(aws:res-amazon-eventbridge-event)[PDFAnalysed Event]
1414
group printAnalyser(cloud)[PrintAnalyser]
1515

docs/collections/_diagrams/sequence-print-metadata.md

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -6,49 +6,49 @@ title: sequence-expire-print-ttl
66

77
```mermaid
88
sequenceDiagram
9-
participant scannerlambda as Lambda<br/>PrintScanner
9+
participant scannerLambda as Lambda<br/>PrintScanner
1010
participant analyserLambda as Lambda<br/>PrintAnalyser
1111
participant senderLambda as Lambda<br/>PrintSender
12-
participant moveLambda as Lambda<br/>MoveLetters
13-
participant unscannedS3 as S3<br/>UnscannedLetters
12+
participant moveLambda as Lambda<br/>MoveFiles
13+
participant unscannedS3 as S3<br/>UnscannedFiles
1414
participant gd as GuardDuty
15-
participant safeS3 as S3<br/>SafeLetters
16-
participant quarantinedS3 as S3<br/>QuarantinedLetters
15+
participant safeS3 as S3<br/>SafeFiles
16+
participant quarantinedS3 as S3<br/>QuarantinedFiles
1717
participant eventBus as Event Bus
1818
participant printApi as Print API
1919
20-
eventBus ->> scannerlambda: ItemDequeued event
21-
activate scannerlambda
22-
scannerlambda ->> scannerlambda: Extract & Decode PDF
23-
scannerlambda -) unscannedS3: Store PDF
24-
deactivate scannerlambda
20+
eventBus ->> scannerLambda: ItemDequeued event
21+
activate scannerLambda
22+
scannerLambda ->> scannerLambda: Extract & Decode PDF
23+
scannerLambda -) unscannedS3: Store PDF
24+
deactivate scannerLambda
2525
unscannedS3 -) gd: S3 new object event
2626
activate gd
2727
gd -) gd: Scan for threats
2828
gd -) eventBus: ScanResult event
2929
deactivate gd
3030
eventBus -) moveLambda: ScanResult event
3131
activate moveLambda
32-
alt Move scanned letter
32+
alt Move scanned file
3333
moveLambda ->> safeS3: Store safe PDF
34-
moveLambda ->> eventBus: PrintLetterSafe event
34+
moveLambda ->> eventBus: FileSfe event
3535
else
3636
moveLambda ->> quarantinedS3: Store quarantined PDF
37-
moveLambda ->> eventBus: PrintLetterQuarantined event
37+
moveLambda ->> eventBus: FileQuarantined event
3838
end
3939
moveLambda ->> unscannedS3: Delete unscanned PDF
4040
deactivate moveLambda
41-
eventBus -) analyserLambda: PrintLetterSafe event
41+
eventBus -) analyserLambda: FileSafe event
4242
activate analyserLambda
4343
analyserLambda ->> safeS3: Get scanned PDF
4444
activate safeS3
4545
safeS3 -->> analyserLambda: PDF
4646
deactivate safeS3
4747
analyserLambda ->> analyserLambda: Count pages
4848
analyserLambda ->> analyserLambda: SHA256
49-
analyserLambda ->> eventBus: PrintLetterAnalysed event
49+
analyserLambda ->> eventBus: PDFAnalysed event
5050
deactivate analyserLambda
51-
eventBus -) senderLambda: PrintLetterAnalysed event
51+
eventBus -) senderLambda: PDFAnalysed event
5252
activate senderLambda
5353
senderLambda -) eventBus: letter.PREPARED event
5454
deactivate senderLambda

docs/collections/_events/pdf-analysed.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
22
title: pdf-analysed
33
type: uk.nhs.notify.digital.letters.print.pdf.analysed.v1
4-
nice_name: PrintLetterAnalysed
4+
nice_name: PDFAnalysed
55
service: Print Supplier Services
66
schema_envelope: https://notify.nhs.uk/cloudevents/schemas/digital-letters/2025-10-draft/events/uk.nhs.notify.digital.letters.print.pdf.analysed.v1.schema.json
77
schema_data: https://notify.nhs.uk/cloudevents/schemas/digital-letters/2025-10-draft/data/digital-letters-print-pdf-analysed-data.schema.json

scripts/config/vale/styles/config/vocabularies/words/accept.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@ Octokit
6464
onboarding
6565
outputDir
6666
Podman
67+
[pP][dD][fF]
6768
[pP][dD][mM]
6869
[pP]re
6970
producedby

0 commit comments

Comments
 (0)