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 support for configurable use of Kerberos ticket cache. #145

Merged
merged 3 commits into from
Feb 4, 2015

Conversation

mbidewell
Copy link
Contributor

Allow the Kerberos ticket cache to be used instead of password for authentication.

@buildhive
Copy link

XebiaLabs » overthere #247 SUCCESS
This pull request looks good
(what's this?)

@hierynomus
Copy link
Contributor

This seems to be incomplete, as there is no way to now enable the ticketCache in Overthere. This is just the preparation work for using the ticketCache in the WinRM client.

Furthermore I'd like to see some test cases on how this works, and proof that it does. There are integration tests for a lot of scenarios already, see *ITest.java

@buildhive
Copy link

XebiaLabs » overthere #250 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

XebiaLabs » overthere #251 SUCCESS
This pull request looks good
(what's this?)

@mbidewell
Copy link
Contributor Author

I completed the wiring (sorry for the missing elements). On the integration tests, I have looked over the existing ones and I'm not sure how best formulate one. Enabling useTicketCache, allows the use of a cached Kerberos ticket as a fallback to password authentication. I believe a test would require a user with a cached ticket. Any thoughts how to guarantee this?

hierynomus added a commit that referenced this pull request Feb 4, 2015
Add support for configurable use of Kerberos ticket cache.
@hierynomus hierynomus merged commit 4d8df71 into xebialabs:master Feb 4, 2015
@hierynomus
Copy link
Contributor

Thx... Thinking about how to test this in integration tests / unit tests..

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