In the upcoming 0.3.2 CHANGELOG, I read "Remove the proxy settings handling since we do not use it anymore.", so this bug will disappear (with the feature it affects) in 0.3.2, correct?
No, this bug is about the proxy that tor uses, not the proxy application that Vidalia used to run (which was the affected by the 0.3.2 change).
arma helped me clarifying the CHANGELOG. Here's the new changelog entry for the one you're talking about:
o Remove the ability to launch an http proxy (e.g. Polipo), since Tor
Browser Bundle does not use it anymore. In the future, perhaps
a plugin will allow users to launch external applications like
this. Fix bug 5386.
I have simple fix for that - we will have to add another setting value to AbstractTorSettings
(SETTING_PROXY_x).
Something like SETTING_PROXY_ENABLED that will remember if ui.chkUseProxy is checked.
Other SETTING_PROXY_x settings will be always saved/loaded.
Based on SETTING_PROXY_ENABLED we will "Apply the current network configuration settings to Tor." in NetworkSettings::apply, and enable/disable QGroupBox grpProxySettings.
That would be my first fix for vidalia so i'm not sure what the workflow is;)
Anyway i'm assigning bug to me.
Trac: Username: karulis Cc: N/Atokarulis@wp.pl Owner: chiiph to karulis Status: new to assigned
Even though it looks good, except for minor indentation issues, the alpha patch shouldn't set ProxyEnabled in vidalia.conf. The policy for settings handling in 0.3.2 and on is to have all tor settings live in the torrc (see setVolatileValue(), Torrc class, and related), which is problematic in a case like this one, because if you set the proxy you enable the use of it. One idea was to have a certain kind of torrc comment format, like for example, if a line looks like this:
#* HTTPProxy 127.0.0.1:1234
(note the #* comment), then that's an old value that should be set if the user enables that particular option again.