Online accounts pollutes app's debug output

Bug #1404267 reported by Michael Zanetti
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Client Developer Experience
Fix Released
High
David Barth
accounts-qml-module
Fix Released
High
Alberto Mardegan
webapps-sprint
Fix Released
High
Alberto Mardegan
libaccounts-qt (Ubuntu)
Fix Released
Undecided
Alberto Mardegan
signon (Ubuntu)
Fix Released
Undecided
Alberto Mardegan

Bug Description

Using online accounts in an app prints a lot of debug prints which is really annoying when trying to debug something in the app. This is an excerpt of what's printed by online accounts when starting an app using it:

virtual void OnlineAccounts::Plugin::registerTypes(const char*) Ubuntu.OnlineAccounts
virtual void OnlineAccountsClient::Plugin::registerTypes(const char*) Ubuntu.OnlineAccounts.Client
account-service-model.cpp 292 update
account-service-model.cpp 292 update
account-service.cpp 100 setObjectHandle Accounts::AccountService(0x212ffb0)
account-service.cpp 393 authenticate QMap()
../../../../lib/SignOn/connection-manager.cpp 106 setupSocketConnection p2p error: QDBusError("org.freedesktop.DBus.Error.FileNotFound", "Failed to connect to socket /run/user/32011/signond/socket: No such file or directory") 1
../../../../lib/SignOn/connection-manager.cpp 132 init Peer connection unavailable, activating service
../../../../lib/SignOn/identityimpl.cpp 96 updateState Updating state: "PendingRegistration" SignOn::IdentityImpl(0x1e48410)
../../../../lib/SignOn/authsessionimpl.cpp 94 initInterface
../../../../lib/SignOn/authsessionimpl.cpp 94 initInterface
../../../../lib/SignOn/connection-manager.cpp 106 setupSocketConnection p2p error: QDBusError("org.freedesktop.DBus.Error.FileNotFound", "Failed to connect to socket /run/user/32011/signond/socket: No such file or directory") 1
../../../../lib/SignOn/connection-manager.cpp 150 init Connected to "qt_default_session_bus"
../../../../lib/SignOn/identityimpl.cpp 96 updateState Updating state: "Ready" SignOn::IdentityImpl(0x1e48410)
../../../../lib/SignOn/identityimpl.cpp 96 updateState Updating state: "NeedsUpdate" SignOn::IdentityImpl(0x1e48410)
../../../../lib/SignOn/identityimpl.cpp 96 updateState Updating state: "PendingUpdate" SignOn::IdentityImpl(0x1e48410)
../../../../lib/SignOn/identityimpl.cpp 442 infoUpdated SERVER INFO UPDATED. NeedsUpdate " 3 "

In order to increase the app developer experience OA shouldn't print anything except critical errors that prevent it from working.

Changed in canonical-developer-experience:
assignee: nobody → David Barth (dbarth)
importance: Undecided → High
status: New → Confirmed
Changed in accounts-qml-module:
assignee: nobody → Alberto Mardegan (mardy)
Revision history for this message
David Barth (dbarth) wrote : Re: [Bug 1404267] Re: Online accounts pollutes app's debug output

Do you have a ~/.config/signond.conf file which sets the LoggingLevel may
be? Or changed the /etc/signond.conf ?
Otherwise, agreed, this should not be as verbose in the default case.

On Fri, Oct 23, 2015 at 5:39 PM, Pat McGowan <email address hidden>
wrote:

> ** Changed in: canonical-developer-experience
> Importance: Undecided => High
>
> ** Changed in: canonical-developer-experience
> Status: New => Confirmed
>
> ** Changed in: canonical-developer-experience
> Assignee: (unassigned) => David Barth (dbarth)
>
> ** Changed in: accounts-qml-module
> Assignee: (unassigned) => Alberto Mardegan (mardy)
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/1404267
>
> Title:
> Online accounts pollutes app's debug output
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/accounts-qml-module/+bug/1404267/+subscriptions
>

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

This is not configurable at the moment. We should indeed hide all this messages, by default.

Changed in accounts-qml-module:
status: New → Confirmed
Changed in signon (Ubuntu):
status: New → Confirmed
assignee: nobody → Alberto Mardegan (mardy)
David Barth (dbarth)
Changed in webapps-sprint:
status: New → Confirmed
importance: Undecided → High
milestone: none → sprint-15
assignee: nobody → Alberto Mardegan (mardy)
David Barth (dbarth)
Changed in webapps-sprint:
milestone: sprint-15 → sprint-16
Alberto Mardegan (mardy)
Changed in libaccounts-qt (Ubuntu):
status: New → Confirmed
assignee: nobody → Alberto Mardegan (mardy)
Alberto Mardegan (mardy)
Changed in accounts-qml-module:
status: Confirmed → In Progress
Changed in libaccounts-qt (Ubuntu):
status: Confirmed → In Progress
Changed in signon (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package signon - 8.58+16.04.20151106-0ubuntu1

---------------
signon (8.58+16.04.20151106-0ubuntu1) xenial; urgency=medium

  [ Alberto Mardegan ]
  * Merge from upstream
    - Lib: hide debug output by default (LP: #1404267)

  [ CI Train Bot ]
  * No-change rebuild.

 -- David Barth <email address hidden> Fri, 06 Nov 2015 09:56:25 +0000

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

This bug was fixed in the package libaccounts-qt - 1.14+16.04.20151106.1-0ubuntu1

---------------
libaccounts-qt (1.14+16.04.20151106.1-0ubuntu1) xenial; urgency=medium

  [ Alberto Mardegan ]
  * Merge from upstream:
    - Modernize the project structure
    - Remove debug messages (LP: #1404267)

  [ CI Train Bot ]
  * No-change rebuild.

 -- David Barth <email address hidden> Fri, 06 Nov 2015 13:54:55 +0000

Changed in libaccounts-qt (Ubuntu):
status: In Progress → Fix Released
David Barth (dbarth)
Changed in accounts-qml-module:
status: In Progress → Fix Released
Changed in webapps-sprint:
status: Confirmed → Fix Committed
status: Fix Committed → Fix Released
Changed in canonical-developer-experience:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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