search

kyzer-davis / srtp-assurance-rfc-draft

A template repository for Internet-Draft management
0 stars 2 forks source link

Revise ANBF #15

Closed kyzer-davis closed 10 months ago

kyzer-davis commented 1 year ago

As per Christer:

srtp-context   = srtp-attr
                            srtp-tag
                            [srtp-ssrc";"]
                            [srtp-roc";"]
                            [srtp-seq";"]
                            [srtp-ext";"]

I don’t think this is what you want, because you would have to add “;” also after the last field.

Instead, you probably want something like:

srtp-context   = srtp-attr
                            srtp-tag
                            [srtp-param *(“;” srtp-param)]
srtp-param     = srtp-roc / srtp-seq / srtp-ext

and 

srtp-ext       = 1*VCHAR "=" (1*VCHAR / "unknown")
...
VCHAR          = %x21-7E

I don't think you want to allow ";" (%x3B), because then one may not know whether it is part of the srtp-ext value or a separator between values.

Finally, I also noticed I did not update the ABNF to include the new "list" of multiplexed SSRC/ROC/SEQ values with parenthesis delimiters.