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

EC2Cloud crashes with NPE when instanceCapStr is not passed from JCasC

XMLWordPrintable

    • ec2-1.44

      It is a minor issue in the EC2 Plugin implementation. 
      instanceCapStr in https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/EC2Cloud.java is designed to be optional, and it supports empty strings in the constructor. But it still fails with NPE when a field is not passed from the JCasC configuration.
       
      I consider it is as an important issue, because the field is not exported by default.

      The issue can be fixed by a small patch to properly handle nulls in the constructor arguments

          [JENKINS-57562] EC2Cloud crashes with NPE when instanceCapStr is not passed from JCasC

          Oleg Nenashev created issue -
          Oleg Nenashev made changes -
          Description Original: It is a minor issue in the EC2 Plugin implementation. 
          instanceCapStr in [https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/EC2Cloud.java] is designed to be optional, and it supports empty strings in the constructor. But it still fails with NPE when a field is not passed from the JCasC configuration.
           
          I consider it is as an important issue, because the field is not exported by default           		New: It is a minor issue in the EC2 Plugin implementation. 
           instanceCapStr in [https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/EC2Cloud.java] is designed to be optional, and it supports empty strings in the constructor. But it still fails with NPE when a field is not passed from the JCasC configuration.
            
           I consider it is as an important issue, because the field is not exported by default.

          The issue can be fixed by a small patch to properly handle nulls in the constructor arguments

          Oleg Nenashev added a comment -

          Duplicates JENKINS-56684

          Oleg Nenashev added a comment - Duplicates  JENKINS-56684
          Oleg Nenashev made changes -
          Link New: This issue duplicates JENKINS-56684 [ JENKINS-56684 ]

          Nisarg Shah added a comment -

          oleg_nenashev

          Is this issue still available to work on? If available, I would like to have a look on it.

          Nisarg Shah added a comment - oleg_nenashev Is this issue still available to work on? If available, I would like to have a look on it.

          Tim Jacomb added a comment -

          I believe fixed in https://github.com/jenkinsci/ec2-plugin/pull/358

          although unreleased

          Tim Jacomb added a comment - I believe fixed in  https://github.com/jenkinsci/ec2-plugin/pull/358 although unreleased
          Tim Jacomb made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Fixed but Unreleased [ 10203 ]

          Nisarg Shah added a comment -

          Okay 

          Nisarg Shah added a comment - Okay 
          Oleg Nenashev made changes -
          Released As New: ec2-1.44
          Status Original: Fixed but Unreleased [ 10203 ] New: Resolved [ 5 ]

            thoulen FABRIZIO MANFREDI
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: