Re: [MMUSIC] Roman Danyliw's Discuss on draft-ietf-mmusic-ice-sip-sdp-37: (with DISCUSS and COMMENT)

Hi Roman

 We just submitted version 39 addressing the remaining feedback

Please let us know your thoughts

Thanks
Suhas

On Tue, Aug 13, 2019 at 7:26 AM Roman Danyliw <rdd@cert.org> wrote:

> Hi Christer!
>
> > -----Original Message-----
> > From: Christer Holmberg [mailto:christer.holmberg@ericsson.com]
> > Sent: Tuesday, August 13, 2019 12:19 AM
> > To: Roman Danyliw <rdd@cert.org>; Adam Roach <adam@nostrum.com>;
> > The IESG <iesg@ietf.org>
> > Cc: fandreas@cisco.com; mmusic-chairs@ietf.org; mmusic@ietf.org; draft-
> > ietf-mmusic-ice-sip-sdp@ietf.org
> > Subject: VS: Roman Danyliw's Discuss on draft-ietf-mmusic-ice-sip-sdp-37:
> > (with DISCUSS and COMMENT)
> >
> > Hi Roman,
> >
> >
> >
> > >>> (4) Section 8.  Is there a reason why the security considerations
> > >>> from
> > >>> RFC8445 are not noted as also applying (e.g., Section 19.1 - .4.
> > >>>
> > >>> Would the addition of text at the top of section 8 that says "Please
> > >>> note that the security considerations from sections 19.1 through
> > >>> 19.4 of [RFC8445] also apply to this document." address your concern?
> > >>>
> > ...
> > >
> > > This pull request:
> > >
> > > https://protect2.fireeye.com/url?k=493cd979-15b60db8-493c99e2-
> > 863d9bcb
> > > 726f-
> > ec503dd28b20c537&q=1&u=https%3A%2F%2Fgithub.com%2FsuhasHere%2F
> > ice
> > > -sip-sdp%2Fpull%2F18%2Ffiles
> > >
> > > appears to have the following addition to address this issue but it
> didn't
> > make it to -38:
> > >
> > > <snip>
> > >      <t>
> > >        The generic ICE security considierations are defined in <xref
> > target="RFC8445"/>, and the
> > >        generic SDP offer/answer security considerations are defined in
> <xref
> > target="RFC3264"/>. The
> > >        security considerations also apply to implementations of this
> > document.
> > >      </t>
> > > </snip>
> > >
> > > Also, s/considierations/considerations/
> > >
> > > Can you please add in this text.
> >
> > Correct. Suhas had to do some "manual" work when merging the pull
> > requests, and the changes to the Security Considerations got lost for
> some
> > reason.
> >
> > There is now a new pull request, more or less a copy of the previous
> one, but
> > also with some privacy text (requested by Benjamin), that will be used
> for
> > the next version:
> >
> > https://github.com/suhasHere/ice-sip-sdp/pull/21
> >
> > Sorry for the confusion.
>
> That works for me!  Thank you.
>
> Roman
> _______________________________________________
> mmusic mailing list
> mmusic@ietf.org
> https://www.ietf.org/mailman/listinfo/mmusic
>