[WIP] read all relevant user attributes on login, in one query. saves us some.

[blizzz]

What has happened before:

1. upon login, we queried LDAP for the DN corresponding to the username
2. we would do a bind to verify authentication
3. if they need to be updated, single readQueries will be done for quota, email, avatar
4. home directory attribute would also be read with a single request

What we do now:

1. upon login, we query LDAP for DN, quota, email, avatar, home directory attribute in the same request
2. we do a bing to verfiy authentication
3. we apply (i.e. store or cache) all the values we fetched before without any single additional request

I.e. in general this brings us performance improvements by less queries to the LDAP server. LDAP server will be happy, too, about less work. Helped with https://github.com/owncloud/enterprise/issues/442 in a OC 7 version (although it was not the root problem).

Also solves #12823

todos left

• user tests need to be adjusted and extended

[scrutinizer-notifier]

The inspection completed: 11 new issues, 7 updated code elements

[karlitschek]

@craigpg

[DeepDiver1975]

needs rebase and in WIP state -> OC8.1

[ghost]

Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/12058/

Build result: FAILURE

[...truncated 159 lines...] > git config --local --remove-section credential # timeout=10 > git config remote.origin.url https://github.com/owncloud/core.git # timeout=10 > git config --add remote.origin.fetch +refs/heads/:refs/remotes/origin/ # timeout=10 > git config remote.origin.url https://github.com/owncloud/core.git # timeout=10Fetching upstream changes from https://github.com/owncloud/core.gitusing GIT_SSH to set credentials using .gitcredentials to set credentials > git config --local credential.helper store --file=/tmp/git4683420848359394944.credentials # timeout=10 > git -c core.askpass=true fetch --tags --progress https://github.com/owncloud/core.git +refs/pull/:refs/remotes/origin/pr/ > git config --local --remove-section credential # timeout=10 > git rev-parse a6e7f67^{commit} # timeout=10Checking out Revision a6e7f67 (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f a6e7f67 > git rev-list 2b041b1 # timeout=10 > git remote # timeout=10 > git submodule init # timeout=10 > git submodule sync # timeout=10 > git config --get remote.origin.url # timeout=10 > git submodule update --init --recursiveTriggering pull-request-analyser-ng-simple » SLAVEConfiguration pull-request-analyser-ng-simple » SLAVE is still in the queue: Waiting for next available executor on SLAVEpull-request-analyser-ng-simple » SLAVE completed with result FAILURESetting status of a6e7f67 to FAILURE with url https://ci.owncloud.org/job/pull-request-analyser-ng-simple/12058/ and message: Build finished.
💣 Test FAILed. 💣

[PVince81]

@blizzz is this oldie still valid ?

[blizzz]

With extending it also to user searches it would fix #17362

[blizzz]

replaced by #18469 (too many changes in master since)