Error: TypeError: this.mDeferred is null in gdataRequest.jsm

RESOLVED FIXED in 4.1

Status

Calendar
Provider: GData
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: rkent, Assigned: darktrojan)

Tracking

Lightning 4.0.0.1

Details

Attachments

(2 attachments)

(Reporter)

Description

2 years ago
Error console, repeated many times:

Error: TypeError: this.mDeferred is null
Source File: resource://gdata-provider/modules/gdataRequest.jsm
Line: 244
Any other errors, maybe the one from bug 1101175 ? With calendar.debug.log and calendar.debug.log.verbose enabled, what messages immediately preceed that error?
(Assignee)

Comment 2

2 years ago
I get this too, although not very often. There's one instance on my console now, with no other messages in the 6 minutes before it.

I also noticed that the non-all-day events on my Google calendar are now missing. If I restart Thunderbird, the all-day events will disappear too.
(Assignee)

Comment 3

2 years ago
Created attachment 8571913 [details]
error.log

This bug happened to me again (might be something different from my previous comment), and I managed to get the log from this calendar refresh and the two before it. That might help figure out why 401s are occurring in the first place.

(The error in the last line occurred 7 times, it just doesn't show up like that when I cut and paste.)
(Assignee)

Comment 4

2 years ago
> 00:41:25.699 [calGoogleSession] Token will expire in 0 seconds, refreshing

Perhaps we shouldn't try to reuse a token the very second it expires.
Attachment #8571913 - Attachment mime type: text/x-log → text/plain
(Assignee)

Comment 5

2 years ago
Created attachment 8574259 [details] [diff] [review]
1137455-1.diff

Fairly certain that here's the problem. Request fails, authError. Re-auth. Request succeeds, succeed() resolves the promise. Next function in promise chain from original request calls succeed() again.
Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #8574259 - Flags: review?(philipp)
Comment on attachment 8574259 [details] [diff] [review]
1137455-1.diff

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

Indeed, this might actually fix it. Can you set a commit message and set checkin-needed? I'll see that this gets to be part of 1.0.5
Attachment #8574259 - Flags: review?(philipp) → review+
(Assignee)

Comment 7

2 years ago
https://hg.mozilla.org/comm-central/rev/43811e5e942c

(oops, forgot the reviewer in the commit message)
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
OS: Windows 8.1 → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → 4.1

Updated

2 years ago
Duplicate of this bug: 1144606
You need to log in before you can comment on or make changes to this bug.