RFR: 8277313: Validate header failed for test/jdk/java/net/httpclient/HeadTest.java
With the integration of JDK-8276559, validate header failed as there is one missing comma after 2021 in `test/jdk/java/net/httpclient/HeadTest.java` - Commit messages: - 8277313: Validate header failed for test/jdk/java/net/httpclient/HeadTest.java Changes: https://git.openjdk.java.net/jdk/pull/6424/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6424&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8277313 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/6424.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6424/head:pull/6424 PR: https://git.openjdk.java.net/jdk/pull/6424
Re: RFR: 8277313: Validate header failed for test/jdk/java/net/httpclient/HeadTest.java
On Wed, 17 Nov 2021 04:01:48 GMT, Jaikiran Pai wrote: > Thank you for fixing this @fmatte1. I'm not a Reviewer, but this looks good > to me. > > Could you tell me which test identified this issue, so that I can in future > run them locally if feasible? Thanks Jaikiran for the review, it is test/jdk/java/net/httpclient/HeadTest.java you have missed one comma(,) after 2021 in the header field. We internally validate headers that script has caught this. - PR: https://git.openjdk.java.net/jdk/pull/6424
Integrated: 8277313: Validate header failed for test/jdk/java/net/httpclient/HeadTest.java
On Wed, 17 Nov 2021 03:53:50 GMT, Fairoz Matte wrote: > With the integration of JDK-8276559, validate header failed as there is one > missing comma after 2021 in `test/jdk/java/net/httpclient/HeadTest.java` This pull request has now been integrated. Changeset: 08f65a59 Author:Fairoz Matte Committer: Jayathirth D V URL: https://git.openjdk.java.net/jdk/commit/08f65a59a7bd387974d94253ec7093524a3e92f1 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod 8277313: Validate header failed for test/jdk/java/net/httpclient/HeadTest.java Reviewed-by: jdv - PR: https://git.openjdk.java.net/jdk/pull/6424
