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

EC2Cloud crashes with NPE when instanceCapStr is not passed from JCasC

    XMLWordPrintable

Details

    • ec2-1.44

    Description

      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

      Attachments

        Issue Links

          duplicates

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-56684 Config as Code Nullpointer when instanceCapStr is not initialized

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            Ascending order - Click to sort in descending order
            oleg_nenashev Oleg Nenashev added a comment -

            Duplicates JENKINS-56684

            oleg_nenashev Oleg Nenashev added a comment - Duplicates  JENKINS-56684
            nisarg14 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.

            nisarg14 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.
            timja Tim Jacomb added a comment -

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

            although unreleased

            timja Tim Jacomb added a comment - I believe fixed in  https://github.com/jenkinsci/ec2-plugin/pull/358 although unreleased
            nisarg14 Nisarg Shah added a comment -

            Okay 

            nisarg14 Nisarg Shah added a comment - Okay 

            People

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

              Dates

                Created:
                Updated:
                Resolved: