Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-55136

Log an error when ssh private key is empty

    XMLWordPrintable

Details

    • Improvement
    • Status: Open (View Workflow)
    • Minor
    • Resolution: Unresolved
    • ssh-credentials-plugin
    • None

    Description

      This is fallout from the fix for SECURITY-440 (see https://jenkins.io/security/advisory/2018-06-25/   )

      We upgraded to ensure that fix was installed back in June.
      For some reason it did not affect us until after upgrading to ssh-slaves 1.29.1 (from 1.28)

      We used to have the SSH private key that the master uses to log into the slaves, in $JENKINS_HOME/.ssh/jenkins - ie. not one of the names that jenkins searches for by default.
      When the master tries to connect to the client, it fails and there's no clue given as to why.

      This is the log I get when I try to reconnect to a node affected by this issue:
      ```
      SSHLauncher{host='fiasco', port=22, credentialsId='<elided>', jvmOptions='', javaPath='/usr/lib/jvm/java-8-openjdk-amd64/bin/java', prefixStartSlaveCmd='', suffixStartSlaveCmd='', launchTimeoutSeconds=210, maxNumRetries=10, retryWaitTime=15, sshHostKeyVerificationStrategy=hudson.plugins.sshslaves.verifiers.KnownHostsFileKeyVerificationStrategy, tcpNoDelay=true, trackCredentials=true}
      [12/11/18 10:33:49] [SSH] Opening SSH connection to fiasco:22.
      [12/11/18 10:33:49] [SSH] SSH host key matches key in Known Hosts file. Connection will be allowed.
      [12/11/18 10:33:49] [SSH] Authentication failed.
      Authentication failed.
      [12/11/18 10:33:49] Launch failed - cleaning up connection
      [12/11/18 10:33:49] [SSH] Connection closed.

      ```
      Similarly jenkins.log is silent about what the actual problem is:
      ```
      Dec 12, 2018 4:05:02 PM hudson.slaves.SlaveComputer tryReconnect
      INFO: Attempting to reconnect jessie-fiasco
      ```

      This issue has appeared before, in  https://issues.jenkins-ci.org/browse/JENKINS-24613 and it was claimed there that it has been documented. Can you please check that it actually was documented and point to where that documentation is? All I have been able to find is a reference in 24613 to a comment in the source code (https://github.com/jenkinsci/ssh-credentials-plugin/blob/master/src/main/java/com/cloudbees/jenkins/plugins/sshcredentials/impl/BasicSSHUserPrivateKey.java#L516). The list of accepted filenames is at line 484.

      I'm surprised there was no Big Fat Warning from the plugin about the change to requiring the private key to be pasted into the UI - at least I don't recall one.

      My request:
      Please log a warning in /var/log/jenkins/jenkins.log if the length of the private key in a SSH credential is less than 1 byte or so. That would give operators something to go on.

      Ideally there would be a warning in the UI of the credentials plugin.
      Another option would be a note on the page where you are required to enter the private key that files on disk are no longer accepted, or if they are, what the list of accepted filenames is.

       

      Attachments

        Activity

          xipmox Vince Murphy created issue -
          dnusbaum Devin Nusbaum made changes -
          Field Original Value New Value
          Assignee Devin Nusbaum [ dnusbaum ]

          People

            Unassigned Unassigned
            xipmox Vince Murphy
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: