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 GSS-API support for using the 'gssapi' package (in addition to older 'python-gssapi') #1166

Closed
wants to merge 4 commits into from

Conversation

@akruis
Copy link

@akruis akruis commented Feb 9, 2018

I rebased pull request #861 for 2.2. Additionally I improved backwards compatibility a bit (keep the internal name of the old python-gssapi class).

This pull request could be applied to 2.3 too.
@SebastianDeiss: please review this pull request.

sigmaris added 2 commits Feb 9, 2018
Detect if "python-gssapi" or "gssapi" is installed at import time
and use the appropriate API
@SebastianDeiss
Copy link

@SebastianDeiss SebastianDeiss commented Feb 21, 2018

@akruis: LGTM.

@sscherfke
Copy link

@sscherfke sscherfke commented Jun 1, 2018

When will this be merged and released?

@sscherfke
Copy link

@sscherfke sscherfke commented Aug 30, 2018

@bitprophet Can I help you with anything to get this patch merged?

akruis pushed a commit to scVENUS/paramiko that referenced this issue Oct 5, 2018
Previously testing of GSSAPI (Kerberos) related functions required an
externally provided Kerberos environment. Therefore all GSSAPI tests were
skipped.

Now the package k5test is used to setup a self-contained Kerberos environment.
Because k5test requires the new GSSAPI, this commit also merges
pull request paramiko#1166 and fixes broken GSSAPI test. If k5test is not available
(i.e. on Windows), the tests still get skipped.

The test case test_kex_gss.test_2_gsskex_and_auth_rekey is expected to fail.
@akruis
Copy link
Author

@akruis akruis commented Oct 5, 2018

An updated version of this pull request is available as #1311.

@bitprophet bitprophet closed this Oct 16, 2018
ploxiln added a commit to ploxiln/paramiko-ng that referenced this issue May 20, 2019
Previously testing of GSSAPI (Kerberos) related functions required an
externally provided Kerberos environment. Therefore all GSSAPI tests were
skipped.

Now the package k5test is used to setup a self-contained Kerberos environment.
Because k5test requires the new GSSAPI, this commit also merges
pull request paramiko#1166 and fixes broken GSSAPI test. If k5test is not available
(i.e. on Windows), the tests still get skipped.

The test case test_kex_gss.test_2_gsskex_and_auth_rekey is expected to fail.
ploxiln added a commit to ploxiln/paramiko-ng that referenced this issue May 23, 2019
Previously testing of GSSAPI (Kerberos) related functions required an
externally provided Kerberos environment. Therefore all GSSAPI tests were
skipped.

Now the package k5test is used to setup a self-contained Kerberos environment.
Because k5test requires the new GSSAPI, this commit also merges
pull request paramiko#1166 and fixes broken GSSAPI test. If k5test is not available
(i.e. on Windows), the tests still get skipped.

The test case test_kex_gss.test_2_gsskex_and_auth_rekey is expected to fail.

plus some flake8 warning and python3 fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants