Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider allowing implementation-specific string map keys #58

Open
banburybill opened this issue Feb 28, 2018 · 1 comment
Open

Consider allowing implementation-specific string map keys #58

banburybill opened this issue Feb 28, 2018 · 1 comment

Comments

@banburybill
Copy link
Contributor

banburybill commented Feb 28, 2018

  • Section 7.1: Negative integer map keys are concise, but they come with
    the potential for interoperability-hostile collisions. I think the
    document should encourage restricting their use to tightly-controlled
    closed systems, and then additionally propose a particular namespacing
    pattern (e.g., "{namespaceName}-{keyName}") for use of
    implementation-specific /string/ keys in other cases.

https://mailarchive.ietf.org/arch/msg/dnsop/OADqnhAq-Q3GA9JVde1JDmvLNwo

@banburybill
Copy link
Contributor Author

banburybill commented May 9, 2018

We understand the intention to reduce the chances of implementations attempting to read implementation-dependent items that they believe are items they know, but which in fact belong to a different implementation.

However:

  • we're concerned that string keys pull in the opposite direction to the concision of the rest of the format.
  • as we noted on the list, we intended the negative integer map keys for closed systems. One problem with introducing a generic extension mechanism for open systems is that it potentially complicates implementation. Implementations would need to be able to correctly parse any form of valid CBOR to safely skip over an unknown key, while currently they only need to parse the subset of CBOR constructions used by C-DNS. For example, at present, implementations do not need to parse FLOAT or SIMPLE items and do not need to handle fields having or not having tags.
  • we feel that a such a generic extension mechanism is beyond our current aims for this format. Implementors are, of course, free to experiment with this.

We had no other feedback on this suggestion following our comments on the list or the WG at IETF101. We've therefore not attempted to incorporate this into the latest draft.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant