Skip to content

don't show recurring log msg when paged result was turned off - #7599

Merged
MorrisJobke merged 1 commit into
masterfrom
quieter-debug-log
Jan 2, 2018
Merged

don't show recurring log msg when paged result was turned off#7599
MorrisJobke merged 1 commit into
masterfrom
quieter-debug-log

Conversation

@blizzz

Copy link
Copy Markdown
Member

and only as debug level otherwise.

It's a small change. Steps for testing:

  1. Have LDAP enabled
  2. Set PagingSize to 0 (LDAP settings → advanced → directory settings). This turns of using paged results with the LDAP server.
  3. Trigger a search, for instance in file sharing sidebar
  4. check log

Previously:

See "Paged search was not available" for every LDAP search request, info level.

Now:

The entry does not appear, since it was turned off intentionally.

@nextcloud/ldap

and only as debug level otherwise.
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzzblizzz added this to the Nextcloud 13 milestone Dec 21, 2017
@blizzzblizzz changed the title don't show recurring msg when pages result was turned offdon't show recurring log msg when paged result was turned offDec 21, 2017
@MorrisJobkeMorrisJobke mentioned this pull request Jan 2, 2018
30 tasks
@MorrisJobke
MorrisJobke merged commit 4d0315c into masterJan 2, 2018
@MorrisJobke
MorrisJobke deleted the quieter-debug-log branch January 2, 2018 23:40
@blizzz

Copy link
Copy Markdown
MemberAuthor

backport #7708, stable12 should suffice

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: ldap

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@blizzz@rullzer@MorrisJobke