-
Notifications
You must be signed in to change notification settings - Fork 13
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
implement properties.timeout for account().tokens.add(properties) #18
Comments
Can I do this issue?! |
maybe finish #17 first :) but sure, all yours, let me know if you need any help |
sure thing 👍 |
I’ve invited you both to Hoodie’s contributors team. You can accept it at https://github.com/hoodiehq |
@gr2m @vkai Hey Greg, we just added some code to figure out this issue on Wednesday, but we didn't make test code. The reason is we have no idea where we should put code in the project. Where do we put the test code in this proejct? p.s) In issue #17 I already added |
I’ve merged #36 now, you can add tests for this to |
@gr2m Thanks Greg! |
properties.timeout
is optional and defaults to7200
(2h in seconds).If token is expired, then
sessions.add({token: 'expiredtoken123'})
should reject with404 token expired
The text was updated successfully, but these errors were encountered: