• Icon: Task Task
    • Resolution: Fixed
    • Icon: Major Major
    • github-api-plugin
    • None

      When a Jetty 12 Jenkins test harness is used (i.e., one built from the prototype branch of jenkins-test-harness), the following test fails:

      • jenkins.plugins.github.api.SmokeTest

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

          [JENKINS-73132] Adapt GitHub API for Jetty 12 (EE 8)

          Basil Crow created issue -
          Basil Crow made changes -
          Link New: This issue is blocking JENKINS-73282 [ JENKINS-73282 ]
          Basil Crow made changes -
          Link New: This issue is blocking JENKINS-73312 [ JENKINS-73312 ]
          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#242 (Web Link)" [ 29701 ]
          Basil Crow made changes -
          Link New: This issue depends on JENKINS-73130 [ JENKINS-73130 ]
          Vishal made changes -
          Assignee New: Vishal [ vwagh ]
          Vishal made changes -
          Remote Link New: This issue links to "PR (Web Link)" [ 29914 ]
          Vishal made changes -
          Remote Link Original: This issue links to "PR (Web Link)" [ 29914 ]
          Vishal made changes -
          Remote Link New: This issue links to "jenkinsci/github-api-plugin#249 (Web Link)" [ 29915 ]

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

              Created:
              Updated:
              Resolved: