-
Notifications
You must be signed in to change notification settings - Fork 110
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 production-ready configuration examples in documentation #113
Comments
Would really appreciate docs on OneLogin SAML integration - have been beating my head against it today.. |
Hey there. I think we can work up an example for SAML, but it probably won't be until next week. |
Great! 😀
If I get it working before then, I can contribute to the documentation with
my findings.
…On 4 Feb. 2017 9:18 am, "Ryan Lane" ***@***.***> wrote:
Hey there. I think we can work up an example for SAML, but it probably
won't be until next week.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#113 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABONYw33GwxB9e1QUITbuOleuFKlel_vks5rY6fPgaJpZM4L11Ep>
.
|
I'm stuck, getting this error from confidant:
Even a very quick and dirty example would be really helpful - I think I've got things in the right places, but clearly I'm still not doing it right. 😓 |
We're working on an example that should be ready in a day or two. I'd share something quick and dirty, but I don't have anything off-hand right now. |
Really looking forward to seeing this example! Struggling with OneLogon... 😓 |
@egeland I don't think I've seen your error before. You've probably already got this far, but it looks like your SAML is error-ing here: https://github.com/lyft/confidant/blob/master/confidant/authnz/userauth.py#L573-L582 The session token is being set here: https://github.com/lyft/confidant/blob/master/confidant/authnz/userauth.py#L818 This occurs right after the SAML request is initiated (and therefore it is able to get a request ID to store and check later). You might want to put a break point here and check that |
I pulled and built an image from current HEAD ( 95065eb ) and got logged in via SAML.. so, some progress! 👍 |
We have quickstart docs, but no examples for production deployment.
The text was updated successfully, but these errors were encountered: