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

Added optional LDAP authentication #30

Merged
merged 4 commits into from Oct 1, 2012
Merged

Conversation

@lightswitch05
Copy link
Contributor

lightswitch05 commented Oct 1, 2012

Added an optional LDAP authentication library: https://github.com/gwojtak/Auth_Ldap

By default authentication is turned off. It can be enabled in config/stikked.php:

$config['require_auth'] = true;

If enabling authentication, be sure to configure config/auth_ldap.php with your LDAP settings. If enabled, authentication is required for all interfaces except by the chron job.

If a user who is not authenticated tries to view a specific paste, they are redirected to the login page. Upon authentication, they are automatically sent back to the paste or URL they were trying to access before logging in. While a user is authenticated, a 'Logout' link is shown in the header next to the 'About' link.

Daniel added 4 commits Sep 29, 2012
Added Auth_Ldap for authentication. To enable authentication update
config/stikked.php and config/auth_ldap.php
Logout button will now appear in the header if authentication is enabled
and the user is logged in.
Instead of showing an ugly error, failed logins now show the login page
and a simple error about invalid username or password.
@claudehohl

This comment has been minimized.

Copy link
Owner

claudehohl commented Oct 1, 2012

AWESOME! Thanks very much for your contribution!

I was thinking about creating an enterprise branch, but that would make things too complicated. Also, the ldap authentication is optional, I'll merge this pull request right into the master.

Thanks!

claudehohl added a commit that referenced this pull request Oct 1, 2012
Added optional LDAP authentication
@claudehohl claudehohl merged commit 84d3dbc into claudehohl:master Oct 1, 2012
@lightswitch05

This comment has been minimized.

Copy link
Contributor Author

lightswitch05 commented Oct 1, 2012

Glad I could contribute, if you have any issues or questions why I did things the way I did, just let me know

@claudehohl

This comment has been minimized.

Copy link
Owner

claudehohl commented Oct 1, 2012

Thanks. Have looked at the code, fits great into the whole ecosystem. I'll test the whole auth in my company, do some CSS-styling at the loginpage and run the source through the code formatter. After that, it's ready for releasing.

@cyclopse1

This comment has been minimized.

Copy link

cyclopse1 commented Feb 3, 2013

hey buddy i m not been able to use the login process have uploaded authldap folder too but still getting the msg
An Error Was Encountered

LDAP functionality not present. Either load the module ldap php module or use a php with ldap support compiled in.

Go Home

@lightswitch05

This comment has been minimized.

Copy link
Contributor Author

lightswitch05 commented Feb 3, 2013

@cyclopse1 it has been awhile since I worked with this and no longer have Stikked installed (had potential licensing issues) - but it sounds like your PHP configuration might be messed up. Just like the error message says, you need LDAP enabled in PHP. However, I think I might also remeber getting this error message when my configuration was messed up in config/auth_ldap.php. Your error is actually coming from Auth LDAP and not Stikked. If you do a quick Google, you'll come across several pages addressing this same problem with Auth LDAP

@cyclopse1

This comment has been minimized.

Copy link

cyclopse1 commented Feb 4, 2013

can u help me i want to implement normal login without ldap an i tried to creat page but not been able to access it when upa\loaded it.
 
http://awesomemoviez.com/


From: Daniel notifications@github.com
To: claudehohl/Stikked Stikked@noreply.github.com
Cc: cyclopse1 gmohit3@yahoo.com
Sent: Sunday, 3 February 2013 10:55 PM
Subject: Re: [Stikked] Added optional LDAP authentication (#30)

@cyclopse1 it has been awhile since I worked with this and no longer have Stikked installed (had potential licensing issues) - but it sounds like your PHP configuration might be messed up. Just like the error message says, you need LDAP enabled in PHP. However, I think I might also remeber getting this error message when my configuration was messed up in config/auth_ldap.php. Your error is actually coming from Auth LDAP and not Stikked. If you do a quick Google, you'll come across several pages addressing this same pro blem with Auth LDAP

Reply to this email directly or view it on GitHub.

@claudehohl

This comment has been minimized.

Copy link
Owner

claudehohl commented Nov 7, 2013

Can you help us on this? --> #101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.