Welcome to the Linux Foundation Forum!

GitHub Hooks: Add a Hook to GitHub - Webhook address missing a trailing / in the address

Shivinder
Shivinder Posts: 45
edited December 2022 in LFS261 Class Forum

In the section where it is explained about adding a Webhook address to GitHub Webhooks > Add Webhook page, there is a typo in the Address we need to paste in the Payload URL text field of Github Add Webhook page.

The suggested URL over there is http://JenkinsURLHere:8080/github-webhook

The correct URL should have a trailing slash in it. It should be http://JenkinsURLHere:8080/github-webhook/

If we do not add a trailing slash to the address, then we get a HTTP 302 and the browser redirects the request and the build doesn't trigger.

The screenshot of the incorrect URL is

The page link is - https://trainingportal.linuxfoundation.org/learn/course/devops-and-sre-fundamentals-implementing-continuous-delivery-lfs261/setting-up-continuous-integration-with-jenkins-1/continuous-integration-with-jenkins?page=27

I would like to add the Stack Overflow link here which actually helped me fix my issue - https://stackoverflow.com/questions/49848884/github-webhook-with-jenkins-return-302-found

Comments

  • I was just browsing the GitHub plugin documentation, and can see that the official documentation also states to add the trailing slash in the address. I am pasting the screenshot here.

  • Tagging for attention Flavia. @fcioanca

  • fcioanca
    fcioanca Posts: 1,886

    Hi @Shivinder, thank you for flagging this. I am awaiting confirmation on the change from the course author, and once I have it I will update it. There is no need to tag me, as I get notifications on incoming posts.

  • @fcioanca said:
    Hi @Shivinder, thank you for flagging this. I am awaiting confirmation on the change from the course author, and once I have it I will update it. There is no need to tag me, as I get notifications on incoming posts.

    Thank you Flavia. I wasn’t aware of you being notified about these forum posts.

  • fcioanca
    fcioanca Posts: 1,886

    @Shivinder The trailing slash has been added. Thank you for your help!

Categories

Upcoming Training