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

Public report extensions #620

Closed
martinthomson opened this issue Oct 18, 2024 · 3 comments
Closed

Public report extensions #620

martinthomson opened this issue Oct 18, 2024 · 3 comments

Comments

@martinthomson
Copy link
Contributor

I don't know how I missed this earlier, but I think that the design of report extensions is very much suboptimal for the sorts of things that are in this draft. Those extensions carry public information, so having that data repeated is not ideal.

This draft makes a case for submissions with a reduced overhead. The efficiency gains depend on having extensions to the public report metadata. I'd like to request that report metadata be made extensible.

I can't conceive of any reason to use private report extensions, as currently defined. I'm not saying that it is a bad idea to provide that extension point, though a lack of reasons to use them makes their inclusion questionable. I guess I'm saying that moving upload extensions from private to public might be possible, rather than just adding another extension point.

@cjpatton
Copy link
Collaborator

cjpatton commented Oct 18, 2024

See #619 (comment) for historical context. Extensions used to be public, but we decided after draft 02 to make them private. In fact, it was your suggestion to encrypt them: see #369 (comment).

I'm open to doing something here, but given how far along we are, I think we need to be careful to not change the expectations of existing implementations. In particular, I'm hesitant to get rid of the private extensions altogether.

I'd be fine with adding public extensions to the report meta data. Would the semantics of these extensions need to be any different than for the existing private extensions? Other than the fact that they are necessarily shared by both aggregators.

@cjpatton
Copy link
Collaborator

2024/10/24 interim: We will put up a PR to add a public extension point. If we decide the protocol complexity is too great, then we'll consider removing the private extension points as well. (There are no publicly known use cases for this, but in the future an "encrypted extension" could be defined that makes this use case possible.)

@martinthomson
Copy link
Contributor Author

One thing that came up was that a public "encrypted" extension might end up being a little cumbersome (not just in definition, but also implementation). If that is the case, and we've removed the private extensions, DAP version 2 is always possible. A new version doesn't need to be a whole big deal. QUIC demonstrated that versioning can be done fairly cheaply with the right motivation.

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

No branches or pull requests

3 participants