Closed
Bug 987856
Opened 11 years ago
Closed 11 years ago
Loop Server — Add a "X-Timestamp" header when doing 200 and 401
Categories
(Hello (Loop) :: Server, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: alexis+bugs, Assigned: rhubscher)
References
Details
Attachments
(2 files)
X-Timestamp
This header will be included with all “200” and “401” responses, giving the current POSIX timestamp as seen by the server, in seconds. It may be useful for client to adjust their local clock when generating BrowserID assertions.
Updated•11 years ago
|
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8406900 -
Flags: review?(alexis+bugs)
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → rhubscher
Assignee | ||
Updated•11 years ago
|
Attachment #8406900 -
Flags: review?(nperriault)
Assignee | ||
Comment 2•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 3•11 years ago
|
||
Reopening because we don't want X- headers.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter | ||
Comment 4•11 years ago
|
||
Attachment #8407432 -
Flags: review?(nperriault)
Reporter | ||
Updated•11 years ago
|
Attachment #8406900 -
Flags: review?(nperriault)
Attachment #8406900 -
Flags: review?(alexis+bugs)
Attachment #8406900 -
Flags: review+
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 8407432 [details] [review]
link to github PR fixing the X- headers.
https://github.com/mozilla-services/loop-server/commit/43ec85ec1069a0e29f4c35331e6300c332834dd0
Attachment #8407432 -
Flags: review?(nperriault) → review+
Assignee | ||
Updated•11 years ago
|
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Comment 6•11 years ago
|
||
Out of curiosity, why don't we want any X- headers?
Reporter | ||
Comment 7•11 years ago
|
||
because that's deprecated :) http://tools.ietf.org/html/rfc6648
Comment 8•11 years ago
|
||
Hmmm, interesting.... ok
We are still using them for Sync 1.5, although not X-Timestamp:
https://docs.services.mozilla.com/storage/apis-1.5.html
Comment 9•10 years ago
|
||
Fix either went out with a previous deployment to Production or has been superceded by new code...
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•