Hookshot - Generic webhooks will be routed to /webhook #1681
Comments
|
I'll work on this |
HarHarLinks
added a commit
to HarHarLinks/matrix-docker-ansible-deploy
that referenced
this issue
Mar 31, 2022
spantaleev
added a commit
that referenced
this issue
Jun 25, 2024
By appending `/webhook` to the public URL (becoming `/hookshot/webhooks/webhook`) and by only stripping the `/hookshot/webhooks` prefix, we're effectively following what newer Hookshot versions advise (see #1681). This change appears to be backward-compatible (old webhook URLs like `/hookshot/webhooks/:hookId` still work), until Hookshot behavior changes.
|
Fixed by 4704a60 |
venimus
pushed a commit
to superhero-com/matrix-docker-ansible-deploy
that referenced
this issue
Jul 3, 2024
By appending `/webhook` to the public URL (becoming `/hookshot/webhooks/webhook`) and by only stripping the `/hookshot/webhooks` prefix, we're effectively following what newer Hookshot versions advise (see spantaleev#1681). This change appears to be backward-compatible (old webhook URLs like `/hookshot/webhooks/:hookId` still work), until Hookshot behavior changes.
venimus
pushed a commit
to superhero-com/matrix-docker-ansible-deploy
that referenced
this issue
Jul 3, 2024
Related to 4704a60 Related to spantaleev#1681
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently hookshot generic webhooks get routed to
/:hookIdon the webhooks listener, but matrix-org/matrix-hookshot#227 is chaging things so we now prefer them to be routed to/webhook/:hookId. While previous setups still work, it's deprecated behavior.The text was updated successfully, but these errors were encountered: