Severity: P1. Reviewed at 82b715c. The spec contradicts itself on the version header.
- S9 (
SPEC.md:180): "Conforming servers SHOULD include the response header X-Http-Sql-Version: 0.1"
- S10.1 item 7 (
SPEC.md:208), under "A v0.1 conforming server MUST": "Emit the X-Http-Sql-Version response header."
conformance/README.md H-1 and H-2 test it as required on both success and error responses.
Two of the three say required; the normative prose section says optional.
Resolution: MUST
Both reference servers already emit it unconditionally, and the conformance suite already tests it as required, so raising S9 to MUST costs nothing and matches shipped behavior. The alternative -- demoting S10.1 and H-1/H-2 to optional -- would make version detection unreliable for exactly the multi-backend clients the spec exists to serve.
Acceptance
Origin: codex spec review, verified against source.
Severity: P1. Reviewed at 82b715c. The spec contradicts itself on the version header.
SPEC.md:180): "Conforming servers SHOULD include the response headerX-Http-Sql-Version: 0.1"SPEC.md:208), under "A v0.1 conforming server MUST": "Emit theX-Http-Sql-Versionresponse header."conformance/README.mdH-1 and H-2 test it as required on both success and error responses.Two of the three say required; the normative prose section says optional.
Resolution: MUST
Both reference servers already emit it unconditionally, and the conformance suite already tests it as required, so raising S9 to MUST costs nothing and matches shipped behavior. The alternative -- demoting S10.1 and H-1/H-2 to optional -- would make version detection unreliable for exactly the multi-backend clients the spec exists to serve.
Acceptance
SPEC.md:180reads MUST.Origin: codex spec review, verified against source.