ietf-wg-ppm / draft-ietf-ppm-dap-taskprov

Define DAP extension for in-band task provisioning.
Other
3 stars 4 forks source link

Provide guidance on whether to use public vs private extension for Taskbind #94

Open branlwyd opened 4 days ago

branlwyd commented 4 days ago

DAP-13 introduced the concept of "public extensions", complementing the preexisting (private) extension mechanism.

These extensions can conflict with one another (i.e. it is not possible to send the same extension type in both the public & private extensions), but it is left open if/how the choice of public vs private extensions changes the semantics of any given extension.

I think taskprov should provide some guidance as to which type of extension the Taskbind extension should be used with.

My initial thoughts are: the Taskbind extension uses an empty extension body, and is purposefully included in the extensions for both the Leader & Helper aggregators. Therefore, I think recommending this extension to be included in the public extensions makes sense. I don't have a strong opinion on whether we should block including this extension in the private extensions.

branlwyd commented 3 days ago

Ah, actually, the Client Behavior section says the following:

The extension codepoint taskbind MUST be offered in the extensions field of both Leader and Helper's PlaintextInputShare. In addition, each report's task ID MUST be computed as described in Section 3.

This implies that the extension must be included in the private extensions, since private extensions are included in each aggregators' PlaintextInputShare.

I suggest we at least accept public extensions, too, and probably recommend the use of a public extension.