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

RESOLVED FIXED in 3.8

Status

Calendar
Provider: CalDAV
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

Details

(Whiteboard: [calconnect31-newissues])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
Created attachment 8496816 [details] [diff] [review]
Fix - v1

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+
(Assignee)

Comment 2

4 years ago
Created attachment 8500539 [details] [diff] [review]
Fix - v1
Attachment #8496816 - Attachment is obsolete: true
(Assignee)

Updated

4 years ago
Keywords: checkin-needed

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 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.