Access token date is broken for public requests

RESOLVED FIXED in mozilla23

Status

()

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jdm, Assigned: sankha)

Tracking

unspecified
mozilla23
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mentor=jdm][lang=js])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
>      // check to see if it has expired
>      let accessTokenDate;
>      if (this.lastRequestPrivate) {
>        accessTokenDate = gPrivateAccessTime;
>      } else {
>        Services.prefs.getIntPref(accessTokenPrefName + ".time");
>      }
(Reporter)

Comment 1

6 years ago
In particular, we need to set accessTokenDate to the result of the pref.
Whiteboard: [mentor=jdm][lang=js]
(Assignee)

Comment 2

6 years ago
Created attachment 732264 [details] [diff] [review]
patch v1

Can you review this?
Assignee: nobody → sankha93
Attachment #732264 - Flags: review?(josh)
(Reporter)

Comment 3

6 years ago
Comment on attachment 732264 [details] [diff] [review]
patch v1

Review of attachment 732264 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #732264 - Flags: review?(josh) → review+
https://hg.mozilla.org/mozilla-central/rev/892800c367ab
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.