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

Dropping dependency on golang.org/x/crypto #928

Closed
tandr opened this issue Mar 25, 2019 · 6 comments
Closed

Dropping dependency on golang.org/x/crypto #928

tandr opened this issue Mar 25, 2019 · 6 comments

Comments

@tandr
Copy link
Contributor

tandr commented Mar 25, 2019

This library is pulled for only one check - to see if output is a terminal or not.

Would it be possible just to inline one used function from it? This would allow us (and all people who pulls it in) to drop the dependency on it as well. The import with "crypto" in it triggers additional attention from our legal department, and we have to explain that it does not have anything "cryptographic" in it - just a check for is it a terminal or not.

Ideally, ssh/terminal should be not under crypto , but I will try to ping golang folks about it directly.

@lavoiesl
Copy link
Collaborator

I like this idea; I opened a PR.

@lavoiesl
Copy link
Collaborator

Unfortunately, the implementation is os-dependent and it’d be quite many files to bring over to get rid of ssh/terminal.

I suggest leaving it as is for now.

@tandr
Copy link
Contributor Author

tandr commented Mar 26, 2019

8 files to bring over. Password-reading related stuff can be removed.

golang/go#31044 got reference for golang/go#13104, but honestly, it does not look very promising short term.

@tandr
Copy link
Contributor Author

tandr commented Mar 26, 2019

Actually... there is a whole bunch of file in logrus starting with terminal_*, we can try to copy these functions there.

@dgsb
Copy link
Collaborator

dgsb commented Mar 26, 2019

I'd rather not, it would be hard to keep up with updates

@tandr
Copy link
Contributor Author

tandr commented Mar 26, 2019

There are no updates since ... 2011 and 2015

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