search

evendis / mandrill-rails

Webhook processing and event decoration to make using Mandrill with Rails just that much easier
MIT License
288 stars 36 forks source link

Handling attachments from inbound events #1

Closed jasonpaluck closed 11 years ago

jasonpaluck commented 11 years ago

Thanks for your work on this gem, has saved me a ton of time wrangling JSON responses. Do you have working example or a suggestion for how to parse then save the array of (potential) attachments in an inbound webhook post? I'm properly handling the inbound event via handle_inbound(event_payload) but am struggling to get at the attachment array elements.

Thanks,

Jason

tardate commented 11 years ago

Hi Jason, great to hear you are getting some value from it!

And yes, absolutely. I've built up documentation and examples for most of the event types that I'm using in production.

I must sheepishly admit it has been a lingering TODO to at least get that info into the readme, but I just haven't gotten around to yet. I'll push something up real soon ok.

tardate commented 11 years ago

I've just added some more doco, but realised I didn't have an example for the specific question you had of extracting inbound email attachments. I need to take a fresh look at this to see if Mandrill has improved its attachment delivery (last I looked, it seemed the only solution was to do direct extraction from the mime-encoded raw message).

Have you got far with this yet Jason? Any message payload examples you could share?

tardate commented 11 years ago

Jason, I put some explicit attachment handling in place (but haven't released a gem update yet). Can you check this out - does it work in your cases?

jasonpaluck commented 11 years ago

This is precisely what I was looking for! Thank you kindly for all the quick work, much appreciated. The only code comment I had was to skip decoding for text-based attachments but it appears you beat me to it.

tardate commented 11 years ago

Great Jason. I pushed these changes out in a 0.0.3 version of the gem, so I'll close this issue out for now. Do let me know if you run into any edge-case encoding scenarios that the gem should handle.

Happy mail processing!

jasonpaluck commented 11 years ago

Paul,

Thank you again for your help, I'll update the gem now and get cracking.

Thanks,

Jason

On Feb 14, 2013, at 10:43 AM, Paul Gallagher notifications@github.com wrote:

Great Jason. I pushed these changes out in a 0.0.3 version of the gem, so I'll close this issue out for now. Do let me know if you run into any edge-case encoding scenarios that the gem should handle.

Happy mail processing!

— Reply to this email directly or view it on GitHub.