Closed dgmstuart closed 4 months ago
Confirmed the issue: this fork works: https://github.com/dxw/griddler/commit/f7dfd23b7b16902b761746841888bd031a748d1a
Wish I had time to create a tested PR but we're just hacking together a quick proof of concept. If we end up doing this for real then I'll definitely make a PR.
Is this still an issue? I have not had issues with a newish Rails 5 + Griddler app. I'm using the following dependencies:
griddler (1.4.0)
griddler-mailgun (1.0.3)
rails (5.1.4)
Have been able to successfully receive emails from Mailgun to my app in production (Heroku).
If this gem is compatible with Rails 5, we might close this to avoid future confusion. Although I looked at the source and https://github.com/dxw/griddler/commit/f7dfd23b7b16902b761746841888bd031a748d1a or something like it was not merged, so I'm not sure that it's actually fixed. Maybe I am somehow avoiding it or it would happen in the future...
Thanks!
Hi @panozzaj - I'm afraid I don't know: we only used this on a prototype that one time. I agree, looking at the code I'd expect this to still be failing.
No worries, thanks for your comment in the past with a fix! Just figured I'd give a field report to update on what I was seeing.
I don't have a Rails 5 deployment of Griddler to test against so if someone wants to make a PR....
Seeing this on Rails 5.2.2 with griddler 1.5.2 and griddler-cloudmailin (1.1.0)
Also seeing this error on Rails 5.1.6 with the same griddler 1.5.2 and griddler-cloudmailin 1.1.0. Seems like the patch above is simple, so it's disappointing this hasn't been fixed all this time later.
Rails 5.2.4.3 with griddler-sendgrid is working fine.
@joemsak I believe Thoughtbot took the stance here that Rails 6 won't be supported due to ActionMailbox being a thing
@wingrunr21 oh shoot okay, I didn't realize that! thank you! - oh my, I forgot Rails 6 added this feature!
I’m happy to merge a PR or something but the functionality is duplicative
Hi
The issue here seems to be that the headers are getting passed as an
ActionController::Parameters
object.In the past I gather that this used to lie about its type and pretend to be a hash. Now hashes are handled by the code, but anything that is not an array or hash is incorrectly assumed to be a string (or string-like?)
I'm using https://github.com/thoughtbot/griddler-cloudmailin/ but the issue seems to be in griddler core code.
Full stack trace: