Re: [Insipid] Review of draft-ietf-insipid-logme-reqs-02

Hi Peter,

See inline.

Q2 (editorial/technical):

>> I don't understand REQ4, saying "SIP entities SHOULD log SIP requests 
>> or responses with a "log me" marker."
>> 
>> Also, in e.g. REQ6 it is then said that proxies MAY insert the marker, 
>> which seems to contradict.
>> 
> REQ4 could be re-written slightly more clearly as "SIP entities SHOULD log SIP requests 
> or responses that contain a "log me" marker." The SIP entity checks for the presence of 
> a "log me" marker and writes any request or response that contains a "log me" marker 
> to a log file. 

I think it would be good to say "SIP entities that support the mechanism SHOULD...", to make it clear that we don't expect certain behaviour from SIP entities that do not support the mechanism. Some people may say it's obvious, but I think it's good to have it explicit :)

> REQ 6 allows a SIP proxy to insert a "log me" marker to allow the mechanism to work even if the UE 
> does not support this draft, so this talks about inserting a "log me" marker and not checking for its presence. 

Got it. Thanks for the clarification.

Q3 (editorial/technical):

>> Regarding REQ5, does that also apply to UAs that do NOT support the 
>> mechanism? If so, I think that should be explicitly stated, because it 
>> will have big impact on the technical solution.
>> 
> UEs that do not support the mechanism are not expected to echo the "log me" marker because
> they don't know about it. However, if the "log me" marker solution is a header field parameter 
> of the Session-ID header field, as currently proposed, then I guess that even UEs that do not 
> support inserting a "log me" marker will echo it if it echoes the whole Session-ID header field. 

Sure, but that is irrelevant as far as the requirements are concerned :)

So, if we don't REQUIRE anything from UAs that don't support the mechanism, I again think it's good to explicitly talk about UAs that support the mechanism.

Regards,

Christer

> -----Original Message-----
> From: insipid [mailto:insipid-bounces@ietf.org] On Behalf Of Georg 
> Mayer
> Sent: 8. kesäkuuta 2015 12:42
> To: insipid@ietf.org
> Cc: Peter.Dawes@vodafone.com
> Subject: [Insipid] Review of draft-ietf-insipid-logme-reqs-02
> 
> Hello,
> 
> I finally managed to review draft-ietf-insipid-logme-reqs-02. I have 
> no technical comments on it and support all the requirements therein. 
> In my view the draft can progress as is.
> 
> Cheers,
> Georg
> 
> 
> _______________________________________________
> insipid mailing list
> insipid@ietf.org
> https://www.ietf.org/mailman/listinfo/insipid