• Icon: Task Task
    • Resolution: Fixed
    • Icon: Major Major
    • gitlab-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:

      • com.dabsquared.gitlabjenkins.connection.GitLabConnectionConfigSSLTest

      This is just a matter of changing the API usage from org.eclipse.jetty.server.handler.AbstractHandler to org.eclipse.jetty.server.Handler.Abstract, as also done in two tests on the prototype branch of Jenkins core (which can be used as inspiration here). 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-73135] Adapt GitLab 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-73283 [ JENKINS-73283 ]
          Basil Crow made changes -
          Link New: This issue is blocking JENKINS-73344 [ JENKINS-73344 ]
          Basil Crow made changes -
          Link New: This issue depends on JENKINS-73130 [ JENKINS-73130 ]
          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/gitlab-plugin#1664 (Web Link)" [ 29704 ]
          Mark Waite made changes -
          Assignee New: Andra Maria Puscas [ ampuscas ]
          Mark Waite made changes -
          Remote Link New: This issue links to "PR-1678 Adapt GitLab test for Jetty 12 EE 8 (Web Link)" [ 29918 ]

            ampuscas Andra Maria Puscas
            basil Basil Crow
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: