Online Accounts: Sign-on UI

signon-ui fails to start in Ubuntu Touch

Bug #1237496 reported by Víctor R. Ruiz
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Online Accounts: Sign-on UI
Fix Released
Critical
Alberto Mardegan
unity-mir
Fix Released
Critical
Unassigned
unity-mir (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Using Ubuntu Touch image 20131009.1, with Mir and write permissions.

phablet@ubuntu-phablet:~$ export SSOUI_LOGGING_LEVEL=2
phablet@ubuntu-phablet:~$ export SSOUI_DAEMON_TIMEOUT=3000
phablet@ubuntu-phablet:~$ signon-ui
__pthread_gettid -2
terminate called after throwing an instance of 'std::system_error'
  what(): Operation not permitted
Aborted (core dumped)

Tags: qa-touch

Related branches

lp:~mardy/signon-ui/lp1237496
Rejected for merging into lp:signon-ui
Ken VanDine: Disapprove
PS Jenkins bot (community): Approve (continuous-integration)
Diff: 78 lines (+21/-6)
5 files modified
.bzrignore (+1/-0)
src/browser-process/DefaultPage.qml (+2/-4)
src/com.nokia.singlesignonui.service.in (+1/-1)
src/signon-ui.desktop.in (+11/-0)
src/signon-ui.pro (+6/-1)
lp:~ken-vandine/unity-mir/lp1235337
PS Jenkins bot (community): Approve (continuous-integration)
Mir development team: Pending requested
Diff: 13 lines (+2/-1)
1 file modified
src/modules/Unity/Application/application_manager.cpp (+2/-1)
lp:ubuntu/saucy-proposed/unity-mir
Víctor R. Ruiz (vrruiz)
Changed in signon-ui:
importance: Undecided → Critical
Revision history for this message
David Barth (dbarth) wrote :

This only happens with mir. I can't get the crasher with surfaceflinger.
I suspect it is the Qt API that puts the signon-ui on top of the window stack that is triggering this regression.

Revision history for this message
David Barth (dbarth) wrote :

unity8.log further shows that the app gets rejected (as pointed out by ricmm and om26er):

ApplicationManager REJECTED connection from app with pid 2429 as no desktop_file_hint specified

David Barth (dbarth)
Changed in unity-mir:
importance: Undecided → Critical
Revision history for this message
Michał Sawicz (saviq) wrote :

You need to a) start it via upstart or b) pass --desktop_file_hint=

Changed in unity-mir:
status: New → Won't Fix
Revision history for this message
Ken VanDine (ken-vandine) wrote :

It's dbus activated.

Revision history for this message
Alberto Mardegan (mardy) wrote :

Just tested running signon-ui with the --desktop_file_hint flag set, and everything works fine.

Ricardo said that the requirement for --desktop_file_hint could be removed, that's why I'm adding unity-mir to this bug. If for some reason you can't remove that, just let me know and I'll look for another solution.

Revision history for this message
Ken VanDine (ken-vandine) wrote :

Adding the --desktop_file_hint to signon-ui does fix the crash on start, but it is kind of ugly. Even though signon-ui itself never provides any UI, it still shows up in the running applications list in unity8, along with browser-process which signon-ui spawns with the --desktop_file_hint to provide the webview to login. So the user sees "Account authentication" twice in the running applications, one of which they can't switch to.

Revision history for this message
Ken VanDine (ken-vandine) wrote :

I've proposed a fix from Gerry for unity-mir.

Changed in unity-mir:
status: Won't Fix → Confirmed
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:unity-mir at revision 114, scheduled for release in unity-mir, milestone phone-v1-freeze

Changed in unity-mir:
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity-mir - 0.1+13.10.20131011-0ubuntu1

---------------
unity-mir (0.1+13.10.20131011-0ubuntu1) saucy; urgency=low

  [ Ken VanDine ]
  * added exception for signon-ui (LP: #1237496) . (LP: #1237496)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 114
 -- Ubuntu daily release <email address hidden> Fri, 11 Oct 2013 05:41:03 +0000

Changed in unity-mir (Ubuntu):
status: New → Fix Released
Michał Sawicz (saviq)
Changed in unity-mir:
status: Fix Committed → Fix Released
Víctor R. Ruiz (vrruiz)
Changed in signon-ui:
status: New → Fix Released
Revision history for this message
David Barth (dbarth) wrote :

Re-opening, as tests are still failing on a recent image, though not necessarily for the same reason.

Changed in signon-ui:
status: Fix Released → Triaged
Revision history for this message
David Barth (dbarth) wrote :

The problem has been fixed and signon-ui works properly with Mir on the phone now.

Changed in signon-ui:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.