1485883 - SRTP extension using NSS extension handlers

Status: RESOLVED FIXED

(Core :: WebRTC: Networking, enhancement, P3)

Description

[Martin Thomson [:mt:]]

The implementation of the SRTP extension to DTLS in NSS isn't great.  It also means that you need to modify NSS to add a new SRTP cipher.  Because NSS doesn't need to know about SRTP, and because it has a new facility for adding extensions, this is a great opportunity to simplify some code.

Adding the code to TransportLayerDtls isn't completely trivial, but it's much cleaner writing this in C++.

Comment 1

[Martin Thomson [:mt:]]

Attachment: Bug 1485883 - Handle use_srtp extension in gecko, r?drno

This implements the SRTP extension in TransportLayerDtls.  My hope is
that we can expunge the SRTP code from NSS in a few releases.

Comment 2

[Martin Thomson [:mt:]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=3b1908c50a8ccf6e83090f53a98cc10a688f7963 seems fine so far.

Comment 3

[Nils Ohlmeier [:drno]]

Comment on attachment 9003697 [details]
Bug 1485883 - Handle use_srtp extension in gecko, r?drno

Nils Ohlmeier [:drno] has approved the revision.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2f345fa70dc0
Handle use_srtp extension in gecko, r=drno

Comment 5

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/2f345fa70dc0