Tldr: page request: ldapsearch

[grafuls]

ldapsearch command can be pretty complex, would be nice to see this here.
https://linux.die.net/man/1/ldapsearch

[sbrl]

Thanks for requesting this page, @grafuls! Do you want to have a go yourself?

Pages are best created by people who understand the command they are about.

If you do, take a quick read of CONTRIBUTING.md, which guides you through the process of creating a new page here. Then, just simply send a pull request to this repository - and if you need any assistance, just ask :-)

[lamar-frankie]

I'd like to claim this issue. If @grafuls hasn't started working on it.

[grafuls]

@lamar-frankie I haven't got the time to get to it, all yours.

[lbonanomi]

Is this issue up-for-grabs?

[sbrl]

It looks like it, @lbonanomi. Go ahead :-)

[lbonanomi]

LDAP is mature enough to have spawned several "standard" clients across different platforms, so I opened a draft PR (https://github.com/tldr-pages/tldr/pull/2915) for the generic features that appear across both Linux and SunOS clients.

The major difference seems to be support for timelimit, sizelimit and extensions; Linux has these, SunOS' stock ldap client doesn't. Would it be appropriate to maintain Linux and SunOS pages independently to show Linux users all their options without teasing SunOS users?

[schneiderl]

Hi @lbonanomi
I think that the way to go: having two pages.
1 under common/linux and 1 under common/sunos.

Would you send a PR with the proposed changes?

[lbonanomi]

https://github.com/tldr-pages/tldr/pull/2915 has been upgraded to a regular PR.

I'd appreciate feedback about any examples maintainers think could be dropped, I'd like to add a line for result-paging on the Linux client but don't want to violate the 8 examples limit without a formal go-ahead.

[sbrl]

Thanks, @lbonanomi! I'm not sure which example we could drop.

[lbonanomi]

@sbrl I hate to nag but is there anything left to be done on this issue before it gets closed?

[agnivade]

Nope. In future, you can use the "Fixes #issue_no" string in a PR, which will automatically close an issue when the PR is merged.

[lbonanomi]

Thank you @agnivade!