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

[Feature Request]: LDAP Add config option to make FASLE = Enabled #14926

Open
azmcnutt opened this issue Jun 20, 2024 · 0 comments
Open

[Feature Request]: LDAP Add config option to make FASLE = Enabled #14926

azmcnutt opened this issue Jun 20, 2024 · 0 comments

Comments

@azmcnutt
Copy link

Is your feature request related to a problem? Please describe.

No

Describe the solution you'd like

Some LDAP sources do not provide an LDAP Active Flag where TRUE or 1 = enabled. For instance, Google LDAP has a field called "suspended" where TRUE or 1 = account is not enabled.

Could an option be added to LDAP setup to change from LDAP Active Flag to LDAP Inactive Flag? So if the setting is not checked, the setting would work as normal. However, if the setting was checked or set to Inactive Flag, the LDAP field would return 1 or TRUE when the account is to be deactivated.

Describe alternatives you've considered

Currently I change the source after each upgrade. These are the change I make to the source:
Edit the file app/Console/Commands/LdapSync.php
Around line 337 Make the following change:
From: $filter_var = filter_var($raw_value, FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE);
To: $filter_var = !filter_var($raw_value, FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE);
(add ! in front of = (!)filter)

While this is probably not the correct way to do things, the outcome provides what I need for Syncing with Google LDAP.

Additional context

We have been self hosted for quite some time, but we would like to eliminate on-premises servers. If this change were implemented, my organization would change to Snipe hosting services. I could also take a proposal to leadership to pay to have this feature added for our needs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants