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

Upgrade Jetty to fix bugs in cookies

XMLWordPrintable

      Jetty is causing bugs in junit and analysis graphs because of cookies for paths with whitespaces. See JENKINS-25917

      Jenkins uses Jetty 8.1.13 (https://github.com/jenkinsci/winstone/blob/master/pom.xml#L221)
      which is buggy (see here and here)

      It is needed to upgrade Jetty to 9.0.6 or later.

          [JENKINS-32377] Upgrade Jetty to fix bugs in cookies

          evernat created issue -
          evernat made changes -
          Link New: This issue is blocking JENKINS-25917 [ JENKINS-25917 ]
          evernat made changes -
          Assignee Original: Ulli Hafner [ drulli ]
          Ulli Hafner made changes -
          Link New: This issue depends on JENKINS-23378 [ JENKINS-23378 ]

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/org/jenkinsci/test/acceptance/plugins/analysis_core/GraphConfigurationView.java
          src/test/java/plugins/AbstractAnalysisTest.java
          http://jenkins-ci.org/commit/acceptance-test-harness/b3691320192217d8a2d5e7e60e65119f3b13ea7f
          Log:
          Added a test case that checks trend configuration via cookie.

          Basically a test the checks that JENKINS-25917 and JENKINS-32377 are resolved in Jenkins 2.1.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/org/jenkinsci/test/acceptance/plugins/analysis_core/GraphConfigurationView.java src/test/java/plugins/AbstractAnalysisTest.java http://jenkins-ci.org/commit/acceptance-test-harness/b3691320192217d8a2d5e7e60e65119f3b13ea7f Log: Added a test case that checks trend configuration via cookie. Basically a test the checks that JENKINS-25917 and JENKINS-32377 are resolved in Jenkins 2.1.

          Ulli Hafner added a comment -

          Fixed in Jenkins 2.x.

          Ulli Hafner added a comment - Fixed in Jenkins 2.x.
          Ulli Hafner made changes -
          Assignee New: Kohsuke Kawaguchi [ ci_jenkinsci_org ]
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/test/java/plugins/AbstractAnalysisTest.java
          http://jenkins-ci.org/commit/acceptance-test-harness/75c8f774c6474f5bfb6a67b256b4acff57a977fc
          Log:
          JENKINS-25917 JENKINS-32377 Let cookie test case run with 2.x only.

          Actually use a job name in the cookie test case that contains spaces.

          Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/00068c6c59b0...75c8f774c647

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/test/java/plugins/AbstractAnalysisTest.java http://jenkins-ci.org/commit/acceptance-test-harness/75c8f774c6474f5bfb6a67b256b4acff57a977fc Log: JENKINS-25917 JENKINS-32377 Let cookie test case run with 2.x only. Actually use a job name in the cookie test case that contains spaces. Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/00068c6c59b0...75c8f774c647
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 167990 ] New: JNJira + In-Review [ 198324 ]

            ci_jenkinsci_org Kohsuke Kawaguchi
            evernat evernat
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: