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

LDAP: authentication user is broken #266

Closed
chrisch1974 opened this issue Jul 21, 2015 · 4 comments
Closed

LDAP: authentication user is broken #266

chrisch1974 opened this issue Jul 21, 2015 · 4 comments
Assignees
Labels
feature Adding new functionality
Milestone

Comments

@chrisch1974
Copy link
Contributor

Original approach of the LDAP implantation is that you first need to setup the LDAP server. For the authentication you need to create the same group in OPNsense and LDAP. The permissions in auth.inc are read correctly but they are broken in OPNsense\Core\ACL.php as there are only local users processed. This results in an empty system menu.

Better approach would be to create the user with linking to LDAP in OPNsense. And group permissions should be handled locally.

@AdSchellevis AdSchellevis self-assigned this Jul 21, 2015
@AdSchellevis
Copy link
Member

commit b4845e6 contains the "final" part of the rewrite.
You can import ldap users now in system_usermanager.php and change the permissions afterwards.

@fichtner fichtner added the feature Adding new functionality label Jul 28, 2015
@fichtner fichtner added this to the 16.1 milestone Jul 28, 2015
@fichtner
Copy link
Member

Note that this commit is not part of 15.7.5's opnsense-devel package, but will be available in 15.7.6's *opnsense-devel. Whether or not this code will go into the stable 15.7.6 needs to be decided...

@fichtner
Copy link
Member

@AdSchellevis @chrisch1974 close this then?

@AdSchellevis
Copy link
Member

yes, let's close it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding new functionality
Development

No branches or pull requests

3 participants