> Hi,
>
> I need a code review for moving the contents of the jdk.crypto.ec module into
> java.base. This moves the SunEC JCE Provider (Elliptic Curve) into
> java.base. EC has always been separate from the base module/pkg because of
> its dependence on a native library. That library was rem
I understood what you said.
And thank you for forwarding this mail to security-dev.
If it is decided to fix this issue and this issue is not as much work as I
thought.
If you give me the opportunity to change the code via PR, it will be a
great honor for a lifetime as a Java developer.
Thanks :)
In this PR, I updated the test to use read time-outs. The test is restarted if
the read operations time-out within (default) 30 seconds. The test makes 5
attempts before giving up.
-
Commit messages:
- 8310070: Test: javax/net/ssl/DTLS/DTLSWontNegotiateV10.java timed out
Changes:
> Added a new method `newStringLatin1NoRepl` to the `JavaLangAccess`.
>
> Reasons:
>
> * Most use cases of `newStringNoRepl` use `ISO_8859_1` as the charset,
> creating a new shortcut can make writing shorter;
> * Since all possible values of `byte` are legal Latin-1 characters,
> `newStringLat
On Mon, 26 Jun 2023 16:02:57 GMT, Chen Liang wrote:
> This is just a copycat of the original newStringNoRepl, which should be
> fixed; newStringUTF8NoRepl has no such issue.
Unfortunately, the behavior of `newStringUTF8NoRepl` is not the same as that of
`newStringNoRepl`, as it actually always
> Added a new method `newStringLatin1NoRepl` to the `JavaLangAccess`.
>
> Reasons:
>
> * Most use cases of `newStringNoRepl` use `ISO_8859_1` as the charset,
> creating a new shortcut can make writing shorter;
> * Since all possible values of `byte` are legal Latin-1 characters,
> `newStringLat
Added a new method `newStringLatin1NoRepl` to the `JavaLangAccess`.
Reasons:
* Most use cases of `newStringNoRepl` use `ISO_8859_1` as the charset, creating
a new shortcut can make writing shorter;
* Since all possible values of `byte` are legal Latin-1 characters,
`newStringLatin1NoRepl` **wil
On Mon, 26 Jun 2023 15:25:08 GMT, Glavo wrote:
> Added a new method `newStringLatin1NoRepl` to the `JavaLangAccess`.
>
> Reasons:
>
> * Most use cases of `newStringNoRepl` use `ISO_8859_1` as the charset,
> creating a new shortcut can make writing shorter;
> * Since all possible values of `byt
On Mon, 26 Jun 2023 15:25:08 GMT, Glavo wrote:
> Added a new method `newStringLatin1NoRepl` to the `JavaLangAccess`.
>
> Reasons:
>
> * Most use cases of `newStringNoRepl` use `ISO_8859_1` as the charset,
> creating a new shortcut can make writing shorter;
> * Since all possible values of `byt
On Mon, 22 May 2023 15:03:15 GMT, Weijun Wang wrote:
> Add realm name to the exception message, and make it the primary exception
> (retry exception added suppressed).
This pull request has now been integrated.
Changeset: e624484e
Author:Weijun Wang
URL:
https://git.openjdk.org/jdk
On Mon, 22 May 2023 15:03:15 GMT, Weijun Wang wrote:
> Add realm name to the exception message, and make it the primary exception
> (retry exception added suppressed).
Marked as reviewed by xuelei (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/14086#pullrequestreview-14
On Mon, 22 May 2023 15:03:15 GMT, Weijun Wang wrote:
> Add realm name to the exception message, and make it the primary exception
> (retry exception added suppressed).
Still worth fixing.
-
PR Comment: https://git.openjdk.org/jdk/pull/14086#issuecomment-1607442146
12 matches
Mail list logo