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

Hope for full support for Microsoft PAC Extension #26

Closed
manugarg opened this issue Sep 26, 2014 · 4 comments
Closed

Hope for full support for Microsoft PAC Extension #26

manugarg opened this issue Sep 26, 2014 · 4 comments

Comments

@manugarg
Copy link
Owner

From hhylx1...@gmail.com on April 23, 2013 08:14:47

Thanks for your work. I've used it to test my script for a long time. But recently, I switch to Chrome which support ipv6 extension for pac.
I see it support a function of enable_microsoft_extensions(). It surely support dnsResolveEx() function. But it lacks support for sortIpAddressList().

I'm using python_pacparser1.3.0 in ubuntu with python2.7.

Here is their definitions: http://msdn.microsoft.com/en-us/library/windows/desktop/gg308477%28v=vs.85%29.aspx Here is Chrome's support code: https://code.google.com/p/chromium/codesearch#chromium/src/net/proxy/proxy_resolver_v8.cc&q=sortIpA&sq=package:chromium&l=37 My test script and pac script are listed below:

Original issue: http://code.google.com/p/pacparser/issues/detail?id=26

@peterskurt
Copy link

I enabled enable_microsoft_extensions() but it seems as if the full set of Microsoft extensions is not supported. I get an error with use of isInNetEx().

@manugarg
Copy link
Owner Author

pacparser doesn't have full support for IPv6 extensions. I am working on adding support for isResolvableEx. Support for other extensions will follow.

@manugarg manugarg self-assigned this Feb 26, 2015
@manugarg
Copy link
Owner Author

Commit 5559ad9 adds support for isInNetEx. Support for isResolvableEx was already added.

@manugarg
Copy link
Owner Author

I have released the new version which has support for additional *Ex routines.

Also, enable_microsoft_extensions is no longer required.

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

No branches or pull requests

2 participants