-
Notifications
You must be signed in to change notification settings - Fork 367
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
Add Sequence Token Generator ? #2175
Comments
And previously, it was a random.
If you like to add one, OK. I don't think, it makes sense to use it as default. |
By the way, assuming that such an attack would make only sense say within 24h. How many, say 4 bytes token are emitted usually in 24h per device? What will be the probability, that one of 4 byte tokens are reused within 24h? |
The API is able to work with a custom implementation of TokenGenerator. If someone is interested, please go for it. In the case someone wants to contribute it, that will be welcome. |
I understand that current default TokenGenerator is random based.
Looking at 4.2. Updated Token Processing Requirements for Clients , It seems that recommended way is a sequence number.
Do you think that SequenceGenerator should be added and/or maybe use as default ?
The text was updated successfully, but these errors were encountered: