给 JDK 官方提了一个 Bug，结果...

你的名字 2023-10-05 10:33 7阅读 0赞

![format_png][]

图 by：石头@北京-望京

> 关于作者：程序猿石头(ID: tangleithu)，现任阿里巴巴技术专家，清华学渣，前大疆后端 Leader。

### 背景 ###

分享一下之前踩的一个坑，背景是这样的：

我们的项目依赖于一个外部服务，该外部服务提供 REST 接口供我方调用，这是很常见的一个场景。本地和测试环境测试都没有问题，一切就绪上了生产后，程序调用接口就总是网络不通。

需要说明的是本地、测试环境、生产环境通过不同的域名访问该外部服务。生产程序调用不通，神奇的是在生产环境通过 `curl` 等命令却能够正常调用对方接口。

![format_png 1][] What??

这 TM 就神奇了，唯一不同的就是发起 HTTP 请求的客户端了，估计就是 http客户端有问题了？通过最后排查发现，居然发现了一枚 “JDK 的 bug”，然后石头就提交到了 JDK 的官网……

![format_png 2][]

下面我们就来重现一下这个问题。

### server 端准备 ###

这里用 Nginx 模拟了一下 上文提到的 REST 服务，假设调用正常返回 `"Hello, World\n"`，Nginx 配置如下：

server {
        listen    80;
        server_name test_1.tanglei.name;
        location /testurl {
            add_header Content-Type 'text/plain; charset=utf-8';
            return 200 "Hello, World\n";
        }
    }

### 不同的 client 请求 ###

下面用不同的 Http client （分别用命令行`curl`，python的 `requests`包，和 Java 的 URL 等尝试）去请求。

*  `curl` 请求，正常。

[root@VM_77_245_centos vhost]# curl -i "http://test_1.tanglei.name/testurl"
    HTTP/1.1 200 OK
    Server: nginx
    Content-Length: 13
    Connection: keep-alive
    Content-Type: text/plain; charset=utf-8
    
    Hello, World
    [root@VM_77_245_centos vhost]#

*  `python requests` 正常。

>>> import requests
    >>> r = requests.get("http://test_1.tanglei.name/testurl")
    >>> r.text
    u'Hello, World\n'

*  Java 的 `java.net.URLConnection` 同样正常。

static String getContent(java.net.URL url) throws Exception {
        java.net.URLConnection conn = url.openConnection();
        java.io.InputStreamReader in = new java.io.InputStreamReader(conn.getInputStream(), "utf-8");
        java.io.BufferedReader reader = new java.io.BufferedReader(in);    
        StringBuilder sb = new StringBuilder();
        int c = -1;
        while ((c = reader.read()) != -1) {
            sb.append((char)c);
        }
        reader.close();
        in.close();
        String response = sb.toString();
        return response;
    }

上面的这个方法 `String getContent(java.net.URL url)` 传入一个构造好的 `java.net.URL` 然后 get 请求，并以 `String` 方式返回 response。

String srcUrl = "http://test_1.tanglei.name/testurl";
    java.net.URL url = new java.net.URL(srcUrl);
    System.out.println("\nurl result:\n" + getContent(url)); // OK

上面的语句输出正常，结果如下：

url result:
    Hello, World

这就尼玛神奇了吧。看看我们程序中用的 httpclient 的实现，结果发现是有用 `java.net.URI`，心想，这不至于吧，用 URI 就不行了么。

![format_png 3][]

换 `java.net.URI` 试试? （这里不展开讲URL和URI的区别联系了，可以简单的认为URL是URI的一个子集，详细的可参考 URI、URL 和 URN\[1\], wiki URI\[2\]）

直接通过`java.net.URI`构造，再调用 `URI.toURL` 得到 `URL`，调用同样正常。

关键的来了，httpclient 源码中用的构造函数是另外一个：

URI(String scheme, String host, String path, String fragment)
    Constructs a hierarchical URI from the given components.

我用这个方法构造`URI`，会构造失败：

new java.net.URI(uri.getScheme(), uri.getHost(), uri.getPath(), null) error: protocol = http host = null
    new java.net.URI(url.getProtocol(), url.getHost(), url.getPath(), null) error: Illegal character in hostname at index 11: http://test_1.tanglei.name/testurl

