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.

Sign up for GitHub

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

Jump to bottom

Stop using failing ujson as default serializer #589

Open
gghez opened this issue Feb 7, 2019 · 1 comment
Open

Stop using failing ujson as default serializer #589

gghez opened this issue Feb 7, 2019 · 1 comment
Labels
bug Good first issue A good issue to start contributing
Projects
Roadmap
Milestone
0.12.2

Comments

@gghez
Copy link

gghez commented Feb 7, 2019

ujson is no more actively maintained and specially for some critical bugs not present in json package.

one example forcing us to revert ujson usage:
ultrajson/ultrajson#325 (2 days ago, mine)
ultrajson/ultrajson#301 (same but 1 year ago and no answer)
@ziirish ziirish added bug Good first issue A good issue to start contributing labels Apr 10, 2019
@ziirish ziirish added this to To Do in Roadmap Apr 10, 2019
@ziirish
Copy link
Collaborator

ziirish commented Apr 10, 2019

Thanks for the report.

We will address this shortly hopefully.

This was referenced Apr 10, 2019
Open
Open
@ziirish ziirish added this to the 0.12.2 milestone Apr 10, 2019
ziirish added a commit that referenced this issue May 6, 2019
@ziirish
stop relying on ujson (fix #589, #587, #507)
bee950a
@SteadBytes SteadBytes moved this from To Do to In Progress in Roadmap May 10, 2019
@jjjamie jjjamie mentioned this issue Aug 18, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Good first issue A good issue to start contributing
Projects
Roadmap
  
In Progress
Milestone
0.12.2
Development

No branches or pull requests
2 participants
@ziirish @gghez