-
Notifications
You must be signed in to change notification settings - Fork 227
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 ticket delegation #27
Conversation
Thanks, sounds interesting. We're about to release 1.0.0 so I'll take a deeper look after that. |
|
||
public KerberosTicketValidation(String username, String servicePrincipal, byte[] responseToken, GSSContext gssContext) { | ||
public KerberosTicketValidation(String username, String servicePrincipal, byte[] responseToken, GSSContext gssContext, GSSCredential delegationCredential) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than removing the constructor we should add an additional constructor to remain passive. Can you update the PR?
@thmarti Thanks for your PR! I commented in the code diff. |
I have created some sample code to delegate tickets to other HTTP services. More info on: http://blog.concetto.io/post/2015/09/30/Microservices-and-Kerberos-authentication |
Hi Are you still interested in this? Is there more we need to do for you to accept this PR? |
Hi @bjarkih You need a couple of things:
|
Thanks @thmarti! I will give this a try and let you know how it goes. |
@thmarti I am trying to recreate your PR as well. I get the changes that you have made but I am trying to figure out how to get to the point that I am able to have a Subject object that I will ultimately allow me to call external services using a Subject.doAs method. Thanks, |
@UlrichColby:
|
Thanks @thmarti for your pull request i've been able to integrate with a kerberized back end services really easily via my kerberos protected rest service. I had to update to the latest code from master and resolve the conflicts which can be found here https://github.com/sterowney/spring-security-kerberos Is there any plans for a 1.0.2.RELEASE with these great features? Happy to help |
In SunJaasKerberosTicketValidator.LoginConfig.getAppConfigurationEntry(), didn't you have to put isInitiator to true ? |
@ludochane: To validate a ticket 'isInitiator' has to be false. What is your scenario? What exactly are you trying to do? |
Hi @thmarti, |
What kind of Browser do use to request the initial SSO service ticket? If
it is Chrome on Windows, you need to enable ticket delegation via a
registry key. On Firefox you need to specify which HOST you trust for SSO
and ticket delegation in about:config.
…On Wed, Jul 5, 2017 at 2:45 PM, Ludovic Chane ***@***.***> wrote:
Hi @thmarti <https://github.com/thmarti>,
I try to do credentials delegation. I made it work but in my case, calling
context.getCredDelegState() always returns false. Except when I put
isInitiator to true.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABBhUNdRoJkM24QB02dx0QAdjToN0DT9ks5sK4VigaJpZM4EBs6y>
.
|
Hi @dariusan, |
@ludochane You will have to set |
@koraktor, oh yes |
What is the exact status on this topic? |
Sorry for the delayed response on this ticket. If the PR can be rebased off of master (I'm a bit pressed for time to do this myself), I'd be glad to review and merge. |
It's updated here if needed? https://github.com/sterowney/spring-security-kerberos |
@sterowney Thanks! Can you submit a PR please? |
@ludochane |
Closing in favor of #122 |
Hi
For our project we've extended spring-security-kerberos to support ticket delegation. If you're interested to have this included in the plugin we'd write a few tests and maybe a sample project and some documentaiton (whatever your exact requirements are).
Regards,
Thomas Marti
I have signed and agree to the terms of the SpringSource Individual Contributor License Agreement.