Skip to content
Snippets Groups Projects

Small changes to method LDAP documentation

Closed username-removed-444214 requested to merge mrchrisw/ldap-docs into master
1 unresolved thread
  • Open LDAP = OpenLDAP
  • sslthis will be on port 636 whiletls``
  • Net::LDAP library

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
53 53 incorrect indentation. Be sure to retain the indentation given in the example.
54 54 Copy/paste can sometimes cause problems.
55 55
56 > **Note:** The `method` value `ssl` corresponds to 'Simple TLS' in the LDAP
56 > **Note:** The `method` value `ssl` corresponds to 'Simple TLS' in the Net::LDAP
  • Let's keep this as-is. Users don't care what the library is so LDAP library is sufficient.

  • @dblessing Actually I disagree. In this sentence you're talking about the "method" (Simple TLS) that corresponds to the specific library. In this case the actual libraries name is Net::LDAP or net-ldap not simply LDAP.

    If a user is reading this sentence it's likely they understand the internals and are trying to understand the differing methods. e.g What is Simple TLS in the LDAP library? It's simple_tls in the Net::LDAP library.

    Edited by username-removed-444214
  • The point of this documentation is to avoid confusion between ssl and tls. Some SSL-savvy people may say, 'Oh, TLS is better than SSL. I should use that.' But in all actuality that is not true in this case. Perhaps the entire sentence/paragraphs needs restructured in that case. The intention of the docs is to stay away from the deep internals - this is a user doc, not a developer doc, etc.

    What do you think?

  • Please register or sign in to reply
  • Thanks, @MrChrisW. Please update based on the comment then it's :thumbsup_tone1:

  • Please register or sign in to reply
    Loading