From: Gabriele B. <g.b...@co...> - 2004-02-02 12:47:42
|
Il ven, 2004-01-30 alle 17:59, Gilles Detillieux ha scritto: > It certainly seems like a bug to me, and your patch seems correct as far > as I can tell. However, I don't see the need for supporting a dash or > "noproxy". With your patch, if you set http_proxy to an empty string > within the URL block, it should do the right thing. What was missing > before was the code you added to delete the old "proxy" value and set > the pointer to null when proxyURL's length is 0. I instinctively thought like you did, Gilles. I set it to empty but it was not working. I checked the code and it seems that, if an attribute is empty in the configuration file, it is not overridden. I will inspect the code however. > Later, this should be able to be overridden by the http_proxy > environement variable, but it currently cannot. > or any reason for the expectation that it should be able to be overridden, > so why even mention this at all? Can anyone explain this one? I agree ... I think we should get rid of this. What about you guys? Otherwise I can think of other environmental variables to be used, such as LOCALE for instance. But, I agree with Gilles, either we give this chance in a very general way (for all its possible uses) or we don't give it at all. Ciao, -Gabriele |