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

Remove or replace failure dependency #78

Closed
cschaible opened this issue Aug 4, 2022 · 1 comment · Fixed by #81
Closed

Remove or replace failure dependency #78

cschaible opened this issue Aug 4, 2022 · 1 comment · Fixed by #81

Comments

@cschaible
Copy link

Describe the issue
The Cargo.toml file defines a dependency to the failure crate. According to this PR here, the library is unmaintained and additionally there is a RUST-SEC warning for the library.

Additional context
To be honest - I'm not sure if the schema_registry_converter is affected at all. Searching for the term failure in the github search returned just the entry in the Cargo.toml (in docker-compose and in a license file). Maybe it was removed / replaced in the past and the entry in the Cargo.toml wasn't removed?

Nevertheless I would appreciate, if the (unused) dependency could be removed or replaced.

@gklijs
Copy link
Owner

gklijs commented Aug 5, 2022

I'll look into it. I do vaguely remember there was an pr that did something with the error handling. It might left out removing the dependency.

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