Fix AddRemoteCandidate to accept nil candidate

empty candidate.Candidate won't make an error when pass into pion/ice

agent.go

@@ -706,7 +706,12 @@ func (a *Agent) checkKeepalive() {
 
 // AddRemoteCandidate adds a new remote candidate
 func (a *Agent) AddRemoteCandidate(c Candidate) error {
-	// canot check for network yet because it might not be applied
+	// accept nil candidate. aka end-of-candidates will not fail
+	if c == nil {
+		return nil
+	}
+
+	// cannot check for network yet because it might not be applied
 	// when mDNS hostame is used.
 	if c.TCPType() == TCPTypeActive {
 		// TCP Candidates with tcptype active will probe server passive ones, so

agent_test.go

@@ -1686,3 +1686,14 @@ func TestLiteLifecycle(t *testing.T) {
 	<-bFailed
 	assert.NoError(t, bAgent.Close())
 }
+
+func TestNilCandidate(t *testing.T) {
+	var config AgentConfig
+	a, err := NewAgent(&config)
+	if err != nil {
+		t.Fatalf("Error constructing ice.Agent: %v", err)
+	}
+
+	err = a.AddRemoteCandidate(nil)
+	assert.NoError(t, err)
+}