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

KERNEL: Replace the library NXJSON(LGPL) with json-parser(BSD) #904

Closed
winlinvip opened this issue May 24, 2017 · 5 comments
Closed

KERNEL: Replace the library NXJSON(LGPL) with json-parser(BSD) #904

winlinvip opened this issue May 24, 2017 · 5 comments
Assignees
@winlinvip
Labels
EnglishNative This issue is conveyed exclusively in English. Feature It's a new feature.
Milestone
3.0

Comments

@winlinvip
Copy link
Member

Replace NXJSON for LICENSE issue: https://github.com/ossrs/srs/wiki/LicenseMixing#nxjson
@winlinvip winlinvip added the Feature It's a new feature. label May 24, 2017
@winlinvip winlinvip added this to the srs 3.0 release milestone May 24, 2017
@winlinvip
Copy link
Member Author

winlinvip commented May 25, 2017 •
edited
Loading

BSD: https://github.com/udp/json-parser
MIT: https://github.com/DaveGamble/cJSON
MIT: https://github.com/akheron/jansson
GPLv2+MPL+BSD: https://github.com/ossrs/state-threads#license

@winlinvip
Copy link
Member Author

The jansson doesn't support single file.
cJSON is not sure to work with C++.
Seems json-parser is fine.

@winlinvip
Copy link
Member Author

Update the research/api-server/server.py, response an object:

{
    "code": 0,
    "data": null
}

SRS http hooks also support a single code:int response, which is SRS1 style.

@winlinvip
Copy link
Member Author

Replace NXJSON which use LGPL.

winlinvip added a commit that referenced this issue May 29, 2017
@winlinvip
Fix #904, replace NXJSON(LGPL) with json-parser(BSD). 3.0.23
54411e0
@winlinvip
Copy link
Member Author

Fixed

@winlinvip winlinvip changed the title KERNEL: Replace the JSON parser NXJSON KERNEL: Replace the library NXJSON(LGPL) with json-parser(BSD) May 29, 2017
@winlinvip winlinvip self-assigned this Sep 15, 2021
@winlinvip winlinvip added the EnglishNative This issue is conveyed exclusively in English. label Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@winlinvip winlinvip

Labels
EnglishNative This issue is conveyed exclusively in English. Feature It's a new feature.
Projects
None yet
Milestone
3.0
Development

No branches or pull requests
1 participant
@winlinvip