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
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Feb 11, 2016

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

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Feb 11, 2016

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

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Feb 11, 2016

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

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Feb 11, 2016

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

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Feb 11, 2016

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.

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