<div dir="ltr">Hi,<br><div><br></div><div>Various URL scheme code paths contain branches for local or non-local file URLs:</div><div><br></div><div>See URLJarFile::isFileURL for an example where the host is compared as follows:</div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><font face="monospace">String host = url.getHost();<br></font><font face="monospace">if (host == null || host.isEmpty() || host.equals("~") ||<br></font><font face="monospace">    host.equalsIgnoreCase("localhost"))</font><div><font face="monospace">    return true; </font></div></blockquote><div><br></div><div>Equivalent checks are found in unix and windows Handler::openConnection.</div><div><br></div><div>In JarFileFactory::urlFor however, the logic is slightly different in that the comparison with ~ (tilde) is missing:</div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><font face="monospace">String host = url.getHost();</font></div><div><font face="monospace">if (host != null && !host.isEmpty() &&</font></div><div><font face="monospace">        !host.equalsIgnoreCase("localhost")) {</font></div><div><br></div></blockquote><div>I have tried to find an RFC or any other mention of "~" in host names, but found nothing. Can anyone shine light on the meaning and interpretation of this character as a host name?</div><div><br></div><div>In any case, I think if we could add the tilde logic to JarFileFactory::urlFor, then all these four checks would be equivalent and a prime candidate for a consolidating cleanup PR, perhaps also expanding a bit on the documentation for the consolidated method(s).</div><div><br></div><div>My take is that the missing test was a simple oversight, that adding it should not be harmful and that this would enable a nice consolidation of this particular logic. .</div><div><br></div><div>Thoughts?</div><div><br></div><div>Eirik.</div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div></div>