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

Add unmaintained ethabi advisory #1743

Closed
wants to merge 1 commit into from

Conversation

maxammann
Copy link
Contributor

See the readme of the project.

@amousset
Copy link
Member

The README only states that the project is not actively maintained, and the author is responsive. If still "passively maintained" (i.e. no active development but required security fixes would be made), then it does not match our criteria for the unmaintained advisories.

@vkgnosis Do you think an advisory makes sense or is the project still (passively) maintained?

@maxammann
Copy link
Contributor Author

I agree that we should ask again. This is the backstory on why I created this: rust-ethereum/ethabi#301

@Shnatsel
Copy link
Member

The maintainer's stance doesn't seem to be substantially different from the typical "full disclosure" policy.

As Alexis has pointed out, this crate does not match our criteria for the unmaintained advisories. I'm going to go ahead and close this.

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 this pull request may close these issues.

3 participants