-
Notifications
You must be signed in to change notification settings - Fork 204
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
Unmarshalable data behaves poorly #212
Comments
Ugh, sorry for the extraneous closing of this. Applying the patch in #213 to our private repo unintentionally closed this. |
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you label it "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
I noticed the following in a few of our event logs:
A couple of observations/suggestions (which I'm happy to implement in a PR if desired):
Extra
data. It would not be difficult (although efficiency would suffer) to loop through each element ofExtra
, and strip only those that don't marshal.2 would be far less important if 1 were implemented, to make debugging simpler. But both might be ideal.
Thoughts?
The text was updated successfully, but these errors were encountered: