Re: [sip-overload] draft-ietf-soc-overload-control-13

[Christer Holmberg <[e-mail]>]

Hi,

I don’t think the current ABNF allows a list without comma separation. Keep in mind that “algo-list” does not represent the list, only a single value in the list.

But, I understand your confusion, and in fact I think ”algo-list” is a little misleading. I think it would be better to call it “algo-value”, or something similar.

oc-algo     = "oc-algo" EQUAL DQUOTE algo-value *(COMMA algo-value)DQUOTE
algo-value   = "loss" / *(other-algo)
other-algo  = %x41-5A / %x61-7A / %x30-39

Regards,

Christer



Lähettäjä: [e-mail] [mailto:[e-mail]] Puolesta Yu, James
Lähetetty: 22. heinäkuuta 2013 23:12
Vastaanottaja: Janet P Gunn
Kopio: [e-mail]; [e-mail]; [e-mail]; [e-mail]; [e-mail]
Aihe: Re: [sip-overload] draft-ietf-soc-overload-control-13

Janet,

My comment is not on having multiple algorithms in the via header but on the syntax in the current I-D that could allow multiple algorithms that are not separated by “,”.  Please check the syntax by having “rate” and “foo”.

James

From: Janet P Gunn [mailto:[e-mail]]
Sent: Monday, July 22, 2013 3:54 PM
To: Yu, James
Cc: [e-mail]<mailto:[e-mail]>; [e-mail]<mailto:[e-mail]>; [e-mail]<mailto:[e-mail]>; [e-mail]<mailto:[e-mail]>; [e-mail]<mailto:[e-mail]>; [e-mail]<mailto:[e-mail]>
Subject: Re: [sip-overload] draft-ietf-soc-overload-control-13

There is at least one "other-algo", "rate".

When algo-list is included in the Via headers of a message sent by the client to the server, it may have multiple algorithms included in the algo-list - all the algorithms supported by the client.

When algo-list is included in the Via headers sent from the server to the client, it should have " one and only one algorithm" in the list.

Janet





From:        "Yu, James" <[e-mail]<mailto:[e-mail]>>
To:        "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>, "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>, "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>
Cc:        "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>, "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>
Date:        07/22/2013 03:19 PM
Subject:        [sip-overload]  draft-ietf-soc-overload-control-13
Sent by:        [e-mail]<mailto:[e-mail]>
________________________________



Hi, Henning, Vijay, Volker,

When I reviewed draft-ietf-soc-overload-rate-control-04.txt, I glanced through the syntax in this I-D to understand the oc related parameters.

When looking at “algo-list”, I wonder if there are two “other-algo” such as “abc” and “xyz”, ‘abc” and “xyz” won’t be comma-separated based on the oc-algo syntax.  It seems that algo-list would be

 algo-list = “loss“ / other-algo

This would ensure that one and only one algorithm is present in algo-list.  If there is no algorithm other than “loss”, *(COMMA algo-list) would take care of that.

Since algo-list contains a single algorithm, may be it is better to use “algo” only.  Or if algo-list contains a list of comma separated algorithms, then alog-list and oc-algo can be modified so that oc-algo has a single “algo-list”.  The latter may be better because the current syntax would allow the same algorithm to appear more than once.

James_______________________________________________
sip-overload mailing list
[e-mail]<mailto:[e-mail]>
https://www.ietf.org/mailman/listinfo/sip-overload
________________________________
No virus found in this message.
Checked by AVG - www.avg.com<http://www.avg.com>
Version: 2013.0.2904 / Virus Database: 3204/6510 - Release Date: 07/22/13