Closed
Bug 433936
Opened 16 years ago
Closed 16 years ago
Cannot login to Litmus using QA Companion 0.1.13
Categories
(Other Applications Graveyard :: QA Companion, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: chris.blore, Assigned: cmtalbert)
Details
Attachments
(1 file)
7.42 KB,
patch
|
jay
:
review+
|
Details | Diff | Splinter Review |
Using the latest version of QA Companion (0.1.13), I am unable to login to Litmus. Steps to reproduce: 1. Enter Litmus account details. The Account Creation window opens regardless of whether or not you already have a Litmus account. 2. QA Companion does not login to Litmus and you receive the following error message: ERROR: [Exception... "'Can't add a login with both a httpRealm and formSubmitURL.' when calling method: [nsILoginManager::addLogin]" nsresult: "0x8057001e (NS_ERROR_XPC_JS_THREW_STRING)" location: "JS frame :: chrome://qa/content/prefs.js :: anonymous :: line 182" data: no] I am using FF3 RC1 Build 1: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9) Gecko/2008051202 Firefox/3.0
Comment 1•16 years ago
|
||
Looks like something got changed in login manager to break this. I'll look into it.
Assignee: nobody → zach
Severity: normal → critical
This patch fixes the error by complying with the nsILoginManager's new rules for submissions. It also improves our login handling for stored passwords. If you start QAC, it will do the following: * Check for your stored password * If found, will validate your login * If not found or if validation fails, it asks you to log into litmus * If you login successfully, it stores your information * If you do not login successfully, it asks you to log in again. The "first time" login experience is unchanged and unaffected by this.
(In reply to comment #1) > Looks like something got changed in login manager to break this. I'll look into > it. > Zach, Sorry, I didn't realize you were already looking into this. I went ahead and took it since we have a big RC1 test day tomorrow, and folks will probably want a QAC that works with ffx 3.0 RC1.
Comment 4•16 years ago
|
||
That's fine Clint. I had only spent a few minutes poking at it anyway. Thanks for grabbing it so quickly. Patch looks great to me. Perhaps we should direct people to irc for help if they have forgotten their passwords, since that's what the litmus login page says to do?
Comment 5•16 years ago
|
||
Comment on attachment 321176 [details] [diff] [review] Fixes the error and does better management of currently logged in sessions r=jay, but please bump the maxversion to 3.0.* before checkin, so I won't have to do it manually through the addons site next time. :-)
Attachment #321176 -
Flags: review?(jay) → review+
In reply to comment 4: Good idea, Zach, I'm going to change the text of the "second try" login message to: Incorrect login information. Please enter the email address and password for your Litmus Account, if you have forgotten it, ask on IRC in #qa to have it reset. In reply to comment 5: Done. Otherwise, the patch is the same as what is above. Checking in testing/extensions/community/install.rdf; /cvsroot/mozilla/testing/extensions/community/install.rdf,v <-- install.rdf new revision: 1.23; previous revision: 1.22 done Checking in testing/extensions/community/chrome/content/prefs.js; /cvsroot/mozilla/testing/extensions/community/chrome/content/prefs.js,v <-- prefs.js new revision: 1.7; previous revision: 1.6 done Checking in testing/extensions/community/chrome/content/qa.js; /cvsroot/mozilla/testing/extensions/community/chrome/content/qa.js,v <-- qa.js new revision: 1.8; previous revision: 1.7 done Checking in testing/extensions/community/chrome/locale/en-US/qa.properties; /cvsroot/mozilla/testing/extensions/community/chrome/locale/en-US/qa.properties,v <-- qa.properties new revision: 1.6; previous revision: 1.5 done --> FIXED
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•