Integrated: 8365811: test/jdk/java/net/CookieHandler/B6644726.java failure - "Should have 5 cookies. Got only 4, expires probably didn't parse correctly"
Jaikiran Pai
jpai at openjdk.org
Wed Aug 20 06:10:49 UTC 2025
On Tue, 19 Aug 2025 16:53:45 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
> Can I please get a review of this trivial test-only fix which addresses the failure in `test/jdk/java/net/CookieHandler/B6644726.java`? This addresses https://bugs.openjdk.org/browse/JDK-8365811.
>
> The test verifies the behaviour of cookie parsing through the `java.net.CookieStore` API. For that it uses a couple of cookies which have an expiry date set to August 19th 2025 (which is today). The assertions in this test start failing once those 2 cookies expire (which is today).
>
> The fix in this PR updates the cookie expiry year to a date very far in future to prevent such failures.
This pull request has now been integrated.
Changeset: b453eb63
Author: Jaikiran Pai <jpai at openjdk.org>
Committer: Alan Bateman <alanb at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/b453eb63c641e1e69b4aef57a220ebe45b9d1693
Stats: 5 lines in 1 file changed: 1 ins; 0 del; 4 mod
8365811: test/jdk/java/net/CookieHandler/B6644726.java failure - "Should have 5 cookies. Got only 4, expires probably didn't parse correctly"
Reviewed-by: syan, alanb
-------------
PR: https://git.openjdk.org/jdk/pull/26846
More information about the net-dev
mailing list