Decision to allow colon character (:
) in Record Keys
#2224
bnewbold
announced in
Dev Announcements
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
We went back and forth on this for months, but have finally decided to just allow colons (
:
) in record keys. We are updating the spec, and will be updating the golang and typescript SDKs soon.The backstory on this is that the specs forbid it, but the reference PDS implementation didn't enforce the spec fully. A few popular feed generator records got created. We had concerns about potentially requiring URL encoding, or some weird syntax corner cases, but after the team read and re-read the relevant IETF RFCs several times, we decided to just chill out and relax the syntax.
We recommend folks hold off from actually creating new record keys with this character for a couple weeks, so that SDK and client app developers have time to update their software.
Relevant PRs:
The "interop test files" have been updated with examples.
cc: @MarshalX @myConsciousness @DavidBuchanan314 @sugyan as some SDK and lower-level devs who I can imagine might be impacted
Beta Was this translation helpful? Give feedback.
All reactions