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

Support ill formatted UNC paths containing backslashes #41

Open
GoogleCodeExporter opened this issue May 1, 2015 · 2 comments
Open

Comments

@GoogleCodeExporter
Copy link

We've seen some machines with PAC configuration where the url to the PAC file 
is of the form:

file://\\servername\path\to\file.pac

These can't be read because the backslashes are probably not supported by 
windows. 

Since it's difficult to have control over the machines where the software is 
installed, any change of being resilient to this configuration error by 
replacing all backslashes with forward slashes in the PAC url?

Original issue reported on code.google.com by [email protected] on 26 Apr 2013 at 2:58

@GoogleCodeExporter
Copy link
Author

[deleted comment]

@GoogleCodeExporter
Copy link
Author

Based on the 20121203 source code, I would fix this in 
UrlPacScriptSource.java:79

file = new File(new URL(scriptUrl).toURI());

would become:

file = new File(new URL(scriptUrl.replaceAll("\\\\", "/")).toURI());

Original comment by [email protected] on 26 Apr 2013 at 3:04

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

1 participant