ietf
[Top] [All Lists]

Re: Gen-ART review of draft-ietf-sasl-gs2-18

2010-01-08 06:22:50
"Spencer Dawkins" <spencer(_at_)wonderhamster(_dot_)org> writes:

Summary: This document is almost ready for publication as a Proposed
Standard. I did have one minor question about 13.3 (in my LATE
review), but it should not be difficult to resolve, if an AD agrees
with my question.

Hi Spencer.  Thank you for your careful review.  I'm now applying your
comments to the document.

I did tag a fair number of nits, but these aren't part of the Gen-ART
review, and are simply included as a convenience for anyone else who
edits the document.

This is appreciated!

1.  Introduction

  The GS1 bridge failed to gain wide deployment for any GSS-API
  mechanism other than The "Kerberos V5 GSS-API mechanism" [RFC1964]

Spencer (nit): s/The "Kerberos/"The Kerberos/

Fixed.

  [RFC4121], and has a number of problems that lead us to desire a new

Spencer (nit): s/lead/led/

Fixed.

  bridge.  Specifically: a) GS1 was not round-trip optimized, b) GS1
  did not support channel binding [RFC5056].  These problems and the
  opportunity to create the next SASL password-based mechanism, SCRAM

Spencer (nit): please expand SCRAM on first use.

Fixed.

  [I-D.ietf-sasl-scram], as a GSS-API mechanism used by SASL
  applications via GS2, provide the motivation for GS2.

  In particular, the current consensus of the SASL community appears to
  be that SASL "security layers" (i.e., confidentiality and integrity
  protection of application data after authentication) are too complex
  and, since SASL applications tend to have an option to run over a
  Transport Layer Security (TLS) [RFC5246] channel, redundant and best
  replaced with channel binding.

Spencer (nit): it's a LONG way from "too complex" to "redundant" in
this sentence ;-) suggest moving "redundant" before the subclause,
just for readability.

Fixed.

3.3.  Examples

  The last step translate each decimal value using table 3 in Base32

Spencer (nit): s/translate/translates/?

Fixed.

  [RFC4648].  Thus the SASL mechanism name for the SPKM-1 GSSAPI
  mechanism is "GS2-DT4PIK22T6A".

8.  GSS-API Parameters

  The mutual_req_flag MUST be set.  If channel binding is used then the
  client MUST check that the corresponding ret_flag is set when the
  context is fully establish, else authentication MUST fail.

Spencer (nit): s/establish/established/

This paragraph was rewritten completely due to other changes, so this
doesn't apply any more.

  Use or non-use of deleg_req_flag and anon_req_flag is an
  implementation-specific detail.  SASL and GS2 implementors are
  encouraged to provide programming interfaces by which clients may
  choose to delegate credentials and by which servers may receive them.
  SASL and GS2 implementors are encouraged to provide programming
  interfaces which provide a good mapping of GSS-API naming options.

11.  GSS_Inquire_mech_for_SASLname call

  To allow SASL clients to more efficiently identify which GSS-API
  mechanism a particular SASL mechanism name refers to we specify a new
  GSS-API utility function for this purpose.

Spencer (nit): whew! hard to parse. Suggest "We specify a new GSS-API
utility function to allow SASL clients to more efficiently identify
the GSS-API mechanism that a particular SASL mechanism name refers
to", or something like that?

I agreed, and used your text after a s/clients/implementations/ -- it is
not SASL clients that may want to use the new GSS-API interface, but the
actual SASL GS2 implementations.  SASL clients (e.g., applications) will
(hopefully) never need to worry about the GSS-API interface.

13.3.  Additional Recommendations

  If the application requires security layers then it MUST prefer the
  SASL "GSSAPI" mechanism over "GS2-KRB5" or "GS2-KRB5-PLUS".

Spencer (minor): If "prefer the mechanism" is the right way to
describe this, I apologize, but I don't know what the MUST means in
practice - if this needs to be at MUST strength, I'd expect text like
"MUST use X and MUST NOT use Y or Z", or "MUST use X unless the server
doesn't support X".

Good point.  After reading the discussion between Nico and Alexey, I
changed the paragraph into:

   If the application requires SASL security layers then it MUST use the
   SASL "GSSAPI" mechanism [RFC4752] instead of "GS2-KRB5" or "GS2-KRB5-
   PLUS".

I believe this captures the intent more clearly.

14.  GSS-API Mechanisms that negotiate other mechanisms

  A GSS-API mechanism that negotiate other mechanisms interact badly

Spencer (nit): s/negotiate/negotiates/, and probably s/interact/will
interact/ ?

Fixed.

  with the SASL mechanism negotiation.  There are two problems.  The
  first is an interoperability problem and the second is a security
  concern.  The problems are described and resolved below.

14.1.  The interoperability problem

  If a client implement GSS-API mechanism X, potentially negotiated
  through a GSS-API mechanism Y, and the server also implement GSS-API

Spencer (nit): s/implement/implements/

Fixed.

  mechanism X negotiated through a GSS-API mechanism Z, the
  authentication negotiation will fail.

14.2.  Security problem

  If a client's policy is to first prefer GSSAPI mechanism X, then non-
  GSSAPI mechanism Y, then GSSAPI mechanism Z, and if a server supports
  mechanisms Y and Z but not X, then if the client attempts to
  negotiate mechanism X by using a GSS-API mechanism that negotiate

Spencer (nit): s/negotiate/negotiates/

Fixed.

  other mechanisms (such as SPNEGO), it may end up using mechanism Z

Spencer (nit): you provide a reference for SPNEGO in the next section,
but this is the first occurance...

I added a pointer to the first occurance too.

  when it ideally should have used mechanism Y. For this reason, the
  use of GSS-API mechanisms that negotiate other mechanisms are
  disallowed under GS2.

16.  Security Considerations

  GS2 does not directly use any cryptographic algorithms, therefore it
  is automatically "algorithm agile", or, as agile as the GSS-API
  mechanisms that are available for use in SASL applications via GS2.
  The exception is the use of SHA-1 for deriving SASL mechanism names,
  but no cryptographic properties are required.  The required property

Spencer (nit): I would suggest "SHA-1 is used to derive SASL mechanism
names, but no cryptographic properties are required" - the current
text says "we don't use crypto, except when we do" :-)

Right, it was a bit backward.  I used this instead:

   SHA-1 is used to derive SASL mechanism names, but no traditional
   cryptographic properties are required -- the required property is
   that the truncated output for distinct inputs are different for
   practical input values.  GS2 does not use any other cryptographic
   algorithm.  Therefor GS2 is "algorithm agile", or, as agile as the
   GSS-API mechanisms that are available for use in SASL applications
   via GS2.

/Simon
_______________________________________________
Ietf mailing list
Ietf(_at_)ietf(_dot_)org
https://www.ietf.org/mailman/listinfo/ietf

<Prev in Thread] Current Thread [Next in Thread>