trust-store

no trust prompt appearing in clock or weather app after images on the 4th of may

Bug #1578665 reported by Dave Morley
26
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Critical
Michał Sawicz
Canonical System Image 11 "OTA11"
trust-store
Invalid
Undecided
Unassigned
qtmir (Ubuntu)
Fix Released
Critical
Daniel d'Andrada
unity8 (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

STEPS:
1. Bootstrap the latest rc-proposed
2. Open clock app after a long start up time you will note that you were not asked
3. Open the weather app and see similar behaviour

EXPECTED:
All most instant popup for trust store to allow location access

ACTUAL:
A long delay before the app opens and then no trust prompt and no location detection.

Tags: lt-blocker

Related branches

lp://staging/~dandrader/qtmir/promptBeforeSurfaceDraws
Nick Dedekind (community): Approve
Unity8 CI Bot (community): Approve (continuous-integration)
Diff: 138 lines (+78/-2)
5 files modified
src/modules/Unity/Application/mirsurfacelistmodel.cpp (+5/-0)
src/modules/Unity/Application/mirsurfacelistmodel.h (+1/-0)
src/modules/Unity/Application/session.cpp (+23/-2)
src/modules/Unity/Application/session.h (+6/-0)
tests/modules/SessionManager/session_test.cpp (+43/-0)
lp://staging/~dandrader/qtmir/surfaceDrawn
Superseded for merging into lp://staging/qtmir
Nick Dedekind (community): Approve (code)
Unity8 CI Bot (community): Needs Fixing (continuous-integration)
Diff: 399 lines (+32/-57)
17 files modified
CMakeLists.txt (+1/-1)
debian/control (+2/-2)
src/modules/Unity/Application/mirsurface.cpp (+1/-1)
src/modules/Unity/Application/mirsurface.h (+2/-2)
src/modules/Unity/Application/mirsurfaceinterface.h (+0/-3)
src/modules/Unity/Application/mirsurfacemanager.cpp (+2/-6)
src/modules/Unity/Application/session.cpp (+0/-13)
src/modules/Unity/Application/session.h (+1/-3)
src/modules/Unity/Application/session_interface.h (+1/-1)
tests/framework/fake_mirsurface.cpp (+2/-2)
tests/framework/fake_mirsurface.h (+2/-1)
tests/framework/fake_session.cpp (+1/-1)
tests/framework/fake_session.h (+1/-1)
tests/framework/mock_session.h (+1/-1)
tests/modules/Application/application_test.cpp (+8/-8)
tests/modules/ApplicationManager/application_manager_test.cpp (+1/-1)
tests/modules/SessionManager/session_test.cpp (+6/-10)
lp://staging/~dandrader/unity8/surfaceDrawn
Rejected for merging into lp://staging/unity8
Daniel d'Andrada (community): Disapprove
Unity8 CI Bot: Needs Fixing (continuous-integration)
Nick Dedekind (community): Needs Fixing
Diff: 400 lines (+107/-23)
8 files modified
CMakeLists.txt (+1/-1)
debian/control (+3/-3)
qml/Stages/ApplicationWindow.qml (+44/-8)
qml/Stages/SurfaceContainer.qml (+2/-1)
tests/mocks/Unity/Application/ApplicationInfo.cpp (+9/-1)
tests/mocks/Unity/Application/MirSurface.cpp (+8/-0)
tests/mocks/Unity/Application/MirSurface.h (+5/-0)
tests/qmltests/Stages/tst_ApplicationWindow.qml (+35/-9)
Revision history for this message
Dave Morley (davmor2) wrote :

Will bisect when this came in latter today hopefully.

Revision history for this message
Víctor R. Ruiz (vrruiz) wrote :

I confirmed with Weather app.

Changed in trust-store:
status: New → Confirmed
Łukasz Zemczak (sil2100)
tags: added: lt-blocker
Changed in canonical-devices-system-image:
milestone: none → 11
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
assignee: nobody → Pat McGowan (pat-mcgowan)
importance: Undecided → Critical
status: New → Confirmed
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

@dave did you ever bisect?

Revision history for this message
Dave Morley (davmor2) wrote :

No looking at it this morning as I have some breathing space again.

Revision history for this message
Dave Morley (davmor2) wrote :

324 from the 3rd works looking at 325 from the 5th now

Revision history for this message
Dave Morley (davmor2) wrote :

Was introduced in image 325 http://people.canonical.com/~lzemczak/landing-team/ubuntu-touch/rc-proposed/bq-aquaris.en/krillin/325.commitlog which had seed changes, unity8, nm1.2, aethercast, and the list goes on.

Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
assignee: Pat McGowan (pat-mcgowan) → Thomas Voß (thomas-voss)
Revision history for this message
Thomas Voß (thomas-voss) wrote :

With #6, I'm marking Unity8 as affected. A bunch of changes for Unity8/QtUbuntu et al. went in, whereas no changes for trust-store or location-service landed.

Changed in trust-store:
status: Confirmed → Invalid
Revision history for this message
Michał Sawicz (saviq) wrote :

Here's a log from starting weather app - we can see prompt being created but indeed it's never displayed:

http://pastebin.ubuntu.com/16345317/

Changed in canonical-devices-system-image:
assignee: Thomas Voß (thomas-voss) → Michał Sawicz (saviq)
Daniel d'Andrada (dandrader)
Changed in qtmir (Ubuntu):
assignee: nobody → Daniel d'Andrada (dandrader)
Michał Sawicz (saviq)
Changed in qtmir (Ubuntu):
status: New → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Michał Sawicz (saviq)
affects: unity8 → unity8 (Ubuntu)
Launchpad Janitor (janitor)
Changed in unity8 (Ubuntu):
status: New → Confirmed
Daniel d'Andrada (dandrader)
Changed in qtmir (Ubuntu):
importance: Undecided → Critical
Daniel d'Andrada (dandrader)
Changed in unity8 (Ubuntu):
status: New → Invalid
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
status: Confirmed → In Progress
Jean-Baptiste Lallement (jibel)
Changed in canonical-devices-system-image:
status: In Progress → Fix Committed
Revision history for this message
Michal Predotka (mpredotka) wrote :

M10 rc-proposed r105
Today the Clock app started fine with the prompt about allowing location access. Unfortunately there is a problem with an app which needs microphone access. It works fine on my bq e4.5 with stable channel but does not show the access prompt on M10 with rc-proposed. The app is called 'Guitar Tools'.

Revision history for this message
Michal Predotka (mpredotka) wrote :

Same problem with 'Tuner' app.

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Confirmed seems another instance
I see the trust prompt running but its hidden
 5602 ? Sl 0:00 /usr/lib/arm-linux-gnueabihf/trust-prompt --mir_server_socket fd://17 --icon /opt/click.ubuntu.com/.click/users/phablet/guitar-tools.t-mon/guitar-tools/guitar-tools.png --name Guitar

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

Tracking this issue with Tuner and Guitar Tools in bug 1581559.

Changed in qtmir (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtmir - 0.4.8+16.10.20160520.1-0ubuntu1

---------------
qtmir (0.4.8+16.10.20160520.1-0ubuntu1) yakkety; urgency=medium

  [ Alan Griffiths ]
  * Reduce coupling to MirServer - it has been misused as a context
    object.

  [ Daniel d'Andrada ]
  * Also interpret the cursor names used by Mir client API (LP:
    #1447839)
  * Application: Don't respawn if closed while still starting up (LP:
    #1575577)

  [ Daniel d'Andrada, Michał Sawicz ]
  * Centralize logging categories

  [ Gerry Boland ]
  * UAL can throw on creating Application if invalid appId, catch
    instead of aborting (LP: #1578258)

  [ Timo Jyrinki ]
  * Use FindQt5PlatformSupport to find platform support, other methods
    not available on Qt 5.6 anymore. (LP: #1554404)

 -- Michał Sawicz <email address hidden> Fri, 20 May 2016 08:41:51 +0000

Changed in qtmir (Ubuntu):
status: Fix Committed → Fix Released
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Bug attachments

Remote bug watches

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