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 333667 - [PATCH] MusicBrainz metadata lookup does not function with a proxy server
[PATCH] MusicBrainz metadata lookup does not function with a proxy server
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: general
git master
Other All
: Normal minor
: 2.x
Assigned To: Banshee Maintainers
Banshee Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-03-06 21:31 UTC by Mikayla Hutchinson
Modified: 2006-03-15 18:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
A simple patch to add proxy support (1.06 KB, patch)
2006-03-06 21:34 UTC, Mikayla Hutchinson
committed Details | Review

Description Mikayla Hutchinson 2006-03-06 21:31:34 UTC
Please describe the problem:
Banshee is unable to perform metadata lookups when internet access is only
available via an http proxy server.

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Mikayla Hutchinson 2006-03-06 21:34:21 UTC
Created attachment 60799 [details] [review]
A simple patch to add proxy support

Here is a small patch that automatically sets the proxy server for libmusicbrainz using Mono's proxy settings. These can be set in .config files, and since October are by default loaded from the conventional http_proxy environment variable.

I wan't sure exactly where to add in this support, but the easiest way of making sure it gets everywhere was to put it into the MusicBrainz.Client constructor. If it were to happen at any other time, for example after setting the server, there might be some nasty surprises for people who want to set the proxy manually. For this scenario I added a public method to make it easy to re-lookup the proxy based on server address.

Note that GlobalProxySelection has been deprecated in .NET 2, but replacements doesn't exist in .NET 1.1. A proxy server address can be found in gconf, but I didn't think a gconf dependency would be good for musicbrainz-sharp.

I guess this should really go into the musicbrainz-sharp repository, so that other projects can get it when they sync.

Could someone test this patch on a machine that doesn't need a proxy, to make sure it doesn't break anything? It works fine with the proxy on mine :-)
Comment 2 Aaron Bockover 2006-03-15 18:03:37 UTC
Committed. Thanks!