Skip to content

Commit a249b53

Browse files
committed
docs(adr): make the 0158 TLS update non-perishable
The correction I added said 093db33 (#132) left alert_sinks.py as "the only remaining instance on main". That is a checklist-shaped claim with an expiry date: BACKLOG #323 layer 3 (PR #142) closes the alerts call site, and the sentence goes false the moment it lands. Dating the observation does not help a reader who greps for it in a month and finds nothing. Restated as what happened rather than what is currently true -- #132 closed the two connectors, the alerts call site is tracked as #323 layer 3 -- so it holds whether or not #142 merges, and it says outright that the current state must be grepped rather than cited from here. Deliberately does NOT assert that #142 closed the cell: #142 is open at time of writing, and asserting a merge that has not happened is the same defect pointing the other way. found by: the repo-security-review session, which owns #142 and re-derived all three call sites against origin/main before raising it.
1 parent 2214897 commit a249b53

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

docs/adr/0158-silent-controls-green-signals-that-mean-nothing-and-shape-over-detection.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -170,8 +170,10 @@ been killed" is a counterfactual about a run that never faced them.
170170
argument ([`alert_sinks.py`](../../messagefoundry/pipeline/alert_sinks.py):382-384). The same bare
171171
call stood at [`transports/email.py`](../../messagefoundry/transports/email.py) and
172172
[`transports/direct.py`](../../messagefoundry/transports/direct.py) when this was written; *update
173-
(2026-08-02, after this ADR was committed): 093db339 (#132) gave both an explicit verifying context,
174-
leaving the alert sink as the only remaining instance on `main`.* On the project's own
173+
(after this ADR was committed): 093db339 (#132) gave both connectors an explicit verifying context,
174+
and the alerts call site is tracked separately as BACKLOG #323 layer 3. This instance is recorded
175+
because it is the worked example the taxonomy was derived from, not as a statement of what is
176+
outstanding now -- a reader wanting the current state must grep, not cite this.* On the project's own
175177
interpreter (Python 3.14.6), `smtplib.SMTP.starttls` resolves a `None` context via
176178
`ssl._create_stdlib_context`, which **is** `ssl._create_unverified_context`: `verify_mode`
177179
CERT_NONE, `check_hostname` False. The sink encrypts and authenticates nothing. **A retraction is

0 commit comments

Comments
 (0)