• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • ssh-agent-plugin

      Currently when connecting to ECDHE-RSA-* servers I was getting errors.

      Upgrading to bouncycastle 1.52 resolves these

      Below is an example exception; seen via the stash-notification-plugin, and ssh-agent-plugin

      javax.net.ssl.SSLHandshakeException: Could not generate secret
      at sun.security.ssl.ECDHCrypt.getAgreedSecret(ECDHCrypt.java:99)
      at sun.security.ssl.ClientHandshaker.serverHelloDone(ClientHandshaker.java:1045)
      at sun.security.ssl.ClientHandshaker.processMessage(ClientHandshaker.java:348)
      at sun.security.ssl.Handshaker.processLoop(Handshaker.java:979)
      at sun.security.ssl.Handshaker.process_record(Handshaker.java:914)
      at sun.security.ssl.SSLSocketImpl.readRecord(SSLSocketImpl.java:1050)
      at sun.security.ssl.SSLSocketImpl.performInitialHandshake(SSLSocketImpl.java:1363)
      at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1391)
      at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1375)
      at org.apache.http.conn.ssl.SSLConnectionSocketFactory.createLayeredSocket(SSLConnectionSocketFactory.java:394)
      at org.apache.http.conn.ssl.SSLConnectionSocketFactory.connectSocket(SSLConnectionSocketFactory.java:353)
      at org.apache.http.impl.conn.DefaultHttpClientConnectionOperator.connect(DefaultHttpClientConnectionOperator.java:134)
      at org.apache.http.impl.conn.BasicHttpClientConnectionManager.connect(BasicHttpClientConnectionManager.java:338)
      at org.apache.http.impl.execchain.MainClientExec.establishRoute(MainClientExec.java:380)
      at org.apache.http.impl.execchain.MainClientExec.execute(MainClientExec.java:236)
      at org.apache.http.impl.execchain.ProtocolExec.execute(ProtocolExec.java:184)
      at org.apache.http.impl.execchain.RetryExec.execute(RetryExec.java:88)
      at org.apache.http.impl.execchain.RedirectExec.execute(RedirectExec.java:110)
      at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:184)
      at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:82)
      at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:107)
      at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:55)
      at org.jenkinsci.plugins.stashNotifier.StashNotifier.notifyStash(StashNotifier.java:556)
      at org.jenkinsci.plugins.stashNotifier.StashNotifier.processJenkinsEvent(StashNotifier.java:207)
      at org.jenkinsci.plugins.stashNotifier.StashNotifier.prebuild(StashNotifier.java:160)
      at hudson.model.AbstractBuild$AbstractBuildExecution.preBuild(AbstractBuild.java:834)
      at hudson.model.AbstractBuild$AbstractBuildExecution.preBuild(AbstractBuild.java:829)
      at hudson.model.Build$BuildExecution.doRun(Build.java:144)
      at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:537)
      at hudson.model.Run.execute(Run.java:1741)
      at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
      at hudson.model.ResourceController.execute(ResourceController.java:98)
      at hudson.model.Executor.run(Executor.java:381)
      Caused by: java.security.InvalidKeyException: ECDH key agreement requires ECPublicKey for doPhase
      at org.bouncycastle.jcajce.provider.asymmetric.ec.KeyAgreementSpi.engineDoPhase(Unknown Source)
      at javax.crypto.KeyAgreement.doPhase(KeyAgreement.java:567)
      at sun.security.ssl.ECDHCrypt.getAgreedSecret(ECDHCrypt.java:96)
      ... 32 more
      

          [JENKINS-30109] Upgrade ssh-agent-plugin bouncycastle to 1.53

          Josh Toft created issue -
          Josh Toft made changes -
          Description Original: Currently when connecting to ECDHE-RSA-* servers I was getting errors.

          Upgrading to bouncycastle 1.52 resolves these

          Below is an example exception; seen via the stash-notification-plugin, and ssh-agent-plugin

          javax.net.ssl.SSLHandshakeException: Could not generate secret
          at sun.security.ssl.ECDHCrypt.getAgreedSecret(ECDHCrypt.java:99)
          at sun.security.ssl.ClientHandshaker.serverHelloDone(ClientHandshaker.java:1045)
          at sun.security.ssl.ClientHandshaker.processMessage(ClientHandshaker.java:348)
          at sun.security.ssl.Handshaker.processLoop(Handshaker.java:979)
          at sun.security.ssl.Handshaker.process_record(Handshaker.java:914)
          at sun.security.ssl.SSLSocketImpl.readRecord(SSLSocketImpl.java:1050)
          at sun.security.ssl.SSLSocketImpl.performInitialHandshake(SSLSocketImpl.java:1363)
          at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1391)
          at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1375)
          at org.apache.http.conn.ssl.SSLConnectionSocketFactory.createLayeredSocket(SSLConnectionSocketFactory.java:394)
          at org.apache.http.conn.ssl.SSLConnectionSocketFactory.connectSocket(SSLConnectionSocketFactory.java:353)
          at org.apache.http.impl.conn.DefaultHttpClientConnectionOperator.connect(DefaultHttpClientConnectionOperator.java:134)
          at org.apache.http.impl.conn.BasicHttpClientConnectionManager.connect(BasicHttpClientConnectionManager.java:338)
          at org.apache.http.impl.execchain.MainClientExec.establishRoute(MainClientExec.java:380)
          at org.apache.http.impl.execchain.MainClientExec.execute(MainClientExec.java:236)
          at org.apache.http.impl.execchain.ProtocolExec.execute(ProtocolExec.java:184)
          at org.apache.http.impl.execchain.RetryExec.execute(RetryExec.java:88)
          at org.apache.http.impl.execchain.RedirectExec.execute(RedirectExec.java:110)
          at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:184)
          at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:82)
          at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:107)
          at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:55)
          at org.jenkinsci.plugins.stashNotifier.StashNotifier.notifyStash(StashNotifier.java:556)
          at org.jenkinsci.plugins.stashNotifier.StashNotifier.processJenkinsEvent(StashNotifier.java:207)
          at org.jenkinsci.plugins.stashNotifier.StashNotifier.prebuild(StashNotifier.java:160)
          at hudson.model.AbstractBuild$AbstractBuildExecution.preBuild(AbstractBuild.java:834)
          at hudson.model.AbstractBuild$AbstractBuildExecution.preBuild(AbstractBuild.java:829)
          at hudson.model.Build$BuildExecution.doRun(Build.java:144)
          at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:537)
          at hudson.model.Run.execute(Run.java:1741)
          at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
          at hudson.model.ResourceController.execute(ResourceController.java:98)
          at hudson.model.Executor.run(Executor.java:381)
          Caused by: java.security.InvalidKeyException: ECDH key agreement requires ECPublicKey for doPhase
          at org.bouncycastle.jcajce.provider.asymmetric.ec.KeyAgreementSpi.engineDoPhase(Unknown Source)
          at javax.crypto.KeyAgreement.doPhase(KeyAgreement.java:567)
          at sun.security.ssl.ECDHCrypt.getAgreedSecret(ECDHCrypt.java:96)
          ... 32 more
          New: Currently when connecting to ECDHE-RSA-* servers I was getting errors.

          Upgrading to bouncycastle 1.52 resolves these

          Below is an example exception; seen via the stash-notification-plugin, and ssh-agent-plugin

          {code}
          javax.net.ssl.SSLHandshakeException: Could not generate secret
          at sun.security.ssl.ECDHCrypt.getAgreedSecret(ECDHCrypt.java:99)
          at sun.security.ssl.ClientHandshaker.serverHelloDone(ClientHandshaker.java:1045)
          at sun.security.ssl.ClientHandshaker.processMessage(ClientHandshaker.java:348)
          at sun.security.ssl.Handshaker.processLoop(Handshaker.java:979)
          at sun.security.ssl.Handshaker.process_record(Handshaker.java:914)
          at sun.security.ssl.SSLSocketImpl.readRecord(SSLSocketImpl.java:1050)
          at sun.security.ssl.SSLSocketImpl.performInitialHandshake(SSLSocketImpl.java:1363)
          at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1391)
          at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1375)
          at org.apache.http.conn.ssl.SSLConnectionSocketFactory.createLayeredSocket(SSLConnectionSocketFactory.java:394)
          at org.apache.http.conn.ssl.SSLConnectionSocketFactory.connectSocket(SSLConnectionSocketFactory.java:353)
          at org.apache.http.impl.conn.DefaultHttpClientConnectionOperator.connect(DefaultHttpClientConnectionOperator.java:134)
          at org.apache.http.impl.conn.BasicHttpClientConnectionManager.connect(BasicHttpClientConnectionManager.java:338)
          at org.apache.http.impl.execchain.MainClientExec.establishRoute(MainClientExec.java:380)
          at org.apache.http.impl.execchain.MainClientExec.execute(MainClientExec.java:236)
          at org.apache.http.impl.execchain.ProtocolExec.execute(ProtocolExec.java:184)
          at org.apache.http.impl.execchain.RetryExec.execute(RetryExec.java:88)
          at org.apache.http.impl.execchain.RedirectExec.execute(RedirectExec.java:110)
          at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:184)
          at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:82)
          at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:107)
          at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:55)
          at org.jenkinsci.plugins.stashNotifier.StashNotifier.notifyStash(StashNotifier.java:556)
          at org.jenkinsci.plugins.stashNotifier.StashNotifier.processJenkinsEvent(StashNotifier.java:207)
          at org.jenkinsci.plugins.stashNotifier.StashNotifier.prebuild(StashNotifier.java:160)
          at hudson.model.AbstractBuild$AbstractBuildExecution.preBuild(AbstractBuild.java:834)
          at hudson.model.AbstractBuild$AbstractBuildExecution.preBuild(AbstractBuild.java:829)
          at hudson.model.Build$BuildExecution.doRun(Build.java:144)
          at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:537)
          at hudson.model.Run.execute(Run.java:1741)
          at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
          at hudson.model.ResourceController.execute(ResourceController.java:98)
          at hudson.model.Executor.run(Executor.java:381)
          Caused by: java.security.InvalidKeyException: ECDH key agreement requires ECPublicKey for doPhase
          at org.bouncycastle.jcajce.provider.asymmetric.ec.KeyAgreementSpi.engineDoPhase(Unknown Source)
          at javax.crypto.KeyAgreement.doPhase(KeyAgreement.java:567)
          at sun.security.ssl.ECDHCrypt.getAgreedSecret(ECDHCrypt.java:96)
          ... 32 more
          {code}
          Josh Toft made changes -
          Summary Original: Upgrade ssh-agent-plugin bouncycastle to 1.52 to add ECDH fixes New: Upgrade ssh-agent-plugin bouncycastle to 1.52 to add ECDH support
          Manuel Recena Soto made changes -
          Component/s Original: stashnotifier-plugin [ 17535 ]
          Manuel Recena Soto made changes -
          Assignee New: Josh Toft [ seryl ]
          Manuel Recena Soto made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Josh Toft made changes -
          Summary Original: Upgrade ssh-agent-plugin bouncycastle to 1.52 to add ECDH support New: Upgrade ssh-agent-plugin bouncycastle to 1.52
          Josh Toft made changes -
          Summary Original: Upgrade ssh-agent-plugin bouncycastle to 1.52 New: Upgrade ssh-agent-plugin bouncycastle to 1.53
          Josh Toft made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Closed [ 6 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 165193 ] New: JNJira + In-Review [ 209163 ]

            seryl Josh Toft
            seryl Josh Toft
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: