Skip to content

Add TaggedExtensionLabel for tagged-extension keys - #865

Open
danieleggert wants to merge 1 commit into
harden-2-validate-atomsfrom
harden-3-tagged-ext-label
Open

danieleggert wants to merge 1 commit into
harden-2-validate-atomsfrom
harden-3-tagged-ext-label

Conversation

@danieleggert

@danieleggert danieleggert commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

[3/6] One of the design goals of this library is that the API types can only represent valid state. But that doesn’t hold true today. These 6 stacked PRs are smaller steps towards that goal. #863 #864 #865 #866 #867 #868

Motivation:

RFC 4466 tagged-ext-label keys were Strings written raw.

Modifications:

TaggedExtensionLabel replaces them; literals compile unchanged. Case-insensitive per RFC 4466, so (foo 1 FOO 2) parses as one entry.

@danieleggert
danieleggert force-pushed the harden-3-tagged-ext-label branch from 2e53560 to 20e5b9d Compare October 29, 2026 15:26
@danieleggert
danieleggert force-pushed the harden-3-tagged-ext-label branch from 20e5b9d to 2220077 Compare October 30, 2026 14:42
Generic over the extended type, so a STORE key can't be used where a FETCH key is expected.
@danieleggert
danieleggert force-pushed the harden-3-tagged-ext-label branch from 2220077 to 1d94b70 Compare October 30, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🆕 semver/minor Adds new public API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants