Closed Bug 458858 Opened 16 years ago Closed 16 years ago

Lightning keeps asking for password on startup

Categories

(Calendar :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: davida, Assigned: Fallen)

Details

(Keywords: dogfood)

Attachments

(1 file)

Even though i have the 'remember password' checkbox saved.  And it seems to save it (the form is prefilled), so the dialog seems pointless

Cc'ing standard8 as I expect the password manager migration will have something to do with this.

(this is for a caldav/zimbra calendar, btw)
Flags: tb-integration+
Flags: blocking-calendar1.0?
Flags: blocking-calendar1.0? → blocking-calendar1.0+
Quite annoying to debug, since for me venkman is either crashing or hanging. With gdata this is even worse, I get tons of login dialogs at the same time.
Keywords: dogfood
Attached patch Fix - v1 β€” β€” Splinter Review
This takes care. The problem was that aChannel.URI.port was -1 since it uses the default port in most cases. This patch asks the protocol handler for the default port if no port is given.

For GData, it seems trunk has better concurrency, so lots of login dialogs were shown. Moving this.mLoggingIn takes care.

Also fixes a problem with the command handlers, I had a couple of too much recursion errors on the way.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #343047 - Flags: review?(daniel.boelzle)
Attachment #343047 - Flags: review?(daniel.boelzle) → review+
Comment on attachment 343047 [details] [diff] [review]
Fix - v1

>+        var port = aChannel.URI.port;
use let instead of var (throughout the patch)

>+        if (port == -1) {
>+            var handler = getIOService().getProtocolHandler(aChannel.URI.scheme)
use cal.getIOService() from calUtils.jsm

looks good; r=dbo
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/e99df1f6bfb8>

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
GData part checked in on HEAD, MOZILLA_1_8_BRANCH and SUNBIRD_0_9_BRANCH to ease a possible further branch release of the gdata provider (npotdb).
These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.