Ticket #2046 (closed defect: fixed)

Opened 2 months ago

Last modified 2 months ago

ICE: Use STUN FINGERPRINT attribute when sending keepalives

Reported by: ming Owned by: bennylp
Priority: normal Milestone: release-2.7.1
Component: pjnath Version: trunk
Keywords: Cc:
Backport to 1.x milestone: Backported: no

Description

Last time, during our implementation, we relied on the RFC 5245, which was still in draft stage, which said:

The Indication MUST NOT utilize
   any authentication mechanism, and SHOULD NOT contain any attributes.

However, in the published version:

The Indication MUST NOT utilize any authentication
   mechanism.  It SHOULD contain the FINGERPRINT attribute to aid in
   demultiplexing, but SHOULD NOT contain any other attributes.

So we need to update our implementation.

Thanks to Sean Bright for the patch and George Joseph for the report.

Change History

comment:1 Changed 2 months ago by ming

  • Status changed from new to closed
  • Resolution set to fixed

In 5665:

Fixed #2046: ICE: Use STUN FINGERPRINT attribute when sending keepalives

Note: See TracTickets for help on using tickets.