• Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Minor Minor
    • None
    • pipeline-githubnotify-step-plugin 1.0.0
      jenkins 2.36

      The inference for the github credentials uses getCredentials which return checkout credentials.

      Checkout credentials can be SSH credentials, unsuitable for API calls such as commit status updates.

      When checkout credentials are SSH credentials, inference fails with: java.lang.IllegalArgumentException: The credentialsId does not seem to exist, please check it

      Instead, the plugin should probably use getScanCredentialsId to obtain its credentials.

          [JENKINS-40422] Inferring credentials resolves wrong credential

          Martin Ringehahn created issue -
          Martin Ringehahn made changes -
          Description Original: The inference for the github credentials uses `getCredentials` which return checkout credentials.

          Checkout credentials can be SSH credentials, unsuitable for API calls such as commit status updates.

          When checkout credentials are SSH credentials, inference fails with: {{java.lang.IllegalArgumentException: The credentialsId does not seem to exist, please check it}}

          Instead, the plugin should probably use {{getScanCredentialsId}} to obtain its credentials.
          New: The inference for the github credentials uses {{getCredentials}} which return checkout credentials.

          Checkout credentials can be SSH credentials, unsuitable for API calls such as commit status updates.

          When checkout credentials are SSH credentials, inference fails with: {{java.lang.IllegalArgumentException: The credentialsId does not seem to exist, please check it}}

          Instead, the plugin should probably use {{getScanCredentialsId}} to obtain its credentials.
          Raul Arabaolaza made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Raul Arabaolaza made changes -
          Remote Link New: This issue links to "PR-2 (Web Link)" [ 15148 ]
          Raul Arabaolaza made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          liyatikal made changes -
          Resolution Original: Fixed [ 1 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]

            rarabaolaza Raul Arabaolaza
            chrono Martin Ringehahn
            Votes:
            2 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated: