Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Fix SQL for user search #2831

Merged
merged 2 commits into from Jan 27, 2018
Merged

Fix SQL for user search #2831

merged 2 commits into from Jan 27, 2018

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Jan 26, 2018

fix some syntax errors for user search when search_all_users is enabled

fixes #2801, hopefully

@ara4n
Copy link
Member

ara4n commented Jan 27, 2018

looks good to me. apologies for mangling it so badly first time round.

@ara4n ara4n requested review from NegativeMjark and removed request for NegativeMjark January 27, 2018 17:06
@ara4n ara4n assigned richvdh and unassigned erikjohnston and ara4n Jan 27, 2018
fix some syntax errors for user search when search_all_users is enabled

fixes #2801, hopefully
@richvdh richvdh force-pushed the rav/fix-userdir-search-again branch from 1877ea8 to d1f3490 Compare January 27, 2018 17:22
@richvdh richvdh merged commit 4c65b98 into develop Jan 27, 2018
@richvdh richvdh deleted the rav/fix-userdir-search-again branch February 1, 2018 13:01
@Geeman9
Copy link

Geeman9 commented Feb 8, 2018

Hi, I did Synapse upgrade to the latest develop build and search_all_user didn't work for me. On Riot.im app I get a message USER DIRECTORY (Offline, 0)
I am using LDAP for authentication. I added these lines to homeserver.yaml
user_directory: search_all_users: true
It don't work for LDAP or I did something wrong?
Search works only for users who joins the same room as me.

@turt2live
Copy link
Member

@Geeman9 You'll get more support in #matrix:matrix.org

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

Successfully merging this pull request may close these issues.

None yet

5 participants