Last.fm Covers: improve proxy detection/port to Gio

Bug #822921 reported by Scott Garman
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Exaile
Fix Released
Low
Mathias Brodala

Bug Description

When using exaile, I've noticed when adding tracks to my playlist, the application will freeze up for 10-20 seconds, and the continue normally. I determined that this was only happening for albums where I did not have local cover art. which led me to suspect that the cover art fetching was having problems. I then realized these freeze-ups only happened when I was at work behind a proxy server. I have the proxy server set up correctly in my GNOME settings, and I suspect that the plugin does not honor these settings. Finally, I have confirmed that disabling the Last.fm cover art plugin fixes the symptom.

I realize this isn't incontrovertible proof of the problem, but hopefully it describes it enough that someone can reproduce it further.

I'm using Exaile 3.2.0 on Ubutnu 10.10 (Maverick), x86_64

Related branches

Revision history for this message
reacocard (reacocard) wrote :

Seems like a reasonable supposition. Anyone know how GNOME exposes its proxy settings, and/or how we can most-trivially utilize them? There are probably other areas of Exaile that also fail to respect proxies (amazon covers? lastfm dynamic? audioscrobbling?), so we should look into that too.

Changed in exaile:
importance: Undecided → Medium
milestone: none → 0.3.3.0
Revision history for this message
Joshua Lock (incandescant) wrote :

I believe the accepted way is to read the proxy settings from gconf/dconf

Revision history for this message
Johannes Sasongko (sjohannes) wrote :

I remember similar discussion in the past, and I believe one of the proposed solutions was to use the http_proxy env var (IIRC urllib2 uses this automatically), though I don't know whether GNOME sets this.

Changed in exaile:
milestone: 0.3.3.0 → 0.3.3.1
Revision history for this message
Mathias Brodala (mathbr) wrote :

The standard way to define a proxy server is via the http_proxy environment variable, as mentioned. Since the Last.fm Covers plugin uses urllib, that environment variable is honored.

Thus if the GNOME control center does not keep that variable updated, it’s a bug in the control center.

Changed in exaile:
importance: Medium → Low
milestone: 0.3.3.1 → none
status: New → Invalid
Revision history for this message
Ross Burton (ross) wrote :

Environment variables are rather tricky to set in an application that's already running. They are a last resort in the modern world.

If you use GIO for networking you'll get automatic proxy support (including execution of PAC files, required in some networks). Otherwise there are gsetting keys that GNOME sets for you.

Revision history for this message
Mathias Brodala (mathbr) wrote :

Porting from urllib to Gio is certainly a valid point to check.

Changed in exaile:
milestone: none → 0.3.3.0
status: Invalid → Confirmed
Revision history for this message
Mathias Brodala (mathbr) wrote :

The plugin has been ported to Gio with r4216.

summary: - Last.fm coverart plugin does not use GNOME proxy settings
+ Last.fm Covers: improve proxy detection/port to Gio
Changed in exaile:
assignee: nobody → Mathias Brodala (mathbr)
status: Confirmed → Fix Committed
Changed in exaile:
status: Fix Committed → 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.