Add multiple features, bug fixes and update libssh2 to the latest stable. #90
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This enables giving ssh a prefered list of options such as kex, ciphers, MACs and etc
Add libsssh2_session_method_pref
Add recompiled cython files
Regenerate Cython files. Add `ssh2.session.Session.method_pref()` doc string. Update .gitignore to include src directory to ignore libssh2 rebuilds.
Add additional error codes. Add `session.method_pref()` docs and tests. Add `session.methods()`, docs and tests. Add `session.supported_algs()`, docs and tests. Add `session.flag()`, docs and tests. Bug fix for `session.userauth_publickey_frommemory()` being provided no public key and throwing a type error.
This comment was marked as spam.
This comment was marked as spam.
added 2 commits
November 9, 2019 22:20
Add callback function typedef for keyboard interactive authentication. Add `ctypedef`s for keyboard interactive authentication. Fix keyboard interactive authentication function def in c_ssh. Internally hide that `libssh2_userauth_keyboard_interactive` takes a callback function and not a string...
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
added 9 commits
December 8, 2019 19:29
Add enums to clean up any symbols from libssh2 and make interacting with them much easier to do. Add additional unit tests for ParallelSSH#86
Correct issue in `ssh2.sftp_handel.SFTPHandle.readdir*()` which adds EAGAIN items into the generators.
Rename library to redlibssh2.
|
@pkittenis are you still maintaining this project? This PR has been incubating for quite some time and fixes many outstanding issues. @Red-M I appreciate the work put in here + to get it out on pypi so it's more easily accessible. That said, would you mind cutting a new/edited PR in this repo that points to a commit before you made some of the renaming changes in your master branch? It would be better to merge your changes here rather than forking off forever, and in its current state the PR has unrelated changes. |
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New PR at #94