所以问题发现了，我们的项目中依赖的第三方 httpclient包底层用到了 `java.net.URI`，恰好在 `java.net.URI` 中是不允许以下划线(`_`)作为 `hostname` 字段的。

即 `uri.getHost()` 和 `uri.toURL().getHost()` 居然能不相等。

### 这是 JDK 的 Bug 吧？ ###

有理由怀疑，这是 JDK 的 Bug 吧？

![format_png 4][]

从官网上还真找到了关于包含下划线作为hostname的bug提交issue，戳这里 JDK-8132508 : Bug JDK-8029354 reproduces with underscore in hostname\[3\]，然后发现该 "bug" reporter 的情况貌似跟我的差不多，只不过引爆bug的点不一样。

该 "bug" reviewer 最后以 "Not an Issue" 关闭，给出的理由是：

> RFC 952 disallows \_ underscores in hostnames. So, this is not a bug.

确实，rfc952\[4\] 明确明确说了域名只能由 字母 `(A-Z)`、 数字`(0-9)`、 减号 `(-)` 和 点 `(.)` 组成。

那 OK 吧，既然明确规定了 hostname 不能包含下划线，为啥 `java.net.URL` 确允许呢？

造成 `java.net.URI` 和 `java.net.URL` 在处理 hostname 时的标准不一致，且本身 `java.net.URI` 在构造的时候也带了 "有色"眼镜，同一个url字符串 通过静态方法 `java.net.URI.create(String)` 或者通过带1个参数的构造方法 `java.net.URI(String)` 都能成功构造出 URI 的实例，但通过带4个参数的构造方法就不能构造了。

要知道，在 coding 过程中，**尽早**反馈异常信息更有利于软件开发持续迭代的过程。我们在开发过程中也应该遵循这一点原则。

于是我就去 JDK 官网提交了一个 bug，大意是说 `java.net.URI` 和 `java.net.URL` 在处理hostname的时候标准不一致，容易使开发人员埋藏一些潜在的bug，同时也还把这个问题反馈到 stackoverflow\[5\] 了

> I am wondering, if hostname with underscore is not valid, why the result is differrent between java.net.URI and java.net.URL? Is it a bug or a feature? Here is the example.
> 
> java.net.URL url = new java.net.URL("http://test\_1.tanglei.name");
> 
> System.out.println(url.getHost()); //test\_1.tanglei.name
> 
> java.net.URI uri = new java.net.URI("http://test\_1.tanglei.name");
> 
> System.out.println(uri.getHost()); //null

这个 JDK bug issue 详细信息见 JDK-8170265 : underscore is allowed in java.net.URL while not in java.net.URI\[6\]，openjdk JDK-8170265\[7\]

![format_png 2][]

经过初步 Review，被认为是一个 P4 的 Bug，说的是 `java.net.URL` 遵循的是 `RFC 2396` 规范，确实不允许含有下划线的 hostname，`java.net.URI` 做到了， 而 `java.net.URL` 没有做到。

![format_png 5][]

重点来了，然后，却被另外一个 Reviewer 直接个毙了。给出的原因是 `java.net.URL` 构造方法中，API 文档中说了本来也不会做验证即 `No validation of the inputs is performed by this constructor.` 在线 api doc 戳这里\[8\] (可以点连接，进去搜索关键字 "No validation")

![format_png 6][]

![format_png 7][]

当初没有收到及时反馈，就没有来得及怼回去。

其实就算 "No validation of the inputs is performed by this constructor." 是合理的，里面也只有3个构造函数有这样的说明，按照这样的逻辑是不是说另外的构造函数有验证呢..... (示例中的默认的构造函数都没有说呀)

这里有java.net.URL 的源码\[9\]，看兴趣的同学可以看看。

恩，以上就是结论了。

不过，反正我自己感觉目前 Java API 关于这里的设计不太合理，欢迎大家讨论。

我在SO提问的这个回答\[10\]比较有意思，哈哈。

> The review is somewhat terse, but the reviewer's point is the URL constructor is behaving in accordance with its specification. Since the specification explicitly states that no validation is performed, this is not a bug in the code. This is indisputable.
> 
> What he didn't spell out is that fixing this inconsistency (by changing the URL class specification) would break lots of peoples' 20+ year old code Java code. That would be a really bad idea. It can't happen.
> 
> So ... this inconsistency is a "feature".

