From f3fa47ce89d24277f0da1f2342fe33a13b51cf7c Mon Sep 17 00:00:00 2001 From: Sean DuBois Date: Sat, 29 Sep 2018 01:19:12 -0700 Subject: [PATCH] Remove IceControlled/Priority from ICE keepalive IceControlled is incorrect if agent is controlling, instead of fixing remove instead. We don't need these two values in the keepalive. Resolves #187 --- pkg/ice/agent.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/pkg/ice/agent.go b/pkg/ice/agent.go index d3385dae..e8a6f634 100644 --- a/pkg/ice/agent.go +++ b/pkg/ice/agent.go @@ -172,8 +172,6 @@ func (a *Agent) pingCandidate(local, remote Candidate) { func (a *Agent) keepaliveCandidate(local, remote Candidate) { msg, err := stun.Build(stun.ClassIndication, stun.MethodBinding, stun.GenerateTransactionId(), &stun.Username{Username: a.remoteUfrag + ":" + a.LocalUfrag}, - &stun.IceControlled{TieBreaker: a.tieBreaker}, - &stun.Priority{Priority: uint32(local.GetBase().Priority(HostCandidatePreference, 1))}, &stun.MessageIntegrity{ Key: []byte(a.remotePwd), },