[mythtv-commits] Ticket #9525: [PATCH] Mythbrowser does not use proxy set in http_proxy environment variable

[MythTV]

#9525: [PATCH] Mythbrowser does not use proxy set in http_proxy environment
variable
---------------------------------------+----------------------------
 Reporter:  jan.ceuleers@…             |          Owner:  paulh
     Type:  Patch - Bug Fix            |         Status:  new
 Priority:  minor                      |      Milestone:  unknown
Component:  Plugin - MythBrowser       |        Version:  0.23-fixes
 Severity:  low                        |     Resolution:
 Keywords:                             |  Ticket locked:  0
---------------------------------------+----------------------------

Comment (by nigel@…):

 1. I wanted to add proxy support for CDDB lookups many years ago, but that
 required a different way to do the lookups. When (if?) !HttpComms is used
 for that, '''''it''''' should be the location for doing proxy-compatible
 stuff?
 2. My gut feeling is that using an environment variable is a little clumsy
 - I would have preferred a setting from the database, but then that is
 just another thing that needs a settings UI.
 3. Mac OS X and Windows supposedly have hooks to look up the system proxy
 settings via QNetworkProxyFactory::systemProxyForQuery(). I find it hard
 to believe that the Linux environments don't have a standard proxy setting
 location these days?
 4. Now that the backend also does HTTP traffic, it also needs to set proxy
 "environment"s?
 Paul, my work has a few corporate proxies - I'm happy to take this is you
 don't have any burning desire.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/9525#comment:5>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center