### 后记    ###

觉得本号分享的文章有价值，记得添加星标哦。周更很累，不要白 piao，需要来点正反馈，安排个 “一键三连”（点赞、在看、分享）如何？???? 这将是我持续输出优质文章的最强动力。

![format_png 8][]

附本文链接：

\[1\] URI、URL 和 URN: https://www.ibm.com/developerworks/cn/xml/x-urlni.html

\[2\]wiki URI: https://en.wikipedia.org/wiki/Uniform\_Resource\_Identifier

\[3\]JDK-8132508 : Bug JDK-8029354 reproduces with underscore in hostname: http://bugs.java.com/bugdatabase/view\_bug.do?bug\_id=8132508

\[4\]rfc952: https://tools.ietf.org/html/rfc952

\[5\]stackoverflow: https://stackoverflow.com/questions/44226003/conflicts-between-java-net-url-and-java-net-uri-when-dealing-with-hostname-conta

\[6\]JDK-8170265 : underscore is allowed in java.net.URL while not in java.net.URI: http://bugs.java.com/bugdatabase/view\_bug.do?bug\_id=8170265

\[7\]openjdk JDK-8170265: https://bugs.openjdk.java.net/browse/JDK-8170265

\[8\]在线 api doc 戳这里: https://docs.oracle.com/javase/8/docs/api/java/net/URL.html

\[9\]这里有java.net.URL 的源码: http://www.docjar.com/html/api/java/net/URL.java.html

\[10\]我在SO提问的这个回答: https://stackoverflow.com/questions/44226003/conflicts-between-java-net-url-and-java-net-uri-when-dealing-with-hostname-conta?answertab=active\#tab-to

![format_png 9][]

往期推荐

[![format_png 10][]][format_png 10 1]

[Java新特性：数据类型可以扔掉了？][format_png 10 1]

[![format_png 11][]][format_png 11 1]

[动图演示：手撸堆栈的两种实现方法！][format_png 11 1]

[![format_png 12][]][format_png 12 1]

[JDK15正式发布，新增功能预览！][format_png 12 1]

关注下方二维码，收获更多干货！

![format_png 13][]

