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

Whitelisted signature presets for Java standard APIs and Jenkins core APIs

    XMLWordPrintable

Details

    Description

      Administrators often approve signatures for fundamental methods.

      • Ones in Java APIs like String.parseInt
      • Ones in Jenkins core like Result#isBetterThan

      It's not useful that administrators have to approve all those methods.
      And plugins would be preferred to provide methods returning primitive types.

      I want whitelisted signature presets.

      Attachments

        Issue Links

          Activity

            mkobit Mike Kobit added a comment -

            I was looking to add a bunch of Java 8 java.time but ran into test failures because the plugin seems to only support Java 7.

            > java.lang.AssertionError: method java.time.Instant adjustInto java.time.temporal.Temporal does not exist (or is an override)

            Any thoughts on how to support these?

            mkobit Mike Kobit added a comment - I was looking to add a bunch of Java 8 java.time but ran into test failures because the plugin seems to only support Java 7. > java.lang.AssertionError: method java.time.Instant adjustInto java.time.temporal.Temporal does not exist (or is an override) Any thoughts on how to support these?
            jglick Jesse Glick added a comment -

            mkobit if you are referring to the https://jenkins.ci.cloudbees.com/job/plugins/job/script-security-plugin/ CI job, this was misconfigured, now fixed to run on JDK 8.

            jglick Jesse Glick added a comment - mkobit if you are referring to the https://jenkins.ci.cloudbees.com/job/plugins/job/script-security-plugin/  CI job, this was misconfigured, now fixed to run on JDK 8.
            mkobit Mike Kobit added a comment -

            Awesome, thanks jglick! Going to take another pass at it and open a PR.

            mkobit Mike Kobit added a comment - Awesome, thanks jglick ! Going to take another pass at it and open a PR.
            abayer Andrew Bayer added a comment -

            Closing this as won't fix - if you've got additional signatures you want added, open individual tickets for them.

            abayer Andrew Bayer added a comment - Closing this as won't fix - if you've got additional signatures you want added, open individual tickets for them.
            jglick Jesse Glick added a comment -

            abayer FWIW the idea was to proactively search for safe-looking methods in commonly used packages, rather than wait for failures to trickle in.

            jglick Jesse Glick added a comment - abayer FWIW the idea was to proactively search for safe-looking methods in commonly used packages, rather than wait for failures to trickle in.

            People

              Unassigned Unassigned
              ikedam ikedam
              Votes:
              11 Vote for this issue
              Watchers:
              16 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: