Skip to content

Remove explanation about "sizelimit exceeded" - #922

Merged
MorrisJobke merged 2 commits into
masterfrom
enhancement/noid/extend-sizelimit++
Nov 15, 2018
Merged

Remove explanation about "sizelimit exceeded"#922
MorrisJobke merged 2 commits into
masterfrom
enhancement/noid/extend-sizelimit++

Conversation

@blizzz

Copy link
Copy Markdown
Member

even stated in the php docs: http://php.net/manual/en/function.ldap-search.php

Some directory server hosts will be configured to return no more than a preset number of entries. If this occurs, the server will indicate that it has only returned a partial results set. This also occurs if you use this parameter to limit the count of fetched entries.

even stated in the php docs: http://php.net/manual/en/function.ldap-search.php
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz

blizzz commented Nov 5, 2018

Copy link
Copy Markdown
MemberAuthor

actually… if we would wrap the error handler in ldap, we could throw this away.

EDIT:nextcloud/server#12292 – if this is merged, we should scrap this section instead :)

@blizzzblizzz changed the title extend explanation about "sizelimit exceeded"⛔️️️️️️️️️️️️️️️️️️️️ extend explanation about "sizelimit exceeded"Nov 5, 2018
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobkeMorrisJobke changed the title ⛔️️️️️️️️️️️️️️️️️️️️ extend explanation about "sizelimit exceeded"Remove explanation about "sizelimit exceeded"Nov 15, 2018
@MorrisJobke

Copy link
Copy Markdown
Member

EDIT:nextcloud/server#12292 – if this is merged, we should scrap this section instead :)

Done and I updated this PR to remove the section.

@MorrisJobke
MorrisJobke merged commit cc1b3e6 into masterNov 15, 2018
@MorrisJobke
MorrisJobke deleted the enhancement/noid/extend-sizelimit++ branch November 15, 2018 10:08
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@blizzz@MorrisJobke