Closed Bug 1074159 Opened 10 years ago Closed 10 years ago

Freebusy requests fail with Google CalDAV if the first calendar in the realm is disabled

Categories

(Calendar :: Provider: CalDAV, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

Details

(Whiteboard: [calconnect31-newissues])

Attachments

(1 file, 1 obsolete file)

Attached patch Fix - v1 (obsolete) — — Splinter Review
1. Subscribe to multiple Google CalDAV calendars
2. Do something that the first Google calendar returned by calmgr.getCalendars() is disabled
3. Trigger a freebusy request


Result:

The Authorization header is not sent because oauth is not initialized


Expected:

The authorization header is sent.

The patch for this bug might actually fix the bug that disabled calendars are queried for FB info because firstInRealm will return false if all calendars are disabled.
Attachment #8496816 - Flags: review?(matthew.mecca)
Comment on attachment 8496816 [details] [diff] [review]
Fix - v1

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

Looks good. r=mmecca
Attachment #8496816 - Flags: review?(matthew.mecca) → review+
Attached patch Fix - v1 — — Splinter Review
Attachment #8496816 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 3.6
Target Milestone: 3.6 → 3.8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: