Re: time-stamp-draft-7

[Peter Sylvester <Peter.Sylvester@xxxxxxxxxx>]

> 
> I suggest changing the tagging to:
> 
> TSTInfo ::= SEQUENCE  {
>     [...]
>      tsa                          [0] GeneralName      OPTIONAL,
>      extensions                   [1] Extensions       OPTIONAL
> }
> 

Another way is to change the order of some precding element. Idoubt that
one of the choices of GeneralName will ever become a sequence directly.

TSTInfo ::= SEQUENCE  {
     version                      Integer  { v1(1) },
     policy                       PolicyInformation,
     messageImprint               MessageImprint,
       -- MUST have the same value as the similar field in 
       -- TimeStampReq
     genTime                      GeneralizedTime,
     accuracy                     Accuracy             OPTIONAL,
     serialNumber                 Integer,
     nonce                        Integer              OPTIONAL,
       -- MUST be present if the similar field was present 
       -- in TimeStampReq. In that case it must have the same value.
     tsa                          GeneralName          OPTIONAL,

     extensions                   Extensions       OPTIONAL
}

And in order to follow the 'MIMIMAL' tagging I suggest the following

TimeStampReq ::= SEQUENCE  {
     version                      Integer  { v1(1) },
     messageImprint               MessageImprint,
       --a hash algorithm OID and the hash value of the data to be
       --time stamped
     reqPolicy                PolicyInformation      OPTIONAL,
     nonce                    Integer                OPTIONAL,
     certReq                  BOOLEAN                DEFAULT FALSE,
     extensions               [0] IMPLICIT Extensions OPTIONAL
}

in order to get minimal tagging and to avoid any possible conflict
when some interpretes it a EXPLICIT.