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

Please consider alternatives to json crate #4

Closed
scouten-adobe opened this issue Oct 4, 2023 · 2 comments · Fixed by #5
Closed

Please consider alternatives to json crate #4

scouten-adobe opened this issue Oct 4, 2023 · 2 comments · Fixed by #5

Comments

@scouten-adobe
Copy link

This crate depends on json crate which has been unmaintained for more than three years now with no response to active issues or PRs.

We use cargo deny in our CI workflow which is how this issue was discovered. (Details: https://github.com/contentauth/c2pa-rs/actions/runs/6408578517/job/17397889116?pr=317.)

@vstroebel
Copy link
Owner

#5 Changes the dependency to the jzon crate and makes json support an optional feature.

On the long run using a completely different crate for json support might be a better option.

@scouten-adobe
Copy link
Author

@vstroebel thank you. Looking forward to new crate release once that's merged!

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

Successfully merging a pull request may close this issue.

2 participants