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

Version Pinning Dependencies #69

Open
ibejohn818 opened this issue Nov 6, 2019 · 3 comments
Open

Version Pinning Dependencies #69

ibejohn818 opened this issue Nov 6, 2019 · 3 comments

Comments

@ibejohn818
Copy link

ibejohn818 commented Nov 6, 2019

A recent issue with dependency ecsda 0.14.0 (tlsfuzzer/python-ecdsa#159) caused fatal errors with sshpubkeys library.

To mitigate this type of issue with dependencies I suggest pinning to specific versions VS minimum versions.
https://github.com/ojarva/python-sshpubkeys/blob/master/setup.py#L35
IE: == operator vs >=

Also, since this is a public project, a service like https://pyup.io/ can be used to continually monitor dependencies.

Thanks -JH

@tomato42
Copy link

tomato42 commented Nov 6, 2019

version pinning would also make the dependency resolution not automatically pick up a new release when there's a security fix, like with https://github.com/warner/python-ecdsa/releases/tag/python-ecdsa-0.13.3

given that only the newest released version is supported and ecdsa promises API stability, I would strongly suggest against depending on specific version

@ibejohn818
Copy link
Author

ibejohn818 commented Nov 6, 2019

That is why I suggest using pyup.io since this is a public project it would be free of charge. It monitors updates/CV's and automatically PR's updates for the maintainer to approve

@ojarva
Copy link
Owner

ojarva commented Nov 6, 2019 via email

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

No branches or pull requests

3 participants