After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 759464 - osm login box is unforgiving
osm login box is unforgiving
Status: RESOLVED FIXED
Product: gnome-maps
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gnome-maps-maint
gnome-maps-maint
Depends on:
Blocks:
 
 
Reported: 2015-12-14 18:46 UTC by Andreas Nilsson
Modified: 2016-01-30 11:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
osmEdit: Don't re-use OAuth proxy used for signing in (1.92 KB, patch)
2015-12-18 06:20 UTC, Marcus Lundblad
committed Details | Review

Description Andreas Nilsson 2015-12-14 18:46:12 UTC
It seems there is something very unforgiving with the login dialog.

1. Open the dialog and try to log in.
2. Get the password wrong.
3. It asks you to try again, but it will actually not even accept the correct password this time around.

If you get the password right the first time, it works as it should and you're allowed to log in.
Comment 1 Marcus Lundblad 2015-12-18 06:16:01 UTC
Hmm, at first I couldn't seem to reproduce it. But now I could.
Luckily, I found a solution :-)
Patch coming up.
Comment 2 Marcus Lundblad 2015-12-18 06:20:48 UTC
Created attachment 317605 [details] [review]
osmEdit: Don't re-use OAuth proxy used for signing in

It seems running request_token_async a second time on an OAuthProxy instance
doesn't work well.
So, instead create a new proxy each time the login process is being run.
Comment 3 Jonas Danielsson 2015-12-18 06:24:56 UTC
Review of attachment 317605 [details] [review]:

Thanks for the fix!

::: src/osmConnection.js
@@ -24,3 @@
 
-const Utils = imports.utils;
-

Utils disappeared, I hope it was meant to?
Comment 4 Marcus Lundblad 2015-12-18 06:33:12 UTC
(In reply to Jonas Danielsson from comment #3)
> Review of attachment 317605 [details] [review] [review]:
> 
> Thanks for the fix!
> 
> ::: src/osmConnection.js
> @@ -24,3 @@
>  
> -const Utils = imports.utils;
> -
> 
> Utils disappeared, I hope it was meant to?

Yes, actually Utils wasn't used, so I axed at while at it :-)
Comment 5 Jonas Danielsson 2016-01-30 11:16:54 UTC
Review of attachment 317605 [details] [review]:

Pushed,,, right?