~bigbes/sr-ht-dolt · parade

main · last commit 5 hours ago · ti41arbu

← Back to the parade

sr-ht-dolt-jgv Switch /static/ mount from r.Handle to chimw.GetHead Lined Up

status: open P3 task
bd update sr-ht-dolt-jgv --claim bd close sr-ht-dolt-jgv
Created byEugene Blikh
Ownerbigbes@gmail.com
Created2026-08-17T20:40:00Z
Updated2026-08-17T20:40:00Z
Description
What is wrong: dolt mounts `/static/` with `r.Handle(assets.DefaultPrefix+"*", s.static)`, which registers every HTTP method for that route (unlike `chimw.GetHead`, which registers only GET/HEAD). A same-origin POST to `/static/main.min.<sha>.css` is served the file with a 200 instead of getting a routed 405.

Why it matters: harmless today, but it means dolt's routing tree does not describe what the service actually serves — the exact argument `chimw.GetHead`'s own doc-comment makes for registering the GET/HEAD pair on the tree instead of rewriting methods per request at handler time.

Sibling that already does it correctly: `chimw.GetHead(r, assets.DefaultPrefix+"*", …)` in artifacts, bench, curator, and tokens.

Fix: convert dolt's `/static/` mount from `r.Handle(...)` to `chimw.GetHead(...)`.

Status: verified.

Audit: D38 (cross-repo audit, 2026-08-17)

Depends on

No outgoing dependencies.

Depended on by

Nothing depends on this issue.

No comments.

  • Eugene Blikh created the issue · 2026-08-17T23:40:00Z
Stored rows — what this pane was built from, as read
issues 1 row
id sr-ht-dolt-jgv
content_hash 4540dae377350c5d67132704007733e4d3402f5ddc2525187240a1ebc7f98d6a
title Switch /static/ mount from r.Handle to chimw.GetHead
description What is wrong: dolt mounts `/static/` with `r.Handle(assets.DefaultPrefix+"*", s.static)`, which registers every HTTP method for that route (unlike `chimw.GetHead`, which registers only GET/HEAD). A same-origin POST to `/static/main.min.<sha>.css` is served the file with a 200 instead of getting a routed 405. Why it matters: harmless today, but it means dolt's routing tree does not describe what the service actually serves — the exact argument `chimw.GetHead`'s own doc-comment makes for registering the GET/HEAD pair on the tree instead of rewriting methods per request at handler time. Sibling that already does it correctly: `chimw.GetHead(r, assets.DefaultPrefix+"*", …)` in artifacts, bench, curator, and tokens. Fix: convert dolt's `/static/` mount from `r.Handle(...)` to `chimw.GetHead(...)`. Status: verified. Audit: D38 (cross-repo audit, 2026-08-17)
design
acceptance_criteria
notes
status open
priority 3
issue_type task
assignee NULL
estimated_minutes NULL
created_at 2026-08-17T20:40:00Z
created_by Eugene Blikh
owner bigbes@gmail.com
updated_at 2026-08-17T20:40:00Z
closed_at NULL
closed_by_session
external_ref NULL
spec_id
compaction_level 0
compacted_at NULL
compacted_at_commit NULL
original_size NULL
sender
ephemeral 0
wisp_type
pinned 0
is_template 0
mol_type
work_type
source_system
metadata �{}
source_repo
close_reason
event_kind
actor
target
payload
await_type
await_id
timeout_ns 0
waiters
hook_bead
role_bead
agent_state
last_activity NULL
role_type
rig
due_at NULL
defer_until NULL
no_history 0
started_at NULL
is_blocked 0
events 1 row
id 01a01173-d94b-7d13-a565-094b8a1c2dc3
issue_id sr-ht-dolt-jgv
event_type created
actor Eugene Blikh
old_value
new_value
comment NULL
created_at 2026-08-17T23:40:00Z