allegroplanet / allegro-planet

1 stars 1 forks source link

Allow Webhook Access #56

Closed MarkOates closed 7 years ago

MarkOates commented 7 years ago

GitHub Webhooks currently hit Allegro Planet and Planet responds with a 422. This is due to a requirement on rails that controller actions can only be called with an "authenticity token". Since no token exists when a webhook hits, rails responds with an error.

Also, GitHub Webooks post a pretty large payload. This means the length validation of a maximum length of 512 will not work with this property. Also, rather than showing the payloads with each item on the index page, it's better to move them to a show page and reduce the footprint on the index page.

related: https://github.com/allegroplanet/allegro-planet/pull/55