Closed
Bug 971995
Opened 10 years ago
Closed 10 years ago
Server needs to extract and validate information from temporary URI
Categories
(Hello (Loop) :: Server, defect)
Hello (Loop)
Server
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: abr, Assigned: rgauthier)
References
Details
User Story
The temporary URI will contain information including an expiry, and an HMAC that needs to be validated before allowing a call to proceed.
Attachments
(1 file)
No description provided.
Reporter | ||
Updated•10 years ago
|
Component: General → Server
Comment 1•10 years ago
|
||
The temporary URI will contain information including an expiry, and an HMAC that needs to be validated before allowing a call to proceed.
Comment 2•10 years ago
|
||
Code is here: https://github.com/ametaireau/pants-server/blob/master/pants/views.py#L13-L19 tests are here: https://github.com/ametaireau/pants-server/blob/master/tests/test_functional.py#L32-L44 We weren't sure what do we want for expiration, and if that was needed to encapsulate the simplepush url in the token payload so we haven't done anything about that. I would need more information about the UX we want here: what's the duration of an URL? Should it be one-shot only?
Updated•10 years ago
|
User Story: (updated)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → rgauthier
Comment 3•10 years ago
|
||
Attachment #8384172 -
Flags: review?(ekr)
Updated•10 years ago
|
Attachment #8384172 -
Flags: review?(ekr)
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 4•10 years ago
|
||
Closing since the code has changed a lot since this was opened. Plus we are not using https://github.com/ametaireau/pants-server anymore ;-)
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•