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

[dev.icinga.com #11142] LDAP User Groups backend group_filter #2290

Closed
icinga-migration opened this issue Feb 11, 2016 · 4 comments
Closed
Labels
area/configuration Affects the configuration bug Something isn't working
Milestone

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/11142

Created by gvde on 2016-02-11 13:09:00 +00:00

Assignee: aklimov
Status: Resolved (closed on 2016-02-11 15:40:04 +00:00)
Target Version: 2.2.0
Last Update: 2016-02-11 15:40:04 +00:00 (in Redmine)


https://dev.icinga.org/projects/icingaweb2/repository/revisions/master/entry/library/Icinga/Authentication/UserGroup/LdapUserGroupBackend.php#L708

The group filter is set to "filter". However, the web frontend configuration as well as documentation only mentions "group_filter". Currently, it's not possible to configure the group filter through the frontend but only in groups.ini by manually adding "filter = "cn=mygroupfilter" instead of "group_filter".

The $config~~filter in this line should be replaced with $config~~>group_filter to be consistent and fix this issue.

Changesets

2016-02-11 14:49:28 +00:00 by aklimov 32876ca

LdapUserGroupBackend: respect config option group_filter

refs #11142

2016-02-11 15:14:19 +00:00 by aklimov 5a8c430

Doc: mention that LDAP user group backends respect the config option group_filter since v2.1.3

refs #11142

2016-02-11 15:33:43 +00:00 by aklimov 8a7b840

Doc: mention what users eventually have to do while upgrading to v2.1.3

refs #11142

2016-02-11 15:36:13 +00:00 by aklimov fc30af8

Merge branch 'bugfix/ldap-user-groups-backend-group_filter-11142'

fixes #11142
@icinga-migration
Copy link
Author

Updated by jmeyer on 2016-02-11 14:27:33 +00:00

  • Category changed from Authentication & Authorization to Configuration
  • Status changed from New to Assigned
  • Assigned to set to aklimov
  • Target Version set to 2.2.0

Hi,

thanks for the report! Will be fixed asap.

Best regards,
Johannes

@icinga-migration
Copy link
Author

Updated by aklimov on 2016-02-11 15:16:35 +00:00

  • Status changed from Assigned to Feedback
  • Assigned to changed from aklimov to jmeyer

@icinga-migration
Copy link
Author

Updated by aklimov on 2016-02-11 15:28:21 +00:00

  • Status changed from Feedback to Assigned
  • Assigned to changed from jmeyer to aklimov

@icinga-migration
Copy link
Author

Updated by aklimov on 2016-02-11 15:40:04 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset fc30af8.

@icinga-migration icinga-migration added bug Something isn't working area/configuration Affects the configuration labels Jan 17, 2017
@icinga-migration icinga-migration added this to the 2.2.0 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/configuration Affects the configuration bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant