Skip to content

Add IANA registries#380

Open
mickenordin wants to merge 2 commits into
developfrom
kano-iana
Open

Add IANA registries#380
mickenordin wants to merge 2 commits into
developfrom
kano-iana

Conversation

@mickenordin

@mickenordin mickenordin commented Jun 23, 2026

Copy link
Copy Markdown
Member

This PR adds IANA registires for:

  • notifications
  • resourceTypes
  • shareTypes
  • protocols
  • share payloads

It also registers OCM-MLS notification entries.

The idea with the registries is to try to find where the extension points of OCM should be. We want the protocol to be extensible AND interoperable. Where the spec is unclear or under-developed interoperability suffers. If we add extension points to the specification with out making it possible to explain how to use that extension point, only proprietary single vendor extensions are possible.

So for example writing in the specification text, that other kinds of notifications are allowed and that implementors MUST ignore notifications they don't understand, is adding an extension point, that is guaranteed not to be interoperable. The registries make sure that we get both extensibility AND interoperabilities at the same time.

@mickenordin mickenordin requested review from MahdiBaghbani and glpatcern and removed request for glpatcern June 23, 2026 16:11

@glpatcern glpatcern left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Micke, I have a couple of fixes and a conceptual question

Comment thread IETF-OCM-MLS.md Outdated
Comment thread IETF-OCM.md Outdated
Comment thread IETF-OCM.md
| SHARE_UNSHARED | Share | active | This document |
| REQUEST_RESHARE | Share | experimental | This document |
| RESHARE_UNDO | Share | experimental | This document |
| RESHARE_CHANGE_PERMISSION | Share | experimental | This document |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we also have USER_REMOVED, though I realize we mention it only in spec.yaml, not (yet) in the I-D. That's pending on the notifications work...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, what I want is to add the regestires, but I fully expect that the content change as we fix notifications.

@glpatcern glpatcern left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some more comments. I start to get the principle, that is we define a way for implementers to extend the spec in the parts that may be extended, without needing to touch the core spec. This is very good in view of a final RFC.

Comment thread IETF-OCM.md
Comment on lines +1683 to +1685
A Sending Server MUST NOT send a share for a combination that the
Receiving Server does not advertise in Discovery (see
[Share Creation Notification](#share-creation-notification)).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should allow webdav as a legacy exception

Comment thread IETF-OCM.md
specifications MAY register additional combinations, including ones
that extend an already-registered protocol to a new resource type or
share type; doing so does not modify that protocol's own registration.
The federation combinations are registered in this way by [OCM-MLS].

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I understand the concept, but I disagree with this. What about adding federation here, where Reference is populated with This document and [OCM-MLS], and drop it from the other I-D?
My point is we do allow federation also as share payload, even if it's not specified, so the type belongs here, the internals belong to [OCM-MLS].

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

Successfully merging this pull request may close these issues.

2 participants