Re: RFR: 8331008: KDF Implementation (Preview) [v2]

2024-05-13 Thread Sean Mullan
On Fri, 10 May 2024 21:13:22 GMT, Kevin Driver wrote: >> src/java.base/share/classes/javax/crypto/KDFSpi.java line 45: >> >>> 43: * provides access to the KDF alg params for implementers of the Spi >>> 44: */ >>> 45: protected final AlgorithmParameterSpec algorithmParameterSpec; >>

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Sean Mullan
On Mon, 13 May 2024 14:01:46 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> re-enable preview annotations > > src/java.base/share/classes/javax/crypto/KDFSpi.java line 59: > >> 57: * >> 58:

Re: RFR: 8331008: KDF Implementation (Preview) [v33]

2024-05-13 Thread Sean Mullan
On Mon, 13 May 2024 16:30:54 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v33]

2024-05-13 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Kevin Driver
On Mon, 13 May 2024 09:02:42 GMT, Alan Bateman wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> re-enable preview annotations > > src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 121: > >>

Re: RFR: 8331008: KDF Implementation (Preview) [v32]

2024-05-13 Thread Kevin Driver
On Thu, 9 May 2024 15:18:43 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> remove (now) extraneous null-check > > src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 352: >

Re: RFR: 8331008: KDF Implementation (Preview) [v32]

2024-05-13 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v31]

