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

Node parameter value can't be passed by HTTP GET

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      When passing param via GET request like:
      http://localhost:8080/jenkins/job/Local/buildWithParameters?NODENAME=node1
      jenkins create default node parameter variable.

      I think that the cause is in NodeParameterDefinition class line 77:

      public ParameterValue createValue(String value)

      { return new NodeParameterValue(getName(), value, getDescription()); }

      See

      public NodeParameterValue(String name, String description, String label)

      { super(name, description, label); }

      Value and description need to be switched.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          domi Dominik Bartholdi added a comment -

          great catch, thank you!!!

          Show
          domi Dominik Bartholdi added a comment - great catch, thank you!!!
          Hide
          Permalink
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: imod
          Path:
          src/main/java/org/jvnet/jenkins/plugins/nodelabelparameter/NodeParameterDefinition.java
          http://jenkins-ci.org/commit/nodelabelparameter-plugin/4181f676118f9ec7727ba69ac5227f93210d965f
          Log:
          [FIXED JENKINS-14230] Node parameter value can't be passed by HTTP GET

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: imod Path: src/main/java/org/jvnet/jenkins/plugins/nodelabelparameter/NodeParameterDefinition.java http://jenkins-ci.org/commit/nodelabelparameter-plugin/4181f676118f9ec7727ba69ac5227f93210d965f Log: [FIXED JENKINS-14230] Node parameter value can't be passed by HTTP GET
          Hide
          Permalink
          yury_mikhailenko Yury Mikhailenko added a comment -

          Thanks for quick update

          Show
          yury_mikhailenko Yury Mikhailenko added a comment - Thanks for quick update

            People

            Assignee:
            domi Dominik Bartholdi
            Reporter:
            yury_mikhailenko Yury Mikhailenko
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: