Skip to content
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

Consider changing to Jason json library #27

Open
nacengineer opened this issue Dec 5, 2023 · 1 comment
Open

Consider changing to Jason json library #27

nacengineer opened this issue Dec 5, 2023 · 1 comment

Comments

@nacengineer
Copy link

The poison library has quite a few open issues. Including at least one for coming Elixir deprecations, e.g. use Bitwise, and devinus has been silent on adopting it/them. Not saying poison is dead but 2 years without pulling in easy PRs is kind of that to me.

Consider pivoting to the more maintained Jason library.

@jdfrens
Copy link
Contributor

jdfrens commented May 9, 2024

There's an unadvertised config option: :json_encoder.

config :airbrake_config, 
   ...
  json_encoder: Jason,
  ....

I'm reluctant to make Jason the default because it'll blow up on structs that don't have encoders (where I think Poison just encodes them as maps).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants