After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 701957 - Ignore Google documents without a thumbnail URI
Ignore Google documents without a thumbnail URI
Status: RESOLVED FIXED
Product: gnome-documents
Classification: Core
Component: general
3.8.x
Other Linux
: High normal
: ---
Assigned To: GNOME documents maintainer(s)
GNOME documents maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-06-10 19:18 UTC by Baptiste Mille-Mathias
Modified: 2013-06-14 15:36 UTC
See Also:
GNOME target: ---
GNOME version: 3.7/3.8


Attachments
documents: Ignore Google documents without a thumbnail URI (1006 bytes, patch)
2013-06-14 14:32 UTC, Debarshi Ray
committed Details | Review

Description Baptiste Mille-Mathias 2013-06-10 19:18:21 UTC
I've just upgraded to 3.8.3, and every document I try to open makes document crash.
I've tried just before with 3.8.2, and it was working fine.

LANG=C gnome-documents 
error: The metadata does not have a thumbnail property

(gnome-documents:3147): GnomeDesktop-WARNING **: Unable to create loader for mime type application/msword: Unrecognized image file format

(gnome-documents:3147): GnomeDesktop-WARNING **: Error creating thumbnail for file:///home/baptiste/Documents/CV%20&%20LM/bmillemathias_resume.doc: Unrecognized image file format

(gnome-documents:3147): libgdata-WARNING **: Adding a kind category term, 'http://schemas.google.com/docs/2007#pdf', to an entry of kind 'http://schemas.google.com/docs/2007#document'.

(gnome-documents:3147): libgdata-WARNING **: Adding a kind category term, 'http://schemas.google.com/docs/2007#spreadsheet', to an entry of kind 'http://schemas.google.com/docs/2007#document'.

(gnome-documents:3147): libgdata-WARNING **: Adding a kind category term, 'http://schemas.google.com/docs/2007#pdf', to an entry of kind 'http://schemas.google.com/docs/2007#document'.

(gnome-documents:3147): libgdata-WARNING **: Adding a kind category term, 'http://schemas.google.com/docs/2007#pdf', to an entry of kind 'http://schemas.google.com/docs/2007#document'.

(gnome-documents:3147): libgdata-WARNING **: Adding a kind category term, 'http://schemas.google.com/docs/2007#pdf', to an entry of kind 'http://schemas.google.com/docs/2007#document'.

(gnome-documents:3147): libsoup-CRITICAL **: soup_message_new: assertion `uri_string != NULL' failed
**
libgdata:ERROR:gdata/services/documents/gdata-documents-service.c:307:append_query_headers: assertion failed: (message != NULL)
Abandon
Comment 1 Cosimo Cecchi 2013-06-10 20:50:41 UTC
Might be related to [1] or [2]. Does it only happen with Google documents? Debarshi, can you look into this?

[1] https://git.gnome.org/browse/gnome-documents/commit/?h=gnome-3-8&id=0a03910a9fb32e8c2999e27e0686557d8bd43d4e
[2] https://git.gnome.org/browse/gnome-documents/commit/?h=gnome-3-8&id=128e8ba90167bcad3785bca78be0adeab132fea0
Comment 2 Debarshi Ray 2013-06-11 09:47:47 UTC
Both local and Google documents work fine for me.

Would it be possible to have a backtrace? You can do so by attaching GDB to a running gnome-documents process and then triggering the crash.
Comment 3 Debarshi Ray 2013-06-14 12:42:13 UTC
Looking at the backtrace on this downstream bug https://bugzilla.redhat.com/show_bug.cgi?id=974063 I will lay the blame on the thumbnail downloading code. Investigating.
Comment 4 Debarshi Ray 2013-06-14 13:18:31 UTC
It will be helpful if someone can narrow it down to the particular file that is causing this. You can try putting something like the following in src/documents.js just before the GData.DownloadStream is created in the createThumbnail method of GoogleDocument:
  log(this.uri);
  log(this.name);

Then delete everything inside ~/.cache/thumbnails/ and start gnome-documents from a terminal.
Comment 5 Debarshi Ray 2013-06-14 14:32:59 UTC
Created attachment 246815 [details] [review]
documents: Ignore Google documents without a thumbnail URI

Turns out that some Google documents might not have a server provided thumbnail URI. This is reflected in the documentation for gdata_documents_document_get_thumbnail_uri , which I should have read more closely.

Many thanks to Jiri Eischmann (who filed the downstream report against Fedora) for letting me have the file that was causing this crash.
Comment 6 Debarshi Ray 2013-06-14 14:33:57 UTC
It might be nice to have a 3.8.3.1 with this fix.
Comment 7 Cosimo Cecchi 2013-06-14 15:24:47 UTC
Review of attachment 246815 [details] [review]:

Looks good to me.
Comment 8 Cosimo Cecchi 2013-06-14 15:25:32 UTC
(In reply to comment #6)
> It might be nice to have a 3.8.3.1 with this fix.

Feel free to release; otherwise I'll probably get to it before Monday together with 3.9.3.
Comment 9 Debarshi Ray 2013-06-14 15:34:23 UTC
Comment on attachment 246815 [details] [review]
documents: Ignore Google documents without a thumbnail URI

Thanks for the review. I will release a 3.8.3.1 shortly.