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

Webhook failures after upgrading jetty to 9.4.39.v20210325 in 2.277.3

    • 2.290, 2.289.2

      After upgrading Jenkins from 2.249.2 to 2.277.3, webhooks being fired from github enterprise by way of the ghrpb started randomly failing with errors indicating that the payload was missing from the request. Messages were similar to below:

      SEVERE: Request doesn't contain payload. You are sending url encoded request, so you should pass github payload through 'payload' request parameter

      We saw similarly worded errors coming from the Github plugin as well.

      I believe I've traced the failure to this issue in Jetty which looks like it's been corrected with this PR

      The underlying error was exposed after enabling Jenkins logging on jetty:

       

      May 12, 2021 11:34:33 AM FINE org.eclipse.jetty.server.HttpConnection fillRequestBuffer
      javax.net.ssl.SSLHandshakeException: Encrypted buffer max length exceeded

      Which further down in the stack indicated it was bubbling up from ghprb:

      org.jenkinsci.plugins.ghprb.GhprbRootAction.extractRequestBody(GhprbRootAction.java:286)

       

          [JENKINS-65624] Webhook failures after upgrading jetty to 9.4.39.v20210325 in 2.277.3

          wayne goyer created issue -
          wayne goyer made changes -
          Description Original: After upgrading Jenkins from 2.249.2 to 2.277.3, webhooks being fired from github enterprise by way of the ghrpb started randomly failing with errors indicating that the payload was missing from the request. Messages were similar to below:

           
          {code:java}
          SEVERE: Request doesn't contain payload. You are sending url encoded request, so you should pass github payload through 'payload' request parameter{code}
           

          We saw similarly worded errors coming from the Github plugin as well.

          I believe I've traced the failure to [this issue|https://github.com/eclipse/jetty.project/issues/6121] in Jetty which looks like it's been corrected with [this PR|https://github.com/eclipse/jetty.project/pull/6142]

          The underlying error was exposed after enabling Jenkins logging on jetty:

           
          {noformat}
          May 12, 2021 11:34:33 AM FINE org.eclipse.jetty.server.HttpConnection fillRequestBuffer
          javax.net.ssl.SSLHandshakeException: Encrypted buffer max length exceeded{noformat}
          Which further down in the stack indicated it was bubbling up from ghprb:
          {noformat}
          org.jenkinsci.plugins.ghprb.GhprbRootAction.extractRequestBody(GhprbRootAction.java:286){noformat}
           
          New: After upgrading Jenkins from 2.249.2 to 2.277.3, webhooks being fired from github enterprise by way of the ghrpb started randomly failing with errors indicating that the payload was missing from the request. Messages were similar to below:
          {noformat}
          SEVERE: Request doesn't contain payload. You are sending url encoded request, so you should pass github payload through 'payload' request parameter{noformat}
          We saw similarly worded errors coming from the Github plugin as well.

          I believe I've traced the failure to [this issue|https://github.com/eclipse/jetty.project/issues/6121] in Jetty which looks like it's been corrected with [this PR|https://github.com/eclipse/jetty.project/pull/6142]

          The underlying error was exposed after enabling Jenkins logging on jetty:

           
          {noformat}
          May 12, 2021 11:34:33 AM FINE org.eclipse.jetty.server.HttpConnection fillRequestBuffer
          javax.net.ssl.SSLHandshakeException: Encrypted buffer max length exceeded{noformat}
          Which further down in the stack indicated it was bubbling up from ghprb:
          {noformat}
          org.jenkinsci.plugins.ghprb.GhprbRootAction.extractRequestBody(GhprbRootAction.java:286){noformat}
           
          Oleg Nenashev made changes -
          Component/s New: winstone-jetty [ 20645 ]
          Oleg Nenashev made changes -
          Labels Original: webhook winstone New: regression webhook winstone
          Oleg Nenashev made changes -
          Assignee New: Olivier Lamy [ olamy ]
          Tim Jacomb made changes -
          Labels Original: regression webhook winstone New: lts-candidate regression webhook winstone
          Tim Jacomb made changes -
          Released As New: 2.290
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Fixed but Unreleased [ 10203 ]
          Tim Jacomb made changes -
          Status Original: Fixed but Unreleased [ 10203 ] New: Resolved [ 5 ]
          Tim Jacomb made changes -
          Labels Original: lts-candidate regression webhook winstone New: 2.289.2-fixed lts-candidate regression webhook winstone
          Tim Jacomb made changes -
          Labels Original: 2.289.2-fixed lts-candidate regression webhook winstone New: 2.289.2-fixed regression webhook winstone
          Baptiste Mathus made changes -
          Remote Link New: This issue links to "PR (Web Link)" [ 26781 ]

            olamy Olivier Lamy
            _wayne wayne goyer
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: