Draft:  draft-zeilenga-ldap-turn-02.txt
Reviewer: Scott W Brim [sbrim@cisco.com]
Review Date: Tuesday 7/5/2005 8:28 AM CST
Telechat Date: July 7, 2005

Summary: Ready with comments. 

Review Comments:
Again a quick review (sorry), and I apologize for this not being my
area but ..  a couple small questions:

    turnValue ::= SEQUENCE {
          mutual         BOOLEAN DEFAULT FALSE,
          identifier     LDAPString,
    }

Is that last "," supposed to be there?

In Security Considerations ...

Consider an opening paragraph citing general references for LDAP
security as context.

   - establish each other's identities through appropriate
     authentication mechanism,

Are there default and/or recommended authentication mechanisms for
LDAP?  Just what is considered "appropriate"?  I suggest citations.

   - establish an LDAP association between the initiating peer and the
     responding peer.

Isn't that redundant?  Isn't it impossible to issue a Turn without
having an LDAP association?