- From: Peter Thatcher <pthatcher@google.com>
- Date: Mon, 28 Apr 2014 10:42:06 -0700
- To: Robin Raymond <robin@hookflash.com>
- Cc: "public-ortc@w3.org" <public-ortc@w3.org>
- Message-ID: <CAJrXDUGpfieg7sZD9KMZ3xsVqNHq0J0rgk6pSPUwaz=6LrjUcw@mail.gmail.com>
On Mon, Apr 28, 2014 at 10:33 AM, Robin Raymond <robin@hookflash.com> wrote: > > Thanks for a great summary. > > Yes, I agree those are the three options. I think I prefer option "a". > > Let me list my reasons: > > a. Makes sense to me, attach a new listener to transport or create a brand > new one and it forces a restart > > Wait... you're suggesting swapping the listener inside of the IceTransport? Hmmm... that would be an interesting approach that I hadn't considered. I guess that would be "option D". > b. I worry about the security implications, specifically if ice is still > being used for consent reasons. By not doing this we don't have to fear > that. > > > c. My only concern is that two RTCIceTransports could the same > RTCIceListener so calling a iceTransport1.restart() in that cause would > actually affect the other iceTransport2 indirectly because what's actually > being restarted is not the RTCIceTransport but the RTCIceListener which > both are attached. > If you called IceTransport.restart() on an IceTransport that came from IceListener, that would "detach" itself from the Listener. Why would you want to stay attached to the IceListener? > > > -Robin > > > Peter Thatcher <pthatcher@google.com> > April 28, 2014 at 1:11 PM > > > > On Sat, Apr 26, 2014 at 5:42 AM, Robin Raymond <robin@hookflash.com>wrote: > >> [...] > > > > We've chatted about this before. I think our 3 options were: > > a. Create a new IceTranport (or IceListener). This would require being > able to swap out the .transport of either the DtlsTransport or the > RtpSender/RtpReceiver. This seems elegant, but does add some > implementation complexity, and some burden to the JS. > > b. Allow the JS to set the ICE ufrag/pwd at any time. This would be > really simple, but has some security concerns. > > c. Add an IceTransport.restart() method that changes the ice/frag and > restarts the machine. This is pretty simple and doesn't have any security > concerns, and not much burden to the JS. > > I don't remember if we ever decided on one. I'd be happy with any of > them. > > >> >> >> -Robin >> >> >> >
Attachments
- image/jpeg attachment: compose-unknown-contact.jpg
Received on Monday, 28 April 2014 17:43:14 UTC