From c8de34889a584e7aae363272040eb49e6d516878 Mon Sep 17 00:00:00 2001
From: jbhusson <jonah@elixxir.io>
Date: Thu, 5 May 2022 17:20:29 -0400
Subject: [PATCH] Comments

---
 broadcast/symmetricClient.go | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/broadcast/symmetricClient.go b/broadcast/symmetricClient.go
index 3b5b288df..045b7976b 100644
--- a/broadcast/symmetricClient.go
+++ b/broadcast/symmetricClient.go
@@ -148,6 +148,8 @@ func (s *symmetricClient) Stop() {
 	s.net.DeleteClientService(s.channel.ReceptionID)
 }
 
+// verifyID generates a symmetric ID based on the info in the channel & compares it to the one passed in
+// TODO: it seems very odd to me that we do this, rather than just making the ID a private/ephemeral component like the key
 func (s *symmetricClient) verifyID() bool {
 	gen, err := crypto.NewSymmetricID(s.channel.Name, s.channel.Description, s.channel.Salt, rsa.CreatePublicKeyPem(s.channel.RsaPubKey))
 	if err != nil {
-- 
GitLab