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

[PCT] check script-security latest (jenkins-test-harness - Using JDK internal classes )

    XMLWordPrintable

    Details

    • Similar Issues:
    • Released As:
      script-security 1.55

      Description

      When trying to validate script-security plugin with PCT for Java 11, I got test failures with the following output

      java.lang.reflect.InaccessibleObjectException: Unable to make field private static final java.lang.module.Configuration java.lang.module.Configuration.EMPTY_CONFIGURATION accessible: module java.base does not "opens java.lang.module" to unnamed module @5ebd56e9 
          at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:340) 
          at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:280) 
          at java.base/java.lang.reflect.Field.checkCanSetAccessible(Field.java:176) 
          at java.base/java.lang.reflect.Field.setAccessible(Field.java:170) 
          at org.netbeans.insane.impl.InsaneEngine.processClass(InsaneEngine.java:238) 
          at org.netbeans.insane.impl.InsaneEngine.process(InsaneEngine.java:199) 
          at org.netbeans.insane.impl.InsaneEngine.traverse(InsaneEngine.java:97) 
          at org.netbeans.insane.impl.LiveEngine.traceImpl(LiveEngine.java:188) 
          at org.netbeans.insane.impl.LiveEngine.trace(LiveEngine.java:166) 
          at org.jvnet.hudson.test.MemoryAssert.fromRoots(MemoryAssert.java:292) 
          at org.jvnet.hudson.test.MemoryAssert.assertGC(MemoryAssert.java:178) 
          at org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.GroovyMemoryLeakTest.loaderReleased(GroovyMemoryLeakTest.java:62) 
          at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
          at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
          at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
          at java.base/java.lang.reflect.Method.invoke(Method.java:566) 
          at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
          at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) 
          at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
          at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
          at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
          at org.junit.rules.ExternalResource$1.evaluate(ExternalResource.java:48)
          at org.jvnet.hudson.test.JenkinsRule$1.evaluate(JenkinsRule.java:548)
          at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:298)
          at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:292)
          at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
          at java.base/java.lang.Thread.run(Thread.java:834)
      

      Acceptance criteria

      • re-run PCT to see if still current
      • if not, \o/. If still an issue, either fix it if very simple, or file a clearer JIRA with what needs fixing

        Attachments

          Issue Links

            Activity

            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            The fix has been integrated into the master branch in https://github.com/jenkinsci/script-security-plugin/commit/0b24dde6270a59f4e5782cfacc10c681c9e50912 , but it has not been released yet

            Show
            oleg_nenashev Oleg Nenashev added a comment - The fix has been integrated into the master branch in https://github.com/jenkinsci/script-security-plugin/commit/0b24dde6270a59f4e5782cfacc10c681c9e50912  , but it has not been released yet
            Hide
            vilacides Isa Vilacides added a comment -

            Devin Nusbaum Andrew Bayer are you taking it from here to get it released?

            Show
            vilacides Isa Vilacides added a comment - Devin Nusbaum Andrew Bayer are you taking it from here to get it released?
            Hide
            dnusbaum Devin Nusbaum added a comment -

            I just released script-security 1.55 with the fix for the PCT issues seen on Java 11.

            Show
            dnusbaum Devin Nusbaum added a comment - I just released script-security 1.55 with the fix for the PCT issues seen on Java 11.
            Hide
            vivek Vivek Pandey added a comment -

            Awesome, thanks Devin Nusbaum!

            Show
            vivek Vivek Pandey added a comment - Awesome, thanks Devin Nusbaum !
            Hide
            batmat Baptiste Mathus added a comment -

            Thanks a lot everyone!

            Show
            batmat Baptiste Mathus added a comment - Thanks a lot everyone!

              People

              • Assignee:
                alecharp Adrien Lecharpentier
                Reporter:
                alecharp Adrien Lecharpentier
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: