Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IE search strategy no longer returned when OS_DEFAULT strategy used on Windows #30

Open
jpasski opened this issue Oct 4, 2017 · 3 comments

Comments

@jpasski
Copy link
Contributor

jpasski commented Oct 4, 2017

In proxy-vole 1.0.3, the IE search strategy is added when the OS_DEFAULT search strategy is used on Windows. In proxy-vole 1.0.4, the strategy is not added when OS_DEFAULT is used. Is this expected behavior?

@nerdydrew
Copy link
Contributor

That's correct. In 1.0.3 and earlier, the WinProxySearchStrategy just replicated the IEProxySearchStrategy. The original author left a TODO back in 2009 to implement it with Windows API calls. I did so in 62b129a, which was included in 1.0.4. Now, you can use the Windows strategy to find proxy settings in the registry and the IE strategy for proxy settings at the user level like before.

@MarkusBernhardt
Copy link
Owner

MarkusBernhardt commented Oct 5, 2017 via email

@jpasski
Copy link
Contributor Author

jpasski commented Oct 5, 2017

Thanks for the background @nerdydrew. As a recent consumer of the library, I was surprised by the change. I could see others who have been using this, or the previously maintained, version of the library surprised also. This is especially true if it has been the default for the last 7-8 years. Since the above change will break those existing consumers who do not explicitly add the IE strategy and who upgrade to 1.0.4, I recommend documenting the change more prominently and perhaps bumping the version number to 1.1.

@MarkusBernhardt I'm unfamiliar with Windows, however, I could see the usefulness of allowing users to choose exclusively between the registry settings or the IE settings. I'm using the library with an intercepting proxy tool called Burp that's used in web application security assessments. I could see it being useful to reconfigure IE to go through Burp only leaving the desktop settings in place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants