Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rock is using deprecated webhooks for Mailgun #3379

Closed
1 task done
cabal95 opened this issue Oct 26, 2018 · 2 comments
Closed
1 task done

Rock is using deprecated webhooks for Mailgun #3379

cabal95 opened this issue Oct 26, 2018 · 2 comments
Assignees
Labels
Priority: High Affects most production Rock installs in a way that will be noticed. Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Topic: Communications Related to Rock Communications: Email, SMS, MMS, Push Notifications, etc. Type: Bug Confirmed bugs or reports that are very likely to be bugs. x-Fixed in v8.5
Milestone

Comments

@cabal95
Copy link
Member

cabal95 commented Oct 26, 2018

Prerequisites

Description

Apparently Mailgun has deprecated the version of Webhooks that Rock uses. This is causing confusion for end-users. When they try to follow the guide and set the webhook URL it fails because that is now the "new version" of the webhook.

Deprecated version: https://documentation.mailgun.com/en/latest/api-webhooks-deprecated.html#webhooks-deprecated

New version: https://documentation.mailgun.com/en/latest/api-webhooks.html#webhooks

I haven't had a chance to dig into what has actually changed, but trying to use the current web handler for Mailgun with the new Webhook results in a 406 wrong mime type response.

Expected behavior:

Mailgun webhooks should work

Actual behavior:

Mailgun webhooks do not work unless configured for legacy mode (which could be removed in the future).

Versions

  • Rock Version: 8.4+
  • Client Culture Setting: en-US

┆Attachments: mailgun webhooks.png

@cabal95 cabal95 added Type: Bug Confirmed bugs or reports that are very likely to be bugs. Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Priority: High Affects most production Rock installs in a way that will be noticed. Topic: Communications Related to Rock Communications: Email, SMS, MMS, Push Notifications, etc. labels Oct 26, 2018
@nairdo
Copy link
Member

nairdo commented Oct 31, 2018

It looks like we (thanks @mikejed) updated this in the documentation about a month ago, but we'll work out the webhook to handle the new format as well for an upcoming release.

@cabal95
Copy link
Member Author

cabal95 commented Oct 31, 2018

Awesome. The person I was talking to was probably going off an old PDF version of the guide or something. I hadn't checked the docs myself. :( @cabal95--

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: High Affects most production Rock installs in a way that will be noticed. Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Topic: Communications Related to Rock Communications: Email, SMS, MMS, Push Notifications, etc. Type: Bug Confirmed bugs or reports that are very likely to be bugs. x-Fixed in v8.5
Projects
None yet
Development

No branches or pull requests

5 participants