The default HTTPoison adapter implementation decodes and creates atom keys, which may lead to unsolvable memory leaks.
I see here that __from_json__/2 assumes (in the default implementation) atom keys, but it doesn't have to.
Would it be worth revising that method default implementation to handle string keys, so that we avoid the problem altogether? If I'm not mistaken, we can leverage the schema as well at that point and generate it from the definition.
The default HTTPoison adapter implementation decodes and creates atom keys, which may lead to unsolvable memory leaks.
I see here that
__from_json__/2assumes (in the default implementation) atom keys, but it doesn't have to.Would it be worth revising that method default implementation to handle string keys, so that we avoid the problem altogether? If I'm not mistaken, we can leverage the schema as well at that point and generate it from the definition.