GetStoredMessage fails with "Please use URLs as they are given in events/webhooks" about mailgun-go HOT 9 CLOSED

I see the problem - do you/we know if the domain used for stored messages is always the same?

This seems very odd - the tests run - and I can't see anything strange in those.

from mailgun-go.

I do not know if it is always the same... though the documentation for messages says (https://documentation.mailgun.com/api-sending.html#retrieving-stored-messages):

To retrieve an inbound message that has been stored via the store() action, use the URL found in the stored event (which you can find through the Events API, or in the notify webhook set when creating the store action (store(notify="http://mydomain.com/callback")).

...which would seem to suggest that one should use the URL as given, I think?

from mailgun-go.

Given the documentation quoted above, would it be preferrable to have the StoredMessage methods take the url value directly instead of the key value? Trying that in a local copy of the mailgun-go repo where I just bypass the generateStoredMessageUrl function seems to work fine.

Or are there cases where you would have the message key but not its url, and/or is that too much of a change in expectations for the argument for those methods?

from mailgun-go.

I have the same problem, since stored message are not always on the main API domain, this wrapper is not able to fetch stored messages. I had to create a custom function to get the stored message by its url.

from mailgun-go.

@luxifer is that something we can get into the library?

from mailgun-go.

I'll see. Not as is, but it is possible

Le 4 sept. 2016 5:55 PM, "Michael Banzon" [email protected] a
écrit :

@luxifer https://github.com/luxifer is that something we can get into
the library?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#64 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZlBnUpoQ0Dqz1byRyz6Q26zoNFwE9uks5qmuoBgaJpZM4Jsxej
.

from mailgun-go.

its a 4 line change to fix the GetMessage functions in the library to just take the url.

from mailgun-go.

Pull requests welcome, if you have a fix.

from mailgun-go.

@thrawn01 do we know if this is still an issue?

from mailgun-go.