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

Whitelisted signature presets for Java standard APIs and Jenkins core APIs

    XMLWordPrintable

    Details

    • Similar Issues:

      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

            Hide
            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?

            Show
            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?
            Hide
            jglick Jesse Glick added a comment -

            Mike Kobit 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.

            Show
            jglick Jesse Glick added a comment - Mike Kobit 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.
            Hide
            mkobit Mike Kobit added a comment -

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

            Show
            mkobit Mike Kobit added a comment - Awesome, thanks Jesse Glick ! Going to take another pass at it and open a PR.
            Hide
            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.

            Show
            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.
            Hide
            jglick Jesse Glick added a comment -

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

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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: