Skip to content

[IMPROVE] Add template tag #{userdn} to filter LDAP group member format#11662

Merged
sampaiodiego merged 1 commit intoRocketChat:developfrom
crazy-max:develop
Aug 13, 2018
Merged

[IMPROVE] Add template tag #{userdn} to filter LDAP group member format#11662
sampaiodiego merged 1 commit intoRocketChat:developfrom
crazy-max:develop

Conversation

@crazy-max
Copy link
Copy Markdown
Contributor

@crazy-max crazy-max commented Aug 2, 2018

In some cases (including mine) LDAP servers use the full user dn to filter members belonging to a group.
Here by adding a template tag #{userdn} composed of the complete DN of the user, we can easily translate this into the group member format to find it :

image

Ping @rodrigok :)

@crazy-max crazy-max changed the title [NEW] Add template tag #{userdn} to filter group member format [NEW] Add template tag #{userdn} to filter LDAP group member format Aug 2, 2018
@crazy-max crazy-max changed the title [NEW] Add template tag #{userdn} to filter LDAP group member format [FIX] Add template tag #{userdn} to filter LDAP group member format Aug 3, 2018
@crazy-max
Copy link
Copy Markdown
Contributor Author

@sampaiodiego @geekgonecrazy

Hey guys, can you give me some feedback on this ?
We need this fix in our company :)

Thanks a lot!

@crazy-max
Copy link
Copy Markdown
Contributor Author

@sampaiodiego Will it be merged for the next minor release ? Thanks!

@sampaiodiego sampaiodiego added this to the 0.69.0 milestone Aug 13, 2018
@sampaiodiego sampaiodiego changed the title [FIX] Add template tag #{userdn} to filter LDAP group member format [IMPROVE] Add template tag #{userdn} to filter LDAP group member format Aug 13, 2018
@sampaiodiego sampaiodiego merged commit e4ff555 into RocketChat:develop Aug 13, 2018
@sampaiodiego
Copy link
Copy Markdown
Member

@crazy-max only on next major (0.69)

@crazy-max
Copy link
Copy Markdown
Contributor Author

@sampaiodiego Ok thx

@sampaiodiego sampaiodiego mentioned this pull request Aug 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants