GNOME Bugzilla – Bug 395827
Try to reduce remote accesses
Last modified: 2019-03-20 10:38:19 UTC
$ ./test-parser http://localhost:12345/playlist.asx ###################### parsing ################ added URI 'http://localhost:12345/leopard.mov' with title 'empty' genre 'empty' added URI 'http://localhost:12345/frontrow-music.mov' with title 'empty' genre 'empty' Shows in the logs: 127.0.0.1 - - [12/Jan/2007:16:19:26 +0000] "GET /playlist.asx HTTP/1.1" 200 319 "-" "gnome-vfs/2.14.2 neon/0.25.4" 127.0.0.1 - - [12/Jan/2007:16:19:26 +0000] "GET /playlist.asx HTTP/1.1" 200 319 "-" "gnome-vfs/2.14.2 neon/0.25.4" 127.0.0.1 - - [12/Jan/2007:16:19:26 +0000] "GET /leopard.mov HTTP/1.1" 200 348892 "-" "gnome-vfs/2.14.2 neon/0.25.4" 127.0.0.1 - - [12/Jan/2007:16:19:26 +0000] "GET /frontrow-music.mov HTTP/1.1" 200 473716 "-" "gnome-vfs/2.14.2 neon/0.25.4" It should only access playlist.asx once if possible.
Still the same problem: $ cat root/access_log 127.0.0.1 - - [24/May/2008:13:15:10 +0100] "GET /playlist.asx HTTP/1.1" 200 319 "-" "gvfs/0.2.3" 127.0.0.1 - - [24/May/2008:13:15:10 +0100] "GET /playlist.asx HTTP/1.1" 200 319 "-" "gvfs/0.2.3" This is caused by _get_mime_type_with_data, which gets us the first few bytes of data to analyse, but then close the file, prompting us to reopen it later. We should really be passing a triplet of: - data read thus far (8k) - len of data read thus far - input stream, so we can read further, which would be empty if we finished reading the file
Created attachment 117049 [details] [review] First pass Still need to port all the parsers to use the existing data, probably just needs a helper function that'll either download everything from scratch, or continue an existing downlad.
This should be fixed. 1. Load http://www.yle.fi/elavaarkisto/?s=s&g=1&ag=4&t=410&a=6126 2. Click on the play button 3. Select "Jatka mahdollisista virheistä huolimatta"
(In reply to comment #3) > This should be fixed. > > 1. Load http://www.yle.fi/elavaarkisto/?s=s&g=1&ag=4&t=410&a=6126 > 2. Click on the play button > 3. Select "Jatka mahdollisista virheistä huolimatta" Should the bug be closed then?
No, I meant that we should fix the bug to make the website above work.
-- GitLab Migration Automatic Message -- This bug has been migrated to GNOME's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/totem-pl-parser/issues/7.