Code Monkey home page Code Monkey logo

Comments (8)

moorereason avatar moorereason commented on May 2, 2024

It looks like it's working as intended. You're asking webhook to match a URL parameter that either doesn't exist or is invalid. The command should not be triggered in either case.

from webhook.

fbartels avatar fbartels commented on May 2, 2024

Yes, that was not what I meant. The hook does not execute when the wrong token is sent, but webhook also does tell you how the token should be. A not so secret secret if you will.

In other words, you can't really use payload-hash-sha1 for authentication.

from webhook.

moorereason avatar moorereason commented on May 2, 2024

@fbartels, webhook uses your secret to calculate a HMAC SHA1 of the payload. It's not showing you the secret. If this was unclear, maybe you can suggest an improvement to the documentation or help us to make it more clear to new users.

from webhook.

fbartels avatar fbartels commented on May 2, 2024

No, but its showing you the sha1. And when you call ?token=sha1-of-token, then your call will succeed.

from webhook.

moorereason avatar moorereason commented on May 2, 2024

Unless I'm mistaken, it's not showing the sha1 to the remote agent. It's only logging it on the server side.

from webhook.

adnanh avatar adnanh commented on May 2, 2024

You are correct. It leaks out the calculated SHA1 of the payload that it's expecting, this is a bug, no idea how this slipped out unnoticed!

from webhook.

fbartels avatar fbartels commented on May 2, 2024

Hi Adnan,

thanks for the fix. I did a manual checkout and build of 2.4.0 and can confirm that I cannot reproduce the issue anymore.

from webhook.

adnanh avatar adnanh commented on May 2, 2024

👍 thanks for reporting:)

from webhook.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.