[format_png]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9aTVhEaGhHbllpYnZUbHl4c3R3dXVyREo4cXBabWxKMWQzQnhPaWI4YlVsUkNxVHZKMFEyUVRiZWJ3UW5rYnJtbXVSQ0xUdndscFlkYWQ3bGdpYXRHN3kxUS82NDA?x-oss-process=image/format,png
[format_png 1]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2dpZi9aTVhEaGhHbllpYnZUbHl4c3R3dXVyREo4cXBabWxKMWRaN2libzhQWGtvNEN3OVoxc1NiaFNQeElHWnhWbm1xd0FoSXZRaWNhN0hSMUJkQUZ6ZzF3TjdyZy82NDA?x-oss-process=image/format,png
[format_png 2]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9aTVhEaGhHbllpYnZUbHl4c3R3dXVyREo4cXBabWxKMWRKQTh6dlRxYURCdGRRMWRiWktJd3VXSjVTWWliN3VDTm9pYkNtemlhZXlFRkE4Z0NaUHBUVWFhSWcvNjQw?x-oss-process=image/format,png
[format_png 3]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9aTVhEaGhHbllpYnZUbHl4c3R3dXVyREo4cXBabWxKMWR1Y0VRZjBCc1JKR0FPSkdxa3lQZ2lhNDdRY1F4eGtLZmxWZGhVYms3OG1NSEs4N2lic1VFUnBpYkEvNjQw?x-oss-process=image/format,png
[format_png 4]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9aTVhEaGhHbllpYnZUbHl4c3R3dXVyREo4cXBabWxKMWRyYVEzTTQ5cURpYXpWMzk0VzduVThudUZSWnppYlpZRThwUjRrQnc2Z2ljdXBONG9jbmlhV29xaGF3LzY0MA?x-oss-process=image/format,png
[format_png 5]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9aTVhEaGhHbllpYnZUbHl4c3R3dXVyREo4cXBabWxKMWRpYTd0eklUMFU2Vm8xd0tTT1EzMzFvY1ZGTENiWGlhdExKQlpTcHZaRUw3SVBlU01ka2JEbThZUS82NDA?x-oss-process=image/format,png
[format_png 6]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9aTVhEaGhHbllpYnZUbHl4c3R3dXVyREo4cXBabWxKMWRtMUdaRmJ6eGlhd0VnZ2RhWFdWdWxpY3NYOUFPNHd2WGd4Q3JlbGNpY0lhd3hEd2R1Slp5bDlaM1EvNjQw?x-oss-process=image/format,png
[format_png 7]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2pwZy9aTVhEaGhHbllpYnZUbHl4c3R3dXVyREo4cXBabWxKMWRwWGNyVlJrVGxHOXhBa0tONDFnZjN2Qm8zRENXQWFjUGROUEYzbDBaUlcwTWZqTzFVUjQ5dUEvNjQw?x-oss-process=image/format,png
[format_png 8]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2dpZi9aTVhEaGhHbllpYnVzME05Q2E5a3h5aWNVRzRRTUc1MXZTaDJMQ3RPcGljZkp3cnJpYnU1WmM4ZDlPS0ZJUDJFNVVoY0o5Y3RURWVHOVh2MEVsUkZKcXY2NUEvNjQw?x-oss-process=image/format,png
[format_png 9]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2dpZi9iOTZDaWJDdDcwaWFid2p5b2pMaEEwM1B0eFVua05QUkVudDJGNDh5d2ZYTHBEZERBamljT1RQSThROTR0VkxiSjU4dGJSczEyaWFYREtoVU9XOWdkNE5sRkEvNjQw?x-oss-process=image/format,png
[format_png 10]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2pwZy9Icld3Nlp1WENzZ0ZxeDU1VzBUQlpWcWFvSkJ3QXloRFdrWkhWeWlhaDN1a0RHaHhlcnJYVWliMlhsVnBSdEs3MHQ2WEx5TnlkbXBaTmlhSUNHWEhjWExyZy82NDA?x-oss-process=image/format,png
[format_png 10 1]: http://mp.weixin.qq.com/s?__biz=MzU1NTkwODE4Mw%3D%3D&chksm=fbcc7bfbccbbf2ed181d16d7e2489401dbb23302a7947bd2893b4976ba7064f0cc6ddf5b0d19&idx=1&mid=2247489219&scene=21&sn=9ebebc38b6f22fd71464587352e5dc56#wechat_redirect
[format_png 11]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2pwZy9Icld3Nlp1WENzaWFrUFVCNW9UeWx4eXFzOXJzdlZvYUtvd1VYaWFoU1NIYjJSUmhpYzlnOVBlbXppY3RIaWJRcWdvOUNYMGljckw4TEQzNXFUWnExSFkzbjViUS82NDA?x-oss-process=image/format,png
[format_png 11 1]: http://mp.weixin.qq.com/s?__biz=MzU1NTkwODE4Mw%3D%3D&chksm=fbcc7bd4ccbbf2c25efa3fcdc9ab4fe1b74279f59015cafac24661b3c71b3fdcc19187e8ef3e&idx=1&mid=2247489260&scene=21&sn=c5871b77d084d11319fffc7197be9669#wechat_redirect
[format_png 12]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2pwZy9Icld3Nlp1WENzaWFvYnRnVXdlTXZqRFQyU2licE5hQzE2c3k2RWMwSFRMZlBueFN0SG9yaWF2OTJyUjcwUURwUHNIeDlPMUJteFhjaWE4d3ZlcWMySWtSOEEvNjQw?x-oss-process=image/format,png
[format_png 12 1]: http://mp.weixin.qq.com/s?__biz=MzU1NTkwODE4Mw%3D%3D&chksm=fbcc7b62ccbbf2745841518231573af0d61f9ba9abda3cb522a13bc14c8915e51a3e7072a5d4&idx=1&mid=2247489114&scene=21&sn=1c8946bed1cadcfa49f1c489aee34ce6#wechat_redirect
[format_png 13]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2pwZy9zWnlBSnRpYWVzV2NqajRZNmIyUnNZampPNGNtc0E3OXZKY2RyY080RDZxb2t6V1l6NjQ4S2ZQVFNsSWhCeWVKbGZzQmtLaWFzWHR4a2JSTFRUSUxRcGlidy82NDA?x-oss-process=image/format,png