Re: RFR: 8330108: Increase CipherInputStream buffer size

2024-04-12 Thread David Schlosnagle
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote: > Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes. > > I have seen applications where this small buffer size significantly reduces > throughput, and I've even seen applications which use reflection to modify > the bu

RFR: 8044609: javax.net.debug "ssl" options are not working and documented as expected.

2024-04-12 Thread Sean Coffey
The `javax.net.debug` TLS debug option is buggy since TLSv1.3 implementation was introduced many years ago. Where "ssl" was previously a value to obtain all TLS debug traces (except network type dumps, verbose data), it now prints only a few lines for a standard client TLS connection. The pro

Re: RFR: 8330108: Increase CipherInputStream buffer size

2024-04-12 Thread Anthony Scarpino
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote: > Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes. > > I have seen applications where this small buffer size significantly reduces > throughput, and I've even seen applications which use reflection to modify > the bu

Re: RFR: 8330108: Increase CipherInputStream buffer size

2024-04-12 Thread Sean Mullan
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote: > Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes. > > I have seen applications where this small buffer size significantly reduces > throughput, and I've even seen applications which use reflection to modify > the bu

RFR: 8330108: Increase CipherInputStream buffer size

2024-04-12 Thread Oli Gillespie
Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes. I have seen applications where this small buffer size significantly reduces throughput, and I've even seen applications which use reflection to modify the buffer size to work around the issue. Using the existing `AESGCMCiph

Re: RFR: 8307143: CredentialsCache.cacheName should not be static [v4]

2024-04-12 Thread Weijun Wang
> The CC can be loaded with any file and its name is not static. > > `MemoryCredentialsCache` is removed since it's not used anywhere. We've > already supported native ccache reading directly with JNI method > `Credentials::acquireDefaultNativeCreds`. Weijun Wang has updated the pull request in

Re: RFR: 8307143: CredentialsCache.cacheName should not be static [v3]

2024-04-12 Thread Weijun Wang
> The CC can be loaded with any file and its name is not static. > > `MemoryCredentialsCache` is removed since it's not used anywhere. We've > already supported native ccache reading directly with JNI method > `Credentials::acquireDefaultNativeCreds`. Weijun Wang has updated the pull request in

Re: RFR: 8307143: CredentialsCache.cacheName should not be static [v2]

2024-04-12 Thread Weijun Wang
> The CC can be loaded with any file and its name is not static. > > `MemoryCredentialsCache` is removed since it's not used anywhere. We've > already supported native ccache reading directly with JNI method > `Credentials::acquireDefaultNativeCreds`. Weijun Wang has updated the pull request wi