Opened 6 years ago

Closed 6 years ago

#2046 closed defect (fixed)

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 (1)

comment:1 Changed 6 years ago by ming

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

In 5665:

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

Note: See TracTickets for help on using tickets.