App is unable to retrieve notes

Bug #1351041 reported by Dave Morley
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu Notes app
Fix Released
Critical
Unassigned
accounts-qml-module
New
Undecided
Unassigned
accounts-qml-module (Ubuntu)
New
Undecided
Unassigned
reminders-app (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

If you have notes online you can now no longer retrieve them.

STEP:
1 Open settings app
2 enable accounts
3 add evernote account
4 open reminders

Note the No notes page is displayed.

# Notes

- Something changed between images 157 and 158 [1] that caused this bug to surface. If I downgrade to image 157, notes are still retrieved as expected. Using Reminders 0.5.203 in both cases.
- The AppArmor denials pointed out in the comments are a red herring: when the app is functioning normally, we get these denials too
- Some debugging with mardy and this log [2] indicate that it's due to the authentication params not being returned in the onAuthenticated handler
- This needs to be fixed in Online Accounts, but as a workaround, we can get the required params from the app just before calling the authenticate() function in reminders.qml

[1] http://people.canonical.com/~lzemczak/landing-team/158.commitlog
[2] http://pastebin.ubuntu.com/7968853/

Related branches

Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

Seeing this denial.
[Thu Jul 31 21:04:20 2014] type=1400 audit(1406837062.464:97): apparmor="DENIED" operation="chmod" profile="com.ubuntu.reminders_reminders_0.5.203" name="/run/user/32011/" pid=12954 comm="reminders" requested_mask="w" denied_mask="w" fsuid=32011 ouid=32011

tags: added: lt-blocker lt-category-visible lt-prio-high
David Planella (dpm)
Changed in reminders-app:
status: New → Triaged
importance: Undecided → Critical
Revision history for this message
Alberto Mardegan (mardy) wrote :

Is this related to Online Accounts? Can you please check if you get an access token, and, if not, post the output of dbus-monitor and syslog (after setting the logging level to 2 in /etc/signond.conf)?

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

The apparmor denial is legitimate-- apps are not allowed to have write access to the /run/user/32011/ directory. Reminders (or some underlying library) is improperly trying to do a chmod on /run/user/32011/. I'm not sure why, it should be created 0700.

Revision history for this message
David Planella (dpm) wrote :

It seems to be a change that happened between the latest promoted image (157), where Reminder still works, and the latest proposed image (169 or even earlier), where it doesn't.

Jamie, do you have any tips on debugging what library that write request might be coming from?

Revision history for this message
David Planella (dpm) wrote :

Even after adding the write permission for test purposes (see below), while the AppArmor denial goes away, the app still won't load any notes.

$ sudo vi /var/lib/apparmor/profiles/click_com.ubuntu.reminders_reminders_0.5.203
$ tail -f /var/lib/apparmor/profiles/click_com.ubuntu.reminders_reminders_0.5.203
      # LP: #1260044
      deny /usr/lib/@{multiarch}/oxide-qt/locales/ w,

    }

  # No read paths specified

  # No write paths specified
  /run/user/32011/ w,
}
$ sudo apparmor_parser -r /var/lib/apparmor/profiles/click_com.ubuntu.reminders_reminders_0.5.203

Revision history for this message
David Planella (dpm) wrote :

- We're not getting a token from Online Accounts
- The application log [1] seems to indicate that the AppArmor denial seems to be related to signond -> "Failed to connect to socket /run/user/32011/signond/socket: No such file or directory")

[1] http://pastebin.ubuntu.com/7963184/

David Planella (dpm)
description: updated
Revision history for this message
Alberto Mardegan (mardy) wrote :

When we cannot connect to the signond socket, we use the D-Bus session. That critical message is wrong, in the latest version (not yet landed) it has been downgraded to a simple debug message.

David Planella (dpm)
description: updated
description: updated
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

Fix committed into lp:reminders-app at revision 211, scheduled for release in reminders-app, milestone 0.4

Changed in reminders-app:
status: Triaged → Fix Committed
David Planella (dpm)
Changed in reminders-app:
status: Fix Committed → Fix Released
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

FYI, I filed a new bug for the apparmor denial: bug #1351041

Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (5.1 KiB)

This bug was fixed in the package reminders-app - 0.4+15.04.20141118-0ubuntu1

---------------
reminders-app (0.4+15.04.20141118-0ubuntu1) vivid; urgency=low

  [ Akiva Avraham ]
  * Updated library imports to qtquick-2.3.
  * Upgraded ubuntu.components to 1.1 Upgraded ubuntu.components.xxxx to
    1.0.

  [ Paolo Rotolo ]
  * Added Tags icon.

  [ Alan Pope ]
  * Fix incorrect email address.

  [ Michael Zanetti ]
  * Add count properties to models.
  * add API to save notebooks.
  * correctly update the model when a notebook changes.
  * Some performance optimizations when having many, big notes
  * improve startup sequence
  * suppress some warnings. (LP: #1316800)
  * Add support for tags. (LP: #1379747)
  * Fixed bug 1378014 - Notes should be sorted by modification date by
    default Fixed bug 1380255 - Add day of the month to the notes
    preview. (LP: #1378014, #1380255)
  * Fixed change accounts page. (LP: #1379751)
  * Split the actual evernote code out from the QML plugin into a
    libqtevernote. (LP: #1385994)
  * Improve reminders delegate. (LP: #1318054)
  * fix pulldown positioning. (LP: #1368548)
  * split loading of notes into chunks
  * Don't reload the note's content when reloading a note for its
    resources. (LP: #1380987)
  * improve sorting by falling back on title if the first sortrole is
    equal. (LP: #1380987)
  * add gif support
  * Improve handling of connection errors.
  * include tags when cloning a note.
  * add scrollbars to listviews.
  * Don't always build translations
  * implement full note caching
  * improve note edit view. (LP: #1391410, #1316947)
  * small fixes in the new edit mode
  * Created URI handlers to load the app from the scope. (LP: #1391689,
    #1385995)

  [ Víctor R. Ruiz ]
  * "Search notes" unstranslated. Update to the code and template. (LP:
    #1356944)
  * Another batch of untranslated strings. Updated template. (LP:
    #1356944)

  [ Riccardo Padovani ]
  * Fixed #1340644 - Notes go in default notebook when edited. (LP:
    #1340644)
  * Upgraded the WebView to Oxide. (LP: #1317580)
  * Fixed #1340686 - Selecting a notebook then add a note doesn't
    default the note to the notebook. (LP: #1340686)
  * Added a couple of new i18n.tr, updated the translations template.
  * Updated the camera confirm page to have buttons again Added a check
    to be sure to delete previous temp picture. (LP: #1342249)
  * Update string for no account advice, removed reference to add
    reminder button. (LP: #1340635)
  * Fixed pull down to refresh behavior.
  * Implemented fronted to change notebooks name. (LP: #1340640)
  * Implemented bottomEdge.
  * Added a bouncing ProgressBar. (LP: #1372420)
  * Fix #1379886 - First time reminders tab is shown, the list is offset
    from the top. (LP: #1379886)
  * Fixed #1382730 - Unable to create new notes. (LP: #1382730)
  * Enabled automatic orientation. (LP: #1385429)
  * Fixed bug 1378014 - Notes should be sorted by modification date by
    default Fixed bug 1380255 - Add day of the month to the notes
    preview. (LP: #1378014, #1380255)
  * Fixed change accounts page. (LP: #1379751)
  * Created URI handlers to load the app from the scope. (LP: #...

Read more...

Changed in reminders-app (Ubuntu):
status: New → 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.