Code review comment for lp:~markthomas/serverguide/serverguide-review-7.5

Revision history for this message
Mark Thomas (markthomas) wrote :

Content changes:

(a) I implemented a variation of that change. It is important to highlight that the “ad” provider is what is being covered—the use of the “ldap” provider in sssd for AD authentication is quite different, older, and more difficult. I did omit the name “Microsoft”, as it is also possible to use Samba4 as an AD domain controller. Do you think it is needed?

(b) Fixed

(c) Modified for clarification. I am not trying to specify formatting—that is covered soon after. I am providing the value used rather than have it just “show up” as I’ve seen in other documentation.

One thing I did here was to use a “subdomain” for the Active Directory domain. So often, I see just “example.com”. If someone wants to implement the AD domain as a subdomain, as is often done, I didn’t want it left an an exercise for the reader to figure out what the realm should be by “trial and error” as I’ve had to do.

(d) Modified for clarification.

(e) Fixed

2. format

(a) Fixed
(b) Fixed
(c) Fixed

« Back to merge proposal