2024-05-13 Thread Kevin Driver
On Fri, 10 May 2024 15:51:56 GMT, Sean Mullan wrote: >> src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line >> 201: >> >>> 199: * the PRK (may be null) >>> 200: * @param info >>> 201: * the info (may be null) >> >> I know you use a null `prk` in `Ex

Re: RFR: 8331008: KDF Implementation (Preview) [v31]

2024-05-13 Thread Kevin Driver
On Thu, 9 May 2024 15:17:17 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - method reuse >> - null/boundary/modifiability checks > > src/java.base/share/classes/javax/crypto/spec/HKDFParameterSp

Re: RFR: 8331008: KDF Implementation (Preview) [v31]

2024-05-13 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with two additional commits since th

Re: RFR: 8331008: KDF Implementation (Preview) [v30]

2024-05-13 Thread Kevin Driver
On Thu, 9 May 2024 15:13:25 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> disallow null or empty values for ikm/salt rather than silent ignore > > src/java.base/share/classes/javax/crypto/spec/HK

Re: RFR: 8331008: KDF Implementation (Preview) [v30]

2024-05-13 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v30]

2024-05-13 Thread Kevin Driver
On Fri, 10 May 2024 14:58:38 GMT, Sean Mullan wrote: >> src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line >> 108: >> >>> 106: * >>> 107: * @param ikm >>> 108: * the ikm value (null values will not be added) >> >> Are you sure about allowin

Re: RFR: 8331008: KDF Implementation (Preview) [v29]

2024-05-13 Thread Kevin Driver
On Thu, 9 May 2024 13:01:16 GMT, Sibabrata Sahoo wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment re:returning null from derives > > src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDer

Re: RFR: 8331008: KDF Implementation (Preview) [v29]

2024-05-13 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v29]

2024-05-13 Thread Kevin Driver
On Mon, 13 May 2024 15:23:59 GMT, Kevin Driver wrote: >> src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java >> line 148: >> >>> 146: } >>> 147: >>> 148: return null; >> >> Should it throw exception instead of null? Also it will never reach here. > > An

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Sean Mullan
On Mon, 13 May 2024 13:55:48 GMT, Weijun Wang wrote: >> src/java.base/share/classes/javax/crypto/KDF.java line 50: >> >>> 48: * {@code KDF} objects are instantiated through the {@code getInstance} >>> family >>> 49: * of methods. Key derivation algorithm names follow a naming >>> convention

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-13 Thread Sean Mullan
On Mon, 13 May 2024 15:04:20 GMT, Sean Mullan wrote: >> Do we need to mention "at the same time" because it's thread-safe? > > Yes, how about: "The `deriveKey` and `deriveData` methods of `KDF` objects > must be thread-safe. That is, multiple threads may concurrently invoke these > methods on a

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-13 Thread Sean Mullan
On Mon, 13 May 2024 13:57:10 GMT, Weijun Wang wrote: >> src/java.base/share/classes/javax/crypto/KDF.java line 395: >> >>> 393: * >>> 394: * The {@code deriveKey} method may be called multiple times once a >>> 395: * {@code KDF} object is initialized. >> >> I think this sentence

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Weijun Wang
On Fri, 10 May 2024 20:55:47 GMT, Kevin Driver wrote: >> I agree. Also, if we do want to validate arguments (and I don't know if we >> need to), then I think the `Extract` constructor should be responsible for >> doing that, not the `Builder`. Doing it in `Extract` is safer since it is >> done

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-13 Thread Weijun Wang
On Sun, 12 May 2024 14:43:04 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> commenting out until better understood -- causing failures > > src/java.base/share/classes/javax/crypto/KDF.java line 39

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Weijun Wang
On Mon, 13 May 2024 09:18:55 GMT, Alan Bateman wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> re-enable preview annotations > > src/java.base/share/classes/javax/crypto/KDF.java line 50: > >> 48: * {@code KDF} o

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Weijun Wang
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-13 Thread Sean Mullan
On Sun, 12 May 2024 18:13:37 GMT, Weijun Wang wrote: >> src/java.base/share/classes/javax/crypto/KDF.java line 398: >> >>> 396: * >>> 397: * Delayed provider selection is also supported such that the >>> provider >>> 398: * performing the derive is not selected until the method

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Sean Mullan
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Alan Bateman
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Alan Bateman
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Alan Bateman
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Alan Bateman
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Alan Bateman
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Alan Bateman
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-13 Thread Alan Bateman
On Mon, 13 May 2024 03:46:50 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v28]

2024-05-12 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v27]

2024-05-12 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-12 Thread Weijun Wang
On Sun, 12 May 2024 14:39:40 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> commenting out until better understood -- causing failures > > src/java.base/share/classes/javax/crypto/KDF.java line 39

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-12 Thread Weijun Wang
On Sat, 11 May 2024 02:06:09 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-12 Thread Sean Mullan
On Sat, 11 May 2024 02:06:09 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-10 Thread Alan Bateman
On Sat, 11 May 2024 02:06:09 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v25]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v24]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v23]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v22]

2024-05-10 Thread Kevin Driver
On Fri, 10 May 2024 22:11:04 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v22]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with three additional commits since

Re: RFR: 8331008: KDF Implementation (Preview) [v21]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v18]

2024-05-10 Thread Kevin Driver
On Fri, 10 May 2024 14:54:18 GMT, Sean Mullan wrote: >> src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 70: >> >>> 68: */ >>> 69: public Extract extractOnly() { >>> 70: if (this.ikms.isEmpty() && this.salts.isEmpty()) { >> >> I don't think

Re: RFR: 8331008: KDF Implementation (Preview) [v20]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v2]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 17:29:15 GMT, Sean Mullan wrote: >> src/java.base/share/classes/javax/crypto/KDFSpi.java line 107: >> >>> 105: * if the key derivation implementation cannot support >>> additional calls to >>> 106: * {@code deriveData } or if all {@code >>> DerivationParame

Re: RFR: 8331008: KDF Implementation (Preview) [v2]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 16:42:09 GMT, Weijun Wang wrote: >> Kevin Driver 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 ten additional >> commits si

Re: RFR: 8331008: KDF Implementation (Preview) [v19]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v19]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 15:14:48 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> replace in-place SecretKeySpec alg for IKM/salt with "Generic" > > src/java.base/share/classes/javax/crypto/spec/HKDFPara

Re: RFR: 8331008: KDF Implementation (Preview) [v18]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 15:01:24 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> a start on the preview wire on/off > > src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 47: > >

Re: RFR: 8331008: KDF Implementation (Preview)

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 14:43:55 GMT, Weijun Wang wrote: > Two comments on HKDF: > > 1. Expand length cannot exceed 255 * HashLen. See > https://datatracker.ietf.org/doc/html/rfc5869#section-2.3. > 2. Why disallow empty ikms and salts? For the salt side, RFC5869 allows [no > salt](https://datatrack

Re: RFR: 8331008: KDF Implementation (Preview) [v18]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v17]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v16]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v3]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 21:21:55 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some code review comments > > src/java.base/share/classes/javax/crypto/KDF.java line 1: > >> 1: /* > > Delayed provider

Re: RFR: 8331008: KDF Implementation (Preview) [v15]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v3]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 20:33:57 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some code review comments > > src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java > line 365: >

Re: RFR: 8331008: KDF Implementation (Preview) [v14]

2024-05-10 Thread Kevin Driver
On Wed, 8 May 2024 20:53:54 GMT, Kevin Driver wrote: >> src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java >> line 370: >> >>> 368: } >>> 369: int rounds = (outLen + hmacLen - 1) / hmacLen; >>> 370: kdfOutput = new byte[rounds * hmacLen]; >> >> A

Re: RFR: 8331008: KDF Implementation (Preview) [v14]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 12:31:24 GMT, Sean Mullan wrote: >> src/java.base/share/classes/javax/crypto/KDF.java line 54: >> >>> 52: * the algorithm specifier may be omitted if the KDF algorithm has a >>> fixed or default PRF. >>> 53: * >>> 54: * TODO: finish this javadoc >> >> You should state th

Re: RFR: 8331008: KDF Implementation (Preview) [v14]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v13]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v12]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v11]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 15:30:22 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> continued code review comments > > src/java.base/share/classes/javax/crypto/KDF.java line 58: > >> 56: >> 57: public fi

Re: RFR: 8331008: KDF Implementation (Preview) [v11]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v10]

2024-05-10 Thread Kevin Driver
On Wed, 8 May 2024 20:25:36 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment > > src/java.base/share/classes/java/security/Provider.java line 1607: > >> 1605: addEngine(

Re: RFR: 8331008: KDF Implementation [v10]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 14:10:37 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment > > src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 36: > >> 34: * Parame

Re: RFR: 8331008: KDF Implementation [v10]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v7]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 13:14:45 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> additional delayed provider selection work > > src/java.base/share/classes/javax/crypto/KDF.java line 147: > >> 145:

Re: RFR: 8331008: KDF Implementation [v9]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v9]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 13:16:05 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment > > src/java.base/share/classes/javax/crypto/KDF.java line 150: > >> 148: KDF instance = nul

Re: RFR: 8331008: KDF Implementation [v8]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v8]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 13:07:41 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment > > src/java.base/share/classes/javax/crypto/KDF.java line 132: > >> 130: >> 131: private Strin

Re: RFR: 8331008: KDF Implementation [v7]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Sean Mullan
On Fri, 10 May 2024 14:56:55 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Sean Mullan
On Thu, 9 May 2024 15:11:39 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> change algorithm standard name for HKDFs in SunJCE provider > > src/java.base/share/classes/javax/crypto/spec/HKDFParamet

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Alan Bateman
On Fri, 10 May 2024 14:56:55 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-10 Thread Sean Mullan
On Fri, 10 May 2024 00:15:32 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Sean Mullan
On Thu, 9 May 2024 15:04:53 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> change algorithm standard name for HKDFs in SunJCE provider > > src/java.base/share/classes/javax/crypto/spec/HKDFParamet

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Kevin Driver
On Wed, 8 May 2024 20:29:01 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> change algorithm standard name for HKDFs in SunJCE provider > > src/java.base/share/classes/com/sun/crypto/provider/SunJC

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Kevin Driver
On Fri, 10 May 2024 14:50:51 GMT, Kevin Driver wrote: >> src/java.base/share/classes/com/sun/crypto/provider/SunJCE.java line 463: >> >>> 461: * Key Derivation engines >>> 462: */ >>> 463: ps("KDF", "HKDF/HmacSHA256", >> >> In the JEP, the example uses "HkdfSHA256". Th

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 20:20:00 GMT, Weijun Wang wrote: >> src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java >> line 53: >> >>> 51: * and Expand-only variants. >>> 52: */ >>> 53: abstract class HkdfKeyDerivation extends KDFSpi { >> >> How about just name it `HKDF`? > > M

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-10 Thread Kevin Driver
On Fri, 10 May 2024 12:12:06 GMT, Sean Mullan wrote: >> src/java.base/share/classes/com/sun/crypto/provider/SunJCE.java line 467: >> >>> 465: ps("KDF", "HkdfSHA384", >>> 466: >>> "com.sun.crypto.provider.HkdfKeyDerivation$HkdfSHA384"); >>> 467: ps("KDF", "HkdfSHA

Re: RFR: 8331008: KDF Implementation [v5]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-10 Thread Sean Mullan
On Thu, 9 May 2024 20:18:10 GMT, Kevin Driver wrote: >> src/java.base/share/classes/javax/crypto/KDF.java line 115: >> >>> 113: * {@code getInstance} calls that created this {@code >>> KeyDerivation} object. >>> 114: * >>> 115: * @return the algorithm name of this {@code KeyDeriv

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-10 Thread Sean Mullan
On Fri, 10 May 2024 02:27:31 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> adding in delayed provider selection > > src/java.base/share/classes/com/sun/crypto/provider/SunJCE.java line 467: > >>

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-09 Thread Weijun Wang
On Fri, 10 May 2024 00:15:32 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-09 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-09 Thread Weijun Wang
On Thu, 9 May 2024 19:46:39 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pul

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-09 Thread Weijun Wang
On Thu, 9 May 2024 19:46:39 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pul

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-09 Thread Kevin Driver
On Thu, 9 May 2024 20:31:30 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some code review comments > > src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java > line 223: >

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-09 Thread Weijun Wang
On Thu, 9 May 2024 20:19:41 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some code review comments > > src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java > line 53: > >

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-09 Thread Weijun Wang
On Thu, 9 May 2024 19:46:39 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pul

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-09 Thread Kevin Driver
On Thu, 9 May 2024 14:11:44 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some code review comments > > src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 40: > >> 38: *

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-09 Thread Kevin Driver
On Thu, 9 May 2024 12:22:14 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some code review comments > > src/java.base/share/classes/javax/crypto/KDF.java line 43: > >> 41: >> 42: /** >> 43: *

Re: RFR: 8331008: KDF Implementation [v2]

2024-05-09 Thread Kevin Driver
On Thu, 9 May 2024 17:33:30 GMT, Sean Mullan wrote: >> Kevin Driver 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 ten additional >> commits si

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-09 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v2]

2024-05-09 Thread Sean Mullan
On Thu, 9 May 2024 16:19:36 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pul

  1   2   >