Browse Source

code review changes

Matthew R. Kasun 3 years ago
parent
commit
79e093124a
2 changed files with 3 additions and 0 deletions
  1. 2 0
      controllers/node.go
  2. 1 0
      logic/nodes.go

+ 2 - 0
controllers/node.go

@@ -768,6 +768,7 @@ func isServer(node *models.Node) bool {
 func updateRelay(oldnode, newnode *models.Node) {
 func updateRelay(oldnode, newnode *models.Node) {
 	relay := logic.FindRelay(oldnode)
 	relay := logic.FindRelay(oldnode)
 	newrelay := relay
 	newrelay := relay
+	//check if node's address has been updated and if so, update the relayAddrs of the relay node with the updated address of the relayed node
 	if oldnode.Address != newnode.Address {
 	if oldnode.Address != newnode.Address {
 		for i, ip := range newrelay.RelayAddrs {
 		for i, ip := range newrelay.RelayAddrs {
 			if ip == oldnode.Address {
 			if ip == oldnode.Address {
@@ -776,6 +777,7 @@ func updateRelay(oldnode, newnode *models.Node) {
 			}
 			}
 		}
 		}
 	}
 	}
+	//check if node's address(v6) has been updated and if so, update the relayAddrs of the relay node with the updated address(v6) of the relayed node
 	if oldnode.Address6 != newnode.Address6 {
 	if oldnode.Address6 != newnode.Address6 {
 		for i, ip := range newrelay.RelayAddrs {
 		for i, ip := range newrelay.RelayAddrs {
 			if ip == oldnode.Address {
 			if ip == oldnode.Address {

+ 1 - 0
logic/nodes.go

@@ -682,6 +682,7 @@ func unsetHub(networkName string) error {
 	return nil
 	return nil
 }
 }
 
 
+// FindRelay - returns the node that is the relay for a relayed node
 func FindRelay(node *models.Node) *models.Node {
 func FindRelay(node *models.Node) *models.Node {
 	if node.IsRelayed == "no" {
 	if node.IsRelayed == "no" {
 		return nil
 		return nil