Closed
Bug 919566
Opened 11 years ago
Closed 8 years ago
Cannot login on developer.jcdecaux.com
Categories
(Web Compatibility :: Desktop, defect)
Web Compatibility
Desktop
Tracking
(firefox24 affected, firefox25 affected, firefox26 affected, firefox27 affected, firefox-esr17 unaffected, firefox-esr24 affected)
RESOLVED
FIXED
People
(Reporter: mathieu.marquer, Unassigned)
References
()
Details
(Whiteboard: [country-fr] )
Attachments
(2 files)
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:27.0) Gecko/20100101 Firefox/27.0 (Beta/Release)
Build ID: 20130922030201
Steps to reproduce:
Go on https://developer.jcdecaux.com/#/sign
Try to login with a valid account
Actual results:
If the username/password combination is correct, nothing happens.
Expected results:
You should be logged in the website. Works with latest Chrome stable. Error in the browser console:
angular.min.js:60
Error: $scope.user is undefined
onLoginSubmit@https://developer.jcdecaux.com/res/script/controller/SignCtrl.js:75
SignCtrl/$scope.submit@https://developer.jcdecaux.com/res/script/controller/SignCtrl.js:13
Lc/z/<@https://developer.jcdecaux.com/res/lib/angularjs/angular.min.js:70
e.prototype.$eval@https://developer.jcdecaux.com/res/lib/angularjs/angular.min.js:86
e.prototype.$apply@https://developer.jcdecaux.com/res/lib/angularjs/angular.min.js:86
Dd</<@https://developer.jcdecaux.com/res/lib/angularjs/angular.min.js:141
st.event.dispatch@https://developer.jcdecaux.com/res/lib/jquery/jquery-1.9.0.min.js:2
st.event.add/y.handle@https://developer.jcdecaux.com/res/lib/jquery/jquery-1.9.0.min.js:2
Comment 1•11 years ago
|
||
Thanks for the report!
Can you try with the current release version (i.e., 24) and see if that works? And if so, maybe even Beta and Aurora? The important part for us is to know whether this is a regression on our side or a compatibility issue with the site.
Reporter | ||
Comment 2•11 years ago
|
||
Works with FF24, 25b1, doesn't work with 26a2, current nightly (all tested this time with Linux x64, I don't currently have a Windows with me but as the original report said it doesn't work with Windows 7 x64 with latest nightly).
Comment 3•11 years ago
|
||
Great, thanks - that helps a lot.
WFM with Firefox 27.
Guest account:
U: bugzilla@yopmail.net
P: azerty
Could you test with a clean profile, please.
https://support.mozilla.org/fr/kb/utiliser-gestionnaire-profils-creer-supprimer-profils (I guess you're French)
Flags: needinfo?(mathieu.marquer)
Sorry, it doesn't work with the current Nightly, mine wasn't updated since 2 days.
Flags: needinfo?(mathieu.marquer)
Reporter | ||
Comment 6•11 years ago
|
||
It does indeed work with the bugzilla@yopmail.net, but not with mine... (same in safe mode).
Reporter | ||
Comment 7•11 years ago
|
||
OK, found the track: if the username/password is saved in the browser, and the fields are consequently autocompleted on the page, it won't work and the console will give the JS error above. If the username/password are entered manually, then it works.
Yes, here the STR:
1) Open https://developer.jcdecaux.com/#/sign
2) Login with bugzilla@yopmail.net / azerty and save the password in Firefox
3) Disconnect to the website
4) Autocomplete the login input
Result: the website displays "wrong password".
Regression range:
good=2013-06-20
bad=2013-06-21
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=8ea92aeab783&tochange=7ba8c86f1a56
Before 2013-06-20, Firefox wasn't able to autocomplete the password field when the login is entered, after 2013-06-21 the bug appears.
Comment 9•11 years ago
|
||
Bad: password field is empty.
http://hg.mozilla.org/integration/mozilla-inbound/rev/6ee152636ad4
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20130619 Firefox/24.0 ID:20130619155313
Bad: password field is filled, but the website displays "wrong password".
http://hg.mozilla.org/integration/mozilla-inbound/rev/1213ea1ff91a
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20130619 Firefox/24.0 ID:20130619163255
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=6ee152636ad4&tochange=1213ea1ff91a
Suspected: Bug 839961
I think the site require "input event" to password field.
After password field is auto filled,
Execute the following JS code in Web Console. And Click "Sign in". Then successfully logged in.
var evt = document.createEvent("HTMLEvents");
evt.initEvent("input", true, true );
document.getElementById("password").dispatchEvent(evt);
Updated•11 years ago
|
Blocks: 839961
status-firefox24:
--- → affected
status-firefox25:
--- → affected
status-firefox26:
--- → affected
status-firefox27:
--- → affected
status-firefox-esr17:
--- → unaffected
status-firefox-esr24:
--- → affected
Component: JavaScript Engine → Password Manager
Product: Core → Toolkit
Version: 26 Branch → 24 Branch
Updated•11 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 10•11 years ago
|
||
Thanks for the investigation in comments 8 and 9.
Sounds like this is basically a race condition in the site. Presumably the |input| event from the autofill is simply happening before the site has a listener attached, since we fill in the password earlier now. Not much we can do about that, please contact the site owner and ask them to fix their code.
I'd suspect a workaround is to click in the password field, add any character to the existing password, and then delete it.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
Comment 11•11 years ago
|
||
Let's move this to tech evangelism instead of closing it as invalid, then.
Status: RESOLVED → REOPENED
Component: Password Manager → English US
Keywords: regressionwindow-wanted
Product: Toolkit → Tech Evangelism
Resolution: INVALID → ---
Version: 24 Branch → unspecified
Comment 12•11 years ago
|
||
No "input" event fire for the password field when the password field is auto-filled.
Is this intentional behavior?
Comment 13•11 years ago
|
||
(In reply to Alice0775 White from comment #12)
> Created attachment 809483 [details]
> passwd.html
>
No "input" event fire for the password field when the password field is auto-filled.
Is this intentional behavior?
Flags: needinfo?(dolske)
Comment 14•11 years ago
|
||
I thought it did (implicitly, from the field being modified), but maybe not? Password manager itself certainly never created one. Still a race, if the site was just relying on passwords only being filed after the page loads.
Flags: needinfo?(dolske)
Reporter | ||
Comment 15•11 years ago
|
||
Thanks for investigating, I sent them an email through their contact page.
Reporter | ||
Comment 16•11 years ago
|
||
They acknowledged my email and said they would fix it "ASAP".
Assignee | ||
Updated•10 years ago
|
Assignee: general → nobody
Comment 17•10 years ago
|
||
There is another issue going on. See the screenshot with Firefox on Top and Opera Blink on the bottom.
The site is totally unusable currently.
https://developer.jcdecaux.com/#/home
16:44:37.820 "Error: h is undefined
@https://developer.jcdecaux.com/scripts/b2b31aab.scripts.js:1:17300
d@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:13473
e@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:13585
Nb/this.$get</<@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:29730
q/<@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:22717
f@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:3:30886
q@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:22582
h@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:19244
C/<@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:18937
$/e/</<@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4407
wc/this.$get</i.prototype.$eval@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:18740
wc/this.$get</i.prototype.$apply@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:19024
$/e/<@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4365
d@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:13473
$/e@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4347
$@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4458
Z@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4000
@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:7:6351
kb.Callbacks/l@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:2:4634
kb.Callbacks/m.fireWith@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:2:5437
.ready@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:1:14066
xb@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:1:11413
" 59cdab37.vendor.js:5
16:44:37.821 "Error: a.i18n is undefined
f@https://developer.jcdecaux.com/scripts/b2b31aab.scripts.js:1:16314
@https://developer.jcdecaux.com/scripts/b2b31aab.scripts.js:1:17188
wc/this.$get</i.prototype.$digest@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:17769
wc/this.$get</i.prototype.$apply@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:19068
$/e/<@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4365
d@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:13473
$/e@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4347
$@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4458
Z@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:4:4000
@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:7:6351
kb.Callbacks/l@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:2:4634
kb.Callbacks/m.fireWith@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:2:5437
.ready@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:1:14066
xb@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:1:11413
" 59cdab37.vendor.js:5
16:44:38.152 "Error: a.i18n is undefined
f@https://developer.jcdecaux.com/scripts/b2b31aab.scripts.js:1:16314
wc/this.$get</i.prototype.$broadcast@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:19852
m/<@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:7:17091
vc/h/g.promise.then/j@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:14066
vc/h/g.promise.then/j@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:14066
vc/i/<.then/<@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:14753
wc/this.$get</i.prototype.$eval@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:18740
wc/this.$get</i.prototype.$digest@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:17498
wc/this.$get</i.prototype.$apply@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:19068
f@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:525
r@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:2289
Wb/</w.onreadystatechange@https://developer.jcdecaux.com/scripts/59cdab37.vendor.js:5:2825
" 59cdab37.vendor.js:5
Updated•10 years ago
|
Whiteboard: [country-fr]
Updated•10 years ago
|
Component: English US → Desktop
Comment 18•10 years ago
|
||
The website WFM in FF32/35. Did you test with a clean profile?
Comment 19•10 years ago
|
||
Site looks fine here.
I think the password manager should send change events for the form elements, but I'm not sure about input. Does it send keydown/up/press events? If yes, it certainly should send input. If no, I'm not sure.
Comment 20•8 years ago
|
||
It has been fixed. It is working.
Status: REOPENED → RESOLVED
Closed: 11 years ago → 8 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•6 years ago
|
Product: Tech Evangelism → Web Compatibility
You need to log in
before you can comment on or make changes to this bug.
Description
•