Skip to content

Commit

Permalink
Make type param optional on webhook route
Browse files Browse the repository at this point in the history
Type param is not sent on CLI initiated webhooks

We read the type from the webhook header in all cases
  • Loading branch information
lizkenyon committed Feb 5, 2024
1 parent fea9240 commit 70b3390
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
Unreleased
----------
* Make type param for webhooks route optional. This will fix a bug with CLI initiated webhooks.

21.9.0 (January 16, 2023)
----------
Expand Down
2 changes: 1 addition & 1 deletion config/routes.rb
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,6 @@
end

namespace :webhooks do
post ":type" => :receive
post "(:type)" => :receive
end
end

0 comments on commit 70b3390

Please sign in to comment.