[ietf-nntp] Re: AUTHINFO/SASL responses

[Clive D.W. Feather]

Russ Allbery said:
>> 1   AUTHINFO JUNK
>> 2   AUTHINFO SASL INVALID
>> 3   AUTHINFO SASL VALID JunkBase64
>> 4   AUTHINFO SASL VALID Base64notNeeded=

> Actually, 1 and 2 are identical cases as far as I'm concerned; they're
> both unsupported subcommands (501).

Okay; I'll live with that.

> 3 is a syntax error, yes.  Unfortunately, we use the same error code for
> syntax errors and unsupported subcommands.

Mumble. That's historical, and we don't have to perpetuate it when it's
beneficial to do otherwise. Compare the discussion on 281 and 283. I
think this one case of bad base64-encoding is worth keeping distinct.

> 4 should definitely be a different code.

Right.

-- 
Clive D.W. Feather  | Work:  <clive at demon.net>   | Tel:    +44 20 8495 6138
Internet Expert     | Home:  <clive at davros.org>  | Fax:    +44 870 051 9937
Demon Internet      | WWW: http://www.davros.org | Mobile: +44 7973 377646
Thus plc            |                            |