Hello,
Please review this fix for 9.
The test uses JAAS authentication over Kerberos. The kerberos5 principal name
can be specified in the configuration entry by using the option principal. The
principal can also be set using the system property
sun.security.krb5.principal. This property
]
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Tuesday, June 30, 2015 7:45 AM
To: Sibabrata Sahoo; Security Libs OpenJDK
Subject: Re: [9] RFR: 8075301: Tests for sun.security.krb5.principal system
property
Hi Siba
There is one case not covered:
1. sun.security.krb5.pri
Hi Xuelei,
Please review a minor fix for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8211787
Webrev: http://cr.openjdk.java.net/~ssahoo/8211787/webrev.00/
The original intention to "setSoTimeout()" is on serverSocket where the server
will fail with timeout if it exceed certain amount
Hi Xuelei/Brad,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8224650
Webrev: http://cr.openjdk.java.net/~ssahoo/8224650/webrev.00/
This is a small Test inherited from "SSLSocketTemplate" and reuse most part of
it. The only difference is, it uses supported
Here is the updated webrev:
http://cr.openjdk.java.net/~asmotrak/siba/8130360/webrev.02/
I have one question:
What should be the behavior when the older version of 3rd party JCE provider
jar file(without service descriptor "META-INF/services/*" & working with <=
JDK8) configured by "java.securi
nt: Monday, November 30, 2015 4:54 PM
To: Sibabrata Sahoo; [email protected]; [email protected]
Subject: Re: [9] RFR:8130360: Add tests to verify 3rd party security providers
if they are in signed/unsigned modular JARs
On 30/11/2015 11:13, Sibabrata Sahoo wrote:
> Here i
quot; file
should have the provider configuration entry as ProviderName while in case of
ClassPath the entry should hold the full class name.
Thanks,
Siba
-Original Message-
From: Valerie Peng
Sent: Tuesday, December 08, 2015 4:46 AM
To: Sibabrata Sahoo
Cc: Alan Bateman; security-dev@openjd
- Sign jar functionality removed.
Thanks,
Siba
-Original Message-
From: Valerie Peng
Sent: Wednesday, December 09, 2015 5:37 AM
To: Sibabrata Sahoo
Cc: [email protected]; [email protected]; Alan Bateman
Subject: Re: [9] RFR:8130360: Add tests to verify 3rd party security
/security/auth/login/modules/src/" and all its subfolders need to be
removed.
- Test converted to use TestNG framework.
Thanks,
Siba
-Original Message-----
From: Mandy Chung
Sent: Wednesday, December 09, 2015 11:55 PM
To: Sibabrata Sahoo
Cc: Valerie Peng; [email protected]; se
Hi Mandy/Max,
Here is the updated webrev with very minor change over previous:
http://cr.openjdk.java.net/~asmotrak/siba/8130360/webrev.04/
Thanks
Siba
-Original Message-
From: Sibabrata Sahoo
Sent: Sunday, December 20, 2015 1:34 AM
To: Mandy Chung; Weijun Wang
Cc: Valerie Peng
. SecurityProviderModularTest and JaasModularClientTest.
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Monday, December 21, 2015 7:20 AM
To: Sibabrata Sahoo
Cc: Mandy Chung; Valerie Peng; [email protected];
[email protected]
Subject: Re: [9] RFR:8130360: Add tests to verify 3rd
nuary 02, 2016 3:11 PM
To: Sibabrata Sahoo
Cc: Mandy Chung; Valerie Peng; [email protected]; OpenJDK
Subject: Re: [9] RFR:8130360: Add tests to verify 3rd party security providers
if they are in signed/unsigned modular JARs
Looks better.
One question: the bug description mentions
Updated the bug title and description.
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Sunday, January 03, 2016 1:02 PM
To: Sibabrata Sahoo
Cc: Mandy Chung; Valerie Peng; [email protected]; OpenJDK
Subject: Re: [9] RFR:8130360: Add tests to verify 3rd party security
Hi Valerie,
If everything looks fine in the current version of webrev then can you please
help me push the changes to JAKE repo.
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Monday, January 04, 2016 2:36 PM
To: Sibabrata Sahoo
Cc: Mandy Chung; Valerie Peng; jigsaw
Hello,
Please review the updated test to verify "jdk.security.provider.preferred"
security property in all platform.
Bug: https://bugs.openjdk.java.net/browse/JDK-8150512
Webrev: http://cr.openjdk.java.net/~ssahoo/8150512/webrev.00/
These test were initially planned to run on Solaris
Hi Anthony,
Please find the updated webrev:
http://cr.openjdk.java.net/~ssahoo/8150512/webrev.01/
Following comments associated.
Thanks,
Siba
---
Here are my comments...
- PreferredProviderTest.java:62-70
You appear to be missing SHA1 and SHA-224. They should be added to the
Hello,
Please review the tests for "JDK-8141039:Test Task: Develop new tests for JEP
273: DRBG-Based SecureRandom Implementations".
Bug: https://bugs.openjdk.java.net/browse/JDK-8141039
Webrev: http://cr.openjdk.java.net/~ssahoo/8141039/webrev.00/
The tests covers the new DRBG API wit
e it accordingly.
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Tuesday, May 10, 2016 1:14 PM
To: Sibabrata Sahoo
Cc: [email protected]
Subject: Re: [9] RFR: 8141039: Test Task: Develop new tests for JEP 273:
DRBG-Based SecureRandom Implementations
For all:
andom/ ".
- Removed all reference to MoreDrbgParameters and EntropySource.
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Thursday, May 12, 2016 9:14 AM
To: Sibabrata Sahoo
Cc: [email protected]
Subject: Re: [9] RFR: 8141039: Test Task: Develop new tests f
urity/SecureRandom ".
- Removed reference to MoreDrbgParameters from ApiTest.java
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Monday, May 16, 2016 6:36 PM
To: Sibabrata Sahoo
Cc: [email protected]
Subject: Re: [9] RFR: 8141039: Test Task: Develop new tests f
RBG. So I am keeping the
file unchanged.
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Wednesday, May 18, 2016 1:37 PM
To: Sibabrata Sahoo
Cc: [email protected]
Subject: Re: [9] RFR: 8141039: Test Task: Develop new tests for JEP 273:
DRBG-Based SecureRandom Implementa
Hi Max,
Here is the updated webrev:
http://cr.openjdk.java.net/~ssahoo/8141039/webrev.05/
Can you please help me push the change.
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Thursday, May 19, 2016 1:19 PM
To: Sibabrata Sahoo
Cc: [email protected]
Subject: Re
Hi,
Please review the following fix for SecureRandom test which might cause timeout
in Linux.
webrev: http://cr.openjdk.java.net/~ssahoo/8157417/webrev.00/
JBS: https://bugs.openjdk.java.net/browse/JDK-8157417
The change includes:
- Added "-Djava.security.egd=file:/dev/urand
I missed to include one more test.
Here is the updated one: http://cr.openjdk.java.net/~ssahoo/8157417/webrev.01/
sun/security/provider/SeedGenerator/SeedGeneratorChoice.java might also fail in
Linux without the fix provided.
Thanks,
Siba
From: Sibabrata Sahoo
Sent: Friday, May 20
Hi,
Please review the fix for the following JBS bugs,
JBS: https://bugs.openjdk.java.net/browse/JDK-8157896
https://bugs.openjdk.java.net/browse/JDK-8157898
webrev: http://cr.openjdk.java.net/~ssahoo/8157896/webrev.00/
Description:
These Tests takes a lots of time to generat
Hi Xuelei,
Please find the updated webrev:
http://cr.openjdk.java.net/~ssahoo/8157896/webrev.01/
As suggested, I have increased the timeout for (3072 256) to 700.
Thanks,
Siba
-Original Message-
From: Xuelei Fan
Sent: Saturday, June 04, 2016 6:02 AM
To: Sibabrata Sahoo; Valerie Peng
Hi,
Please review this patch for "8159861:
sun/security/tools/keytool/DefaultSignatureAlgorithm.java timeout"
JBS: https://bugs.openjdk.java.net/browse/JDK-8159861
Webrev: http://cr.openjdk.java.net/~ssahoo/8159861/webrev.00/
Description:
The test generates several set of keypair amo
Hi,
Please review this patch for "8160341: Remove intermittent key from
TestDSAGenParameterSpec.java"
JBS: https://bugs.openjdk.java.net/browse/JDK-8160341
Webrev: http://cr.openjdk.java.net/~ssahoo/8160341/webrev.00/
Description:
Removing intermittent keyword from the test.
Than
Hi,
Please review this patch for "8160624: sun/security/tools/keytool/printssl.sh
failed with "Socket closed"
JBS: https://bugs.openjdk.java.net/browse/JDK-8160624
Webrev: http://cr.openjdk.java.net/~ssahoo/8160624/webrev.00/
Description:
The test is for "keytool should be able to im
Hi,
Please review this patch for "8158116:
com/sun/crypto/provider/KeyAgreement/SupportedDHParamGens.java failed with
timeout"
JBS: https://bugs.openjdk.java.net/browse/JDK-8158116
Webrev: http://cr.openjdk.java.net/~ssahoo/8158116/webrev.00/
Description:
This JBS bug does not have
Hi,
Please review the patch for "8160210:
javax/net/ssl/HttpsURLConnection/Equals.java failed with "SSL peer shut down
incorrectly""
JBS: https://bugs.openjdk.java.net/browse/JDK-8160210
Webrev: http://cr.openjdk.java.net/~ssahoo/8160210/webrev.00/
Description:
This is a rare failur
Hi,
Please review the patch for "8079441: Intermittent failures on Windows with
"Unexpected exit from test [exit code: 1080890248]""
JBS: https://bugs.openjdk.java.net/browse/JDK-8079441
Webrev: http://cr.openjdk.java.net/~ssahoo/8079441/webrev.00/
Description:
This is a rare failure
al Message-
From: Xuelei Fan
Sent: Tuesday, July 05, 2016 6:03 PM
To: Sibabrata Sahoo; [email protected]
Subject: Re: [9] RFR: 8160210: javax/net/ssl/HttpsURLConnection/Equals.java
failed with "SSL peer shut down incorrectly"
You may also want to dump the SSL record so t
Hi,
Please review the following patch for "Additional modular test for
"auth.login.defaultCallbackHandler" property".
JBS: https://bugs.openjdk.java.net/browse/JDK-8151654
Webrev: http://cr.openjdk.java.net/~ssahoo/8151654/webrev.00/
Description:
A custom callback handler for JAAS ca
sage-
From: Wang Weijun
Sent: Sunday, July 17, 2016 1:12 PM
To: Sibabrata Sahoo
Cc: [email protected]; Mandy Chung; Valerie Peng
Subject: Re: [9] RFR: 8151654: Additional modular test for
"auth.login.defaultCallbackHandler" property
Why change C_BUILD_DIR to C_BLD_DI
Hi,
Please review the patch for "deprivilege java.xml.crypto, java.security.jgss,
jdk.security.jgss and jdk.security.auth" modules.
JBS: https://bugs.openjdk.java.net/browse/JDK-8159964
Webrev: http://cr.openjdk.java.net/~ssahoo/8159964/webrev.00/
Description:
This includes a new Tes
Hi Max,
Here is the updated webrev:
http://cr.openjdk.java.net/~ssahoo/8159964/webrev.01/
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Friday, August 05, 2016 4:19 PM
To: Sibabrata Sahoo; [email protected]; Valerie Peng
Subject: Re: [9] RFR: 8159964: Update
Ping..
-Original Message-
From: Sibabrata Sahoo
Sent: Friday, August 05, 2016 7:28 PM
To: Weijun Wang; [email protected]; Valerie Peng
Subject: RE: [9] RFR: 8159964: Update Tests to verify JDK build for
"JDK-8159488 Deprivilege java.xml.crypto"
Hi Max,
Here is t
.net/~ssahoo/8159964/webrev.02/
Thanks,
Siba
-Original Message-
From: Wang Weijun
Sent: Tuesday, August 09, 2016 1:21 PM
To: Sibabrata Sahoo
Cc: [email protected]; Valerie Peng
Subject: Re: [9] RFR: 8159964: Update Tests to verify JDK build for
"JDK-8159488 De
Hi,
Please review the patch for 2 issues on
"sun/security/krb5/auto/MaxRetries.java".
JBS: https://bugs.openjdk.java.net/browse/JDK-8153146
https://bugs.openjdk.java.net/browse/JDK-8087144
Webrev: http://cr.openjdk.java.net/~ssahoo/8087144/webrev.00/
Description:
MaxRetries
Hi,
Please review the patch for intermittent failures in
"sun/security/krb5/auto/BadKdc*"
JBS: https://bugs.openjdk.java.net/browse/JDK-8006690
Webrev: http://cr.openjdk.java.net/~ssahoo/8006690/webrev.00/
Description:
Some times when the test machine is slow the number of retry for
Hi,
Please review the patch for "sun/security/krb5/auto/Unreachable.java fails with
Timeout error"
JBS: https://bugs.openjdk.java.net/browse/JDK-8015595
Webrev: http://cr.openjdk.java.net/~ssahoo/8015595/webrev.00/
Description:
When a KDC port is unreachable, Kerberos login module de
warning.
2) Removed the test from ProblemList.txt for MAC OS because of #1.
3) Uses only one KDC port in the configuration file.
4) Removed static "unreachable.krb5.conf" as the Test creates the file
during runtime.
Thanks,
Siba
From: Sibabrata Sahoo
Sent:
Hi Max,
Please find the updated webrev addressing all comments bellow:
http://cr.openjdk.java.net/~ssahoo/8015595/webrev.01/
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Monday, August 29, 2016 6:36 AM
To: Sibabrata Sahoo; [email protected]
Subject: Re: [9] RFR
Hi Max,
Here is updated webrev: http://cr.openjdk.java.net/~ssahoo/8015595/webrev.02/
Addressed all the following comments.
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Monday, August 29, 2016 6:14 PM
To: Sibabrata Sahoo; [email protected]
Subject: Re: [9] RFR
Hi,
Please review the patch for "sun/security/provider/SecureRandom/AutoReseed.java
failed with timeout"
JBS: https://bugs.openjdk.java.net/browse/JDK-8164922
Webrev: http://cr.openjdk.java.net/~ssahoo/8164922/webrev.00/
Description:
The Test was blocked while generating seed. I have
: Tuesday, August 30, 2016 1:40 PM
To: Sibabrata Sahoo; [email protected]
Subject: Re: [9] RFR: 8164922:
sun/security/provider/SecureRandom/AutoReseed.java failed with timeout in
Ubuntu Linux.
Please just use othervm, and remove the finally block. If oldegd is null the
code change will
Hi,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8156054
Webrev: http://cr.openjdk.java.net/~ssahoo/8156054/webrev.00/
Description:
It checks equals(), implies() and hashCode () of FilePermission when
"jdk.io.permissionsUseCanonicalPath" set and un-set.
Hi,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8165825
Webrev: http://cr.openjdk.java.net/~ssahoo/8165825/webrev.00/
Description:
This is a simple fix for removing intermittent keyword from
sun/security/krb5/auto/Unreachable.java file.
Thanks,
Si
Hi,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8165660
Webrev: http://cr.openjdk.java.net/~ssahoo/8165660/webrev.00/
Description:
This is a simple fix for removing intermittent keyword from
sun/security/krb5/auto/MaxRetries.java file.
Thanks,
Hi Max,
Here is the updated webrev:
http://cr.openjdk.java.net/~ssahoo/8156054/webrev.01/
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Tuesday, September 13, 2016 4:25 PM
To: Sibabrata Sahoo; [email protected]
Subject: Re: [9] RFR: 8156054: Test Task: Develop
Hi Artem,
I think building a String value from the actual result and displaying it
against expected in the Log output will be more convenient here.
Thanks,
Siba
From: Artem Smotrakov
Sent: Tuesday, September 13, 2016 10:54 PM
To: Sibabrata Sahoo; Weijun Wang; security-dev
Hi,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8170247
Webrev: http://cr.openjdk.java.net/~ssahoo/8170247/webrev.00/
Description:
The Test was failing to handle the expected failure for invalid parameters,
when the SecureRandom parameter size is > algo
Hi,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8168423
Webrev: http://cr.openjdk.java.net/~ssahoo/8168423/webrev.00/
Description:
This webrev address all possible cases for Classloader with SecurityManager
having combination of valid/malformed policy
Hi,
Please review the following patch,
JBS: https://bugs.openjdk.java.net/browse/JDK-8161232
Webrev: http://cr.openjdk.java.net/~ssahoo/8161232/webrev.00/
Description:
It was reported the Test was failing with timeout consistently on macOS. There
is no log available anymore and the i
Hi Adam/Sean,
This patch is waiting for your review.
Thanks,
Siba
From: Sibabrata Sahoo
Sent: Friday, December 02, 2016 6:56 PM
To: Sean Mullan; [email protected]
Subject: [9] RFR: 8168423: Test Task: Custom system class loader + security
manager + malformed policy file
LoaderTest.java, the code comments has been removed from @summary
section. But it retains the same at line: 91-102.
Thanks,
Siba
-Original Message-
From: Sean Mullan
Sent: Friday, January 27, 2017 12:07 AM
To: Sibabrata Sahoo; Adam Petcher; [email protected]
Subject: Re: [9] RFR
repository, in revision 16548
Regarding the following comment on " grant codeBase "file:./jars/*" ", we have
already discussed and we are fine here to not make any change.
Thanks,
Siba
-Original Message-
From: Sean Mullan
Sent: Wednesday, February 08, 2017 10:00 PM
To: S
Change looks fine to me. But I am not the reviewer yet.
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Tuesday, June 06, 2017 11:23 AM
To: Security Dev OpenJDK
Cc: Gustavo Galimberti; Sibabrata Sahoo
Subject: RFR 8181461: sun/security/krb5/auto/KdcPolicy.java fails with
Hi,
Please review the patch for the following,
JBS: https://bugs.openjdk.java.net/browse/JDK-8183310
Webrev: http://cr.openjdk.java.net/~ssahoo/8183310/webrev.00/
Change description:
The code has been changed significantly for cleaning up existing code and to
simplify it. During clea
Hi,
This patch is pending for your review.
Thanks,
Siba
From: Sibabrata Sahoo
Sent: Saturday, July 15, 2017 4:00 PM
To: Weijun Wang; Valerie Peng; [email protected]
Subject: [10] RFR: JDK-8183310: java/security/modules/ModularTest.java should
clean up better
Hi
additional Test case for regular jars in modulepath and modular
jars in classpath, which verifies how the type get resolved.
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Wednesday, August 02, 2017 3:53 PM
To: Sibabrata Sahoo
Cc: Valerie Peng; [email protected]
Subjec
Hi Max,
I have splitted the @run to convert each run to consume minimum time to avoid
any timeout for rare cases.
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Wednesday, August 23, 2017 1:43 PM
To: Sibabrata Sahoo
Cc: Valerie Peng; [email protected]
Subject: Re
: Sibabrata Sahoo
Cc: Valerie Peng; [email protected]
Subject: Re: [10] RFR: JDK-8183310: java/security/modules/ModularTest.java
should clean up better
Then maybe you can also make setup() a separate @run.
--Max
> On Aug 23, 2017, at 4:17 PM, Sibabrata Sahoo
> wrote:
>
Hi,
Please review the change for the following patch,
JBS: https://bugs.openjdk.java.net/browse/JDK-8194486
Webrev: http://cr.openjdk.java.net/~ssahoo/8194486/webrev.00/
Description:
There are several failure occurred recently because the name service used by
these Tests were not det
I will remove the commented lines. I think, it should be fine, if I do not
submit the webrev again.
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Wednesday, January 10, 2018 9:07 PM
To: Sibabrata Sahoo
Cc: [email protected]
Subject: Re: [11] RFR: 8194486: Several
ollowing 2 Test files inside
the same folder.
ReplayCacheExpunge.java
ReplayCachePrecise.java
Thanks,
Siba
-Original Message-
From: Weijun Wang
Sent: Thursday, January 11, 2018 11:33 AM
To: Sibabrata Sahoo
Cc: [email protected]
Subject: Re: [11] RFR: 8194486: Several krb5 te
Hi,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8184359
Webrev: http://cr.openjdk.java.net/~ssahoo/8184359/webrev.00/
All the Test files uses KeyAgreement, KeyPairGenerator, Several KeySpecs from
SunJCE library to Test DiffieHellman, ECDH and XDH with cu
Hi Sean,
My comments In-lined..
Thanks,
Siba
-Original Message-
From: Sean Mullan
Sent: Saturday, March 31, 2018 12:13 AM
To: Sibabrata Sahoo ; Adam Petcher
; [email protected]
Subject: Re: RFR: 8200219: Develop new tests for using new elliptic curves:
curve25519 and
y methods from
Convert.java are using the library instead of defining it's own.
4) MultiThreadTest.java generates two key pairs and do two key agreement
operations.
Thanks,
Siba
-Original Message-
From: Sean Mullan
Sent: Thursday, April 05, 2018 8:46 PM
To: Sibabrata
Sean Mullan
Sent: Wednesday, April 11, 2018 1:42 AM
To: Sibabrata Sahoo ; Adam Petcher
; [email protected]
Subject: Re: RFR: 8200219: Develop new tests for using new elliptic curves:
curve25519 and curve448
On 4/9/18 10:13 AM, Sibabrata Sahoo wrote:
> Here is the new patch address
Hi Xuelei,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8205111
Webrev: http://cr.openjdk.java.net/~ssahoo/8205111/webrev.00/
Change:
This Test file verifies all TLS protocols with the supported keytypes.
Thanks,
Siba
ded to have the flexibility to change the parameter(Cipher)
which are not in order(shuffled).
Thanks,
Siba
-Original Message-
From: Xuelei Fan
Sent: Thursday, June 21, 2018 7:28 PM
To: Sibabrata Sahoo ; [email protected]
Subject: Re: [11] RFR: JDK-8205111: Develop new Test to v
Hi,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8205653
Webrev: http://cr.openjdk.java.net/~ssahoo/8205653/webrev.00/
Change:
The Test has been upgraded to address the following 2 cases,
Add protocol support for TLSv1.3. The change is done in the config
Hi Daniel,
I am not a member of HYPERLINK
"mailto:[email protected]"[email protected]. So the review posted
is waiting for moderator's approval.
Thanks,
Siba
From: Sibabrata Sahoo
Sent: Thursday, June 28, 2018 8:40 PM
To: [email protected]
Resending the email after subscribing to HYPERLINK
"mailto:[email protected]"[email protected]
Thanks,
Siba
From: Sibabrata Sahoo
Sent: Thursday, June 28, 2018 8:40 PM
To: HYPERLINK "mailto:[email protected]"[email protected];
HYPERLINK
May I get the approval from [email protected].
Thanks,
Siba
-Original Message-
From: Xuelei Fan
Sent: Thursday, June 28, 2018 9:27 PM
To: Daniel Fuchs ; Sibabrata Sahoo
; [email protected];
[email protected]; [email protected]
Hi,
Please review the change for the following patch,
JBS: https://bugs.openjdk.java.net/browse/JDK-8206355
Webrev: http://cr.openjdk.java.net/~ssahoo/8206355/webrev.00/
It fixes SSLSessionImpl.getLocalPrincipal() implementation when client side
authentication is not enabled.
Than
Hi Xuelei,
Please review the patch for,
JBS: https://bugs.openjdk.java.net/browse/JDK-8208496
Webrev: http://cr.openjdk.java.net/~ssahoo/8208496/webrev.00/
This is a new Test which test concurrent behavior of TLS. It uses 50 client
thread to access a single server port concurrently an
: Sibabrata Sahoo
Cc: Xue-Lei Fan ; [email protected]
Subject: Re: [11] RFR: 8208496: New Test to verify concurrent behavior for TLS.
Hi Siba,
Would it be better to check how many connections the server accepts?
In your case, the server must accept 50 (no more no less) connections;
otherwise
serverSocket.close() and serverSocket.accept() are not declared synchronized in
java/net/ServerSocket.java.
Thanks,
Siba
-Original Message-
From: Xuelei Fan
Sent: Friday, August 03, 2018 10:38 PM
To: Sibabrata Sahoo ; John Jiang
Cc: [email protected]
Subject: Re: [11] RFR
Domain value for system property jdk.https.negotiate.cbt is case-insensitive
now. Included Test has been updated to address the change.
-
Commit messages:
- 8282293: Domain value for system property jdk.https.negotiate.cbt should be
case-insensitive
Changes: https://git.openjdk.j
> Domain value for system property jdk.https.negotiate.cbt is case-insensitive
> now. Included Test has been updated to address the change.
Sibabrata Sahoo has updated the pull request incrementally with one additional
commit since the last revision:
Update HttpsC
> Domain value for system property jdk.https.negotiate.cbt is case-insensitive
> now. Included Test has been updated to address the change.
Sibabrata Sahoo has updated the pull request incrementally with one additional
commit since the last revision:
On Tue, 15 Mar 2022 19:46:09 GMT, Weijun Wang wrote:
>> Sibabrata Sahoo has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update HttpsCB.java
>
> src/java.base/share/clas
On Wed, 9 Mar 2022 15:03:31 GMT, Sibabrata Sahoo wrote:
> Domain value for system property jdk.https.negotiate.cbt is case-insensitive
> now. Included Test has been updated to address the change.
This pull request has now been integrated.
Changeset: 86015e15
Author:Sibabrata Saho
Added coverage to logout method.
-
Commit messages:
- 8281717: Cover logout method for several LoginModule
Changes: https://git.openjdk.java.net/jdk/pull/7940/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7940&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK
> Added coverage to logout method.
Sibabrata Sahoo has updated the pull request incrementally with two additional
commits since the last revision:
- Update AllPlatforms.java
- Update AllPlatforms.java
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/7940/files
-
On Thu, 24 Mar 2022 23:02:47 GMT, Rajan Halade wrote:
>> Sibabrata Sahoo has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Update AllPlatforms.java
>> - Update AllPlatforms.java
>
> t
> Added coverage to logout method.
Sibabrata Sahoo has updated the pull request incrementally with one additional
commit since the last revision:
Update AllPlatforms.java
Updated exception handling to ignore the case where the running platform
doesn't match the test case. In t
On Mon, 11 Apr 2022 04:02:44 GMT, John Jiang wrote:
> JDK-8228396 turned stateless resumption on by default, but the JavaDoc was
> not modified accordingly.
> And a "{" is missing for @systemProperty
> jdk.tls.server.enableSessionTicketExtension.
Marked as reviewed by ssahoo (Committer).
A Test updated to cover the getCodeSigners.
-
Commit messages:
- Update Implies.java
- 8209935: Test to cover CodeSource.getCodeSigners()
Changes: https://git.openjdk.java.net/jdk/pull/8286/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8286&range=00
Issue: https://
On Mon, 18 Apr 2022 13:22:35 GMT, Sibabrata Sahoo wrote:
>> A Test updated to cover the getCodeSigners.
>
> Sibabrata Sahoo has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Update Implies.java
All comments addresse
> A Test updated to cover the getCodeSigners.
Sibabrata Sahoo has updated the pull request incrementally with one additional
commit since the last revision:
Update Implies.java
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/8286/files
- new: ht
A new API to execute kinit.
-
Commit messages:
- 8286969: Add a new test library API to execute kinit in SecurityTools.java
Changes: https://git.openjdk.java.net/jdk/pull/8775/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8775&range=00
Issue: https://bugs.openjdk.ja
On Wed, 18 May 2022 16:19:40 GMT, Sibabrata Sahoo wrote:
> A new API to execute kinit.
This pull request has now been integrated.
Changeset: dbda0e2b
Author: Sibabrata Sahoo
URL:
https://git.openjdk.java.net/jdk/commit/dbda0e2bda5d8ba86f068684941a05387947d993
Stats: 12 lines
> A Test updated to cover the getCodeSigners.
Sibabrata Sahoo has updated the pull request with a new target base due to a
merge or a rebase. The incremental webrev excludes the unrelated changes
brought in by the merge/rebase. The pull request contains four additional
commits since the l
> A Test updated to cover the getCodeSigners.
Sibabrata Sahoo has updated the pull request incrementally with one additional
commit since the last revision:
8209935: Test to cover CodeSource.getCodeSigners()
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/8286/fi
On Thu, 9 Jun 2022 21:12:26 GMT, Sean Mullan wrote:
>> Sibabrata Sahoo has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8209935: Test to cover CodeSource.getCodeSigners()
>
> test/jdk/java/security/Code
> A Test updated to cover the getCodeSigners.
Sibabrata Sahoo has updated the pull request incrementally with two additional
commits since the last revision:
- 8209935: Test to cover CodeSource.getCodeSigners()
- 8209935: Test to cover CodeSource.getCodeSigners()
-
Chan
1 - 100 of 140 matches
Mail list logo
