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
Encoding::UndefinedConversionError #6
Comments
@michaelshobbs, I'm not sure. I think setting the locale to UTF-8 should fix this error. There are a lot of complains out there about this type of encoding errors in Chef. Could you give me more information to know how to reproduce the error? |
Well that's a might bit tricky I think. I use this project to notify me on failed deployments on AWS OpsWorks using dokku. I think it must be it's failure output or that of a failed plugin. Not 100% certain. Could also be that combined with upgrading to Chef 11.10. I can reproduce it in my production environment consistently though. |
@zuazo when you say set the locale, to what specifically are you referring? |
Something like |
I set that but to no avail.
|
That's strange. Are you sure chef is running with this locale? Perhaps some kind of What is the message body? |
I set Not sure how to get at the message body. Any pointers? |
The message body is the content of the email you should receive through SNS. That is what is causing the error. I guess you are using the default body template. Maybe there is an exception to report that has some kind of encoding error/UTF-8 character. I think maybe we should treat everything as UTF-8 and allow to change the charset. |
My template is different than the one to which you linked but yeah I'm sure that is the issue. The message is sent successfully in all cases except when there is a failure in my deployment. Below is my template and a gist of a failed deployment using my patch from #7.
report with UTF-8: https://gist.github.com/michaelshobbs/5b6bc933fb3dc9eb96c6 |
Fixed in #7. Thanks for all your help! |
This is still an issue even with the patch.
|
👍 |
@michaelshobbs, sorry for the delay and thanks for all your help and feedback! Current master released in |
Thanks!! I'll test 2.0.0 in the new year. |
Is there anything we can do in
chef-handler-sns
to "fix" this?The text was updated successfully, but these errors were encountered: