This is an automated email from the ASF dual-hosted git repository.

bowenliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


The following commit(s) were added to refs/heads/master by this push:
     new dbcbeecb1 [KYUUBI #4850] Change default EC spec to secp521r1 for 
internal session variable signature
dbcbeecb1 is described below

commit dbcbeecb16d04c6ef704152a79546aba882a1a15
Author: liangbowen <[email protected]>
AuthorDate: Thu May 18 08:54:39 2023 +0800

    [KYUUBI #4850] Change default EC spec to secp521r1 for internal session 
variable signature
    
    ### _Why are the changes needed?_
    
    - change the default EC spec from secp256k1 to secp521r1, as secp256k1 not 
supported on Java 17
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including 
negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [x] [Run 
test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests)
 locally before make a pull request
    
    Closes #4850 from bowenliang123/secp384r1.
    
    Closes #4850
    
    73e0dad30 [liangbowen] change EC spec to secp521r1
    5b9528ede [liangbowen] change EC spec to secp384r1
    
    Authored-by: liangbowen <[email protected]>
    Signed-off-by: liangbowen <[email protected]>
---
 kyuubi-common/src/main/scala/org/apache/kyuubi/util/SignUtils.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/kyuubi-common/src/main/scala/org/apache/kyuubi/util/SignUtils.scala 
b/kyuubi-common/src/main/scala/org/apache/kyuubi/util/SignUtils.scala
index 6f7ff18df..7fb4fde2e 100644
--- a/kyuubi-common/src/main/scala/org/apache/kyuubi/util/SignUtils.scala
+++ b/kyuubi-common/src/main/scala/org/apache/kyuubi/util/SignUtils.scala
@@ -27,7 +27,7 @@ object SignUtils {
 
   private lazy val ecKeyPairGenerator = {
     val g = KeyPairGenerator.getInstance(KEYPAIR_ALGORITHM_EC)
-    g.initialize(new ECGenParameterSpec("secp256k1"), new SecureRandom())
+    g.initialize(new ECGenParameterSpec("secp521r1"), new SecureRandom())
     g
   }
 

Reply via email to