Don't pass a current user to Member#add_user in LDAP group sync

See the individual commits for explanation of the changes.

This is part of the fix for gitlab-org/gitlab-ee#1159.

CE backport for the change in Members::ApproveAccessRequestService is at gitlab-org/gitlab-ce!7168. /cc @stanhu

Mentioned in issue #1159 (closed)

Mentioned in merge request !829 (closed)

Added ~113221 label

Added ~313021 and removed ~113221 labels

Reassigned to @rymai

Looks good to me. Let's get this in a patch release ASAP.

Resolved all discussions

Reassigned to @stanhu

Reassigned to @rymai

@stanhu @vsizov I force-pushed the fix after @vsizov remark which was valid!

Added 2 commits:

• 8018c125 - Allow Members::ApproveAccessRequestService to accept a new :force param
• 8e748bdb - Don't pass a current user to calling Member#add_user in LDAP group sync

Compare with previous version

Changed title: Don't pass a current user to calling Member#add_user in LDAP group sync → Don't pass a current user to Member#add_user in LDAP group sync

Reassigned to @stanhu

Enabled an automatic merge when the build for 8e748bdb succeeds

Status changed to merged

Mentioned in commit 8e9eee0a

Mentioned in commit 11562fef

Picked into 8-13-stable, will go into 8.13.2.

Removed ~149424 label

mentioned in commit test-nick/gitlab-ce@52a3570a

mentioned in commit aa6aa2e9