Skip to content

[JENKINS-16337] SSH public key field in the user config page lacks form validation #13212

Closed

Description

@jenkins-infra-bot

It is a common mistake to type the SSH key in multiple lines. Look for these typical errors and make sure the keys follow the valid syntax.


Originally reported by kohsuke, imported from: SSH public key field in the user config page lacks form validation
  • assignee: jglick
  • status: Resolved
  • priority: Major
  • component(s): core
  • label(s): diagnostics, ssh, ux
  • resolution: Fixed
  • resolved: 2017-04-12T20:41:01+00:00
  • votes: 1
  • watchers: 4
  • imported: 2025-11-24
Raw content of original issue

It is a common mistake to type the SSH key in multiple lines. Look for these typical errors and make sure the keys follow the valid syntax.

Activity

  1. jenkins-infra-bot commented on Jan 14, 2013

    @jenkins-infra-bot
    Author
  2. jenkins-infra-bot commented on Apr 7, 2017

    @jenkins-infra-bot
    Author
    jglick:
    • Original comment link
    • Raw content of original comment:

      This is in the core (specifically the ssh-cli-auth module), not the ssh plugin.

      AFAICT it does no parsing whatsoever of what you enter. It just splits it by newline, and then when authenticating, generates an expected public key string and does a String.equals against each line in turn.

    This is in the core (specifically the ssh-cli-auth module), not the ssh plugin.

    AFAICT it does no parsing whatsoever of what you enter. It just splits it by newline, and then when authenticating, generates an expected public key string and does a String.equals against each line in turn.

  3. jenkins-infra-bot commented on Apr 11, 2017

    @jenkins-infra-bot
    Author
    scm_issue_link:

    Code changed in jenkins
    User: Jesse Glick
    Path:
    src/main/java/org/jenkinsci/main/modules/cli/auth/ssh/UserPropertyImpl.java
    src/test/java/org/jenkinsci/main/modules/cli/auth/ssh/TheTest.java
    http://jenkins-ci.org/commit/ssh-cli-auth-module/da9030294137c89b5f213a87c4ce081e9c80fac5
    Log:
    [FIXED JENKINS-16337] Provide basic syntactic validation of public keys.
    Does not try to actually decode them.

  4. jenkins-infra-bot commented on Apr 12, 2017

    @jenkins-infra-bot
    Author
    scm_issue_link:

    Code changed in jenkins
    User: Jesse Glick
    Path:
    war/pom.xml
    http://jenkins-ci.org/commit/jenkins/24df43fa448efae530da9ed8d816a1ba3541bbcb
    Log:
    Merge pull request #2840 from jglick/ssh-cli-auth-JENKINS-16337

    JENKINS-16337 ssh-cli-auth 1.4

    Compare: 1c67022...24df43f

  5. jenkins-infra-bot commented on Apr 15, 2017

    @jenkins-infra-bot
    Author
    oleg_nenashev:
    • Original comment link
    • Raw content of original comment:

      Integrated towards 2.55

    Integrated towards 2.55

  6. removed
    bugFor changelog: Minor bug. Will be listed after features
    on Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions