Re: [MMUSIC] ICE-SIP-SDP: Concluding ICE statement

Hi Christer
   Roman has been submitting PRs here in the past. Please feel free to
submit the PR and thanks for starting the discussion.

https://github.com/suhasHere/ice-sip-sdp/blob/master/ice_sip_sdp.coriander

Cheers
Suhas

On Wed, Jul 24, 2019 at 2:33 AM Christer Holmberg <
christer.holmberg@ericsson.com> wrote:

> Hi,
>
>
>
> It would be really good if the draft was on GitHub. Then one could submit
> pull requests with the new text, and one will be able to better follow what
> the changes in the next version of the draft will be.
>
>
>
> Because, this is not the only issue I raised. I also had comments
> regarding the ice-option and ice-pacing attribute.
>
>
>
> Regards,
>
>
>
> Christer
>
>
>
> *From: *mmusic <mmusic-bounces@ietf.org> on behalf of Roman Shpount <
> roman@telurix.com>
> *Date: *Wednesday, 24 July 2019 at 1.41
> *To: *Christer Holmberg <christer.holmberg@ericsson.com>
> *Cc: *"mmusic-chairs@ietf.org" <mmusic-chairs@ietf.org>, "mmusic@ietf.org"
> <mmusic@ietf.org>
> *Subject: *Re: [MMUSIC] ICE-SIP-SDP: Concluding ICE statement
>
>
>
> On Tue, Jul 23, 2019 at 6:33 PM Christer Holmberg <
> christer.holmberg@ericsson.com> wrote:
>
> Once the agent has successfully nominated a pair, the state of the check
> list associated with the pair is set to Completed.. Once the state of
> each check list is set to either Completed or Failed, for each Completed
> check list the agent checks whether the nominated pair matches the
> default candidate pair. If there are one or more pairs that don't match,
> and the peer did not indicate support for the 'ice2' ice-option, the
> controlling
> agent MUST generate a subsequent offer, in which the connection address,
> port and transport protocol in the "c=" and "m=" lines
> associated with each data stream match the corresponding local information
> of the nominated pair for that data stream (Section 3.4.1.2.2.).
> Furthermore, if there are one or more check lists with the state set to
> Failed, the controlling agent MUST generate a subsequent offer in order to
> remove
> the associated data streams by setting the port value of the data streams
> to zero (Section 3.4.1.1.2.), even if  the peer did indicate support for
> the 'ice2' ice-option.
>
>
>
> This looks good to me too. I have copied Suhas so that he can review and
> update draft-ietf-mmusic-ice-sip-sdp section 3.3.4 (
> https://tools.ietf.org/html/draft-ietf-mmusic-ice-sip-sdp-36#section-3.3.4
> )
>
>
>
> Best Regards,
>
> _____________
> Roman Shpount
>
>
> _______________________________________________
> mmusic mailing list
> mmusic@ietf.org
> https://www.ietf.org/mailman/listinfo/mmusic
>