Bug 346068 - Google Signon UI takes ages to load
Summary: Google Signon UI takes ages to load
Status: RESOLVED NOT A BUG
Alias: None
Product: telepathy
Classification: Unmaintained
Component: accounts-kcm (show other bugs)
Version: git-latest
Platform: Other Linux
: NOR normal
Target Milestone: Future
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-11 11:19 UTC by David Edmundson
Modified: 2015-04-13 19:51 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Edmundson 2015-04-11 11:19:07 UTC
Went to set up a google account, clicked google. Nothing happened for about 20 seconds. I had thought it was broken and closed the window before the signon UI appeared. Happens every time.
Comment 1 Martin Klapetek 2015-04-13 10:40:34 UTC
Looks like a dbus timeout.

Does that happen everytime? Can you provide a bustle/dferry log and possibly also the output of kcmshell5 kaccounts?
Comment 2 David Edmundson 2015-04-13 10:48:08 UTC
Think it might have just been a slow network. seems fine today.

Would be nice if it showed the window whilst it loaded.
Comment 3 Martin Klapetek 2015-04-13 14:03:30 UTC
Indeed, signon-ui shows the dialog only on QWebView::loadFinished() signal, which isn't ideal.

I'll see if that can be improved.
Comment 4 Martin Klapetek 2015-04-13 19:51:01 UTC
For the record keeping purposes:

<mck182> hey, what would you think about this patch for signon-ui https://paste.kde.org/p8w8onz9a
<mck182> this way it appears immediately with the spinning wheel and it just happily spins away while the page is being loaded
<mardy> I don't like it, because it often happens that once the webpage is loaded, it immediately redirects you to the callback page, and in that case the current code doesn't show any UI at all
<mardy> the idea is that the client would use a spinner or some other indicator while performing the authentication
<mardy> there's also an underused feature in signond, that lets authentication plugins send progress reports to the client
<mardy> anyway, if you disagree, I don't mind having some #ifdef there
<mardy> ah, but no, ifdef is not very good
<mardy> there might be an Ubuntu machine where signon-ui is used by both KDE and Unity
<mardy> maybe an enviornment variable, or config file?

I think I'll just resort to set the whole UI to disabled and add some spinning wheels here and there.