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

combining function of nonce and id #51

Open
TomCJones opened this issue Apr 29, 2020 · 2 comments
Open

combining function of nonce and id #51

TomCJones opened this issue Apr 29, 2020 · 2 comments

Comments

@TomCJones
Copy link

Based on a iiw discussion there is a proposal to combine functionality. Here are the reqs - would be interested in solutions.

  1. track messages
  2. route message
  3. test validity of message early - ie mitigate DoS
@tplooker
Copy link
Member

There is currently nothing preventing applications from using the id as a nonce, as there is equally nothing preventing applications from declaring and using an explicit nonce field, I agree with all the points you make above in reference to them being valid uses of the id field I am just wondering if there is anything we can add to the spec to make this clearer?

@kdenhartog
Copy link
Contributor

As Paul Dietrich pointed out on that call, some consideration should be taken around the security of combining these pieces of functionality. In particular, is it possible that the behavior of the nonce will conflict the use of ID for message tracking or routing that could potentially lead to confusion and ultimately insecure implementations.

With that in mind, I think the best route would be to make the nonce explicitly separate as we don't have a compactness requirement that would suggest combining them would be the right approach. If there's other reasons other than compactness that we should consider as a benefit to this approach, I'd like to consider them.

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

No branches or pull requests

3 participants