Skip to content
This repository has been archived by the owner on Nov 11, 2019. It is now read-only.

Switch to Tornado-Json Extension #8

Closed
ayushr2 opened this issue Nov 13, 2018 · 0 comments
Closed

Switch to Tornado-Json Extension #8

ayushr2 opened this issue Nov 13, 2018 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@ayushr2
Copy link
Contributor

ayushr2 commented Nov 13, 2018

We should probably switch to this extension: tornado-json
It enables us to define schemas, which leads to better error handling and simplifies the entire logic. Makes it easier to make JSON requests. Circumvents the annoying feature of tornado of providing all args as byte strings (which does not play well with python 3) and mess with lists. We will not have to urlencode and decode on both sides.

@ayushr2 ayushr2 added the enhancement New feature or request label Nov 13, 2018
@ayushr2 ayushr2 self-assigned this Nov 16, 2018
@ayushr2 ayushr2 mentioned this issue Nov 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant