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

Trailing slash shouldn't be required for webhook url

    • Icon: Bug Bug
    • Resolution: Not A Defect
    • Icon: Major Major
    • core
    • github v1.5, Jenkins ver. 1.505

      This wasn't manifesting in the browser, as my browser was automatically prefixing the url with a slash, but I finally confirmed it with plain ol curl

      `curl http://ci.example.com/github-webhook/`

      • result: same error about payload that you see in the browser

      `curl curl http://ci.example.com/github-webhook`

      • result: silence. no output.

      I spent like 2 hours trying to figure this out, but mostly because I didn't realize that the github service "Update settings" button needed to be pushed before "Test Hook" would work. But that's an issue for github

          [JENKINS-17185] Trailing slash shouldn't be required for webhook url

          Patrick Connolly created issue -
          Patrick Connolly made changes -
          Description Original: This wasn't manifesting in the browser, as my browser was automatically prefixing the url with a slash, but I finally confirmed it with plain ol curl

          `curl http://ci.lassonde.mpdagile.com/github-webhook/`
          - result: same error about payload that you see in the browser

          `curl curl http://ci.lassonde.mpdagile.com/github-webhook`
          - result: silence. no output.

          I spent like 2 hours trying to figure this out, but mostly because I didn't realize that the github service "Update settings" button needed to be pushed before "Test Hook" would work. But that's an issue for github :)
          New: This wasn't manifesting in the browser, as my browser was automatically prefixing the url with a slash, but I finally confirmed it with plain ol curl

          `curl http://ci.example.com/github-webhook/`
          - result: same error about payload that you see in the browser

          `curl curl http://ci.example.com/github-webhook`
          - result: silence. no output.

          I spent like 2 hours trying to figure this out, but mostly because I didn't realize that the github service "Update settings" button needed to be pushed before "Test Hook" would work. But that's an issue for github :)
          James Talmage made changes -
          Component/s New: core [ 15593 ]
          Kirill Merkushev made changes -
          Component/s Original: github-plugin [ 15896 ]
          Kirill Merkushev made changes -
          Labels New: stapler
          Daniel Beck made changes -
          Resolution New: Not A Defect [ 7 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 148017 ] New: JNJira + In-Review [ 192682 ]

            Unassigned Unassigned
            patcon Patrick Connolly
            Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: