• Icon: Task Task
    • Resolution: Fixed
    • Icon: Major Major
    • github-api-plugin
    • None
    • 1.321-478.vc9ce627ce001

      When a Jetty 12 EE 9 core (i.e., one built from the jakarta branch of jenkins) and Jetty 12 EE 9 compatible test harness (i.e., one built from the forward branch of jenkins-test-harness) are used, the following test fails:

      • jenkins.plugins.github.api.SmokeTest

      This is just a trivial matter of changing the ServletContextHandler import to org.eclipse.jetty.ee9.servlet.ServletContextHandler. Unfortunately, this is not backward compatible with Jetty 10 or Jetty 12 EE 8, so the PR will have to remain in draft until we release Jetty 12 EE 9 support. Still, we should prepare the PR in advance.

          [JENKINS-73312] Adapt GitHub API for Jetty 12 (EE 9)

          Basil Crow created issue -
          Basil Crow made changes -
          Link New: This issue depends on JENKINS-73132 [ JENKINS-73132 ]
          Basil Crow made changes -
          Link New: This issue is blocking JENKINS-73283 [ JENKINS-73283 ]
          Basil Crow made changes -
          Link New: This issue depends on JENKINS-73278 [ JENKINS-73278 ]
          Basil Crow made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Basil Crow made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Basil Crow made changes -
          Remote Link New: This issue links to "jenkinsci/github-api-plugin#243 (Web Link)" [ 29706 ]
          Basil Crow made changes -
          Assignee New: Basil Crow [ basil ]

          Basil Crow added a comment -

          Basil Crow added a comment - Fixed in https://github.com/jenkinsci/github-api-plugin/pull/243 . Released in 1.321-478.vc9ce627ce001 .
          Basil Crow made changes -
          Released As New: 1.321-478.vc9ce627ce001
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Closed [ 6 ]

            basil Basil Crow
            basil Basil Crow
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: