Remove code complexity when adding relay server

This commit is contained in:
Devon Hudson 2023-01-12 11:35:54 -07:00
parent 7fb194fc61
commit e54e036948
No known key found for this signature in database
GPG key ID: CD06B18E77F6A628

View file

@ -317,18 +317,20 @@ func (s *ServerStatistics) AddRelayServers(relayServers []gomatrixserverlib.Serv
} }
err := s.statistics.DB.AddRelayServersForServer(s.serverName, uniqueList) err := s.statistics.DB.AddRelayServersForServer(s.serverName, uniqueList)
if err == nil { if err != nil {
logrus.WithError(err).Errorf("Failed to add relay servers for %q. Servers: %v", s.serverName, uniqueList)
return
}
for _, newServer := range uniqueList { for _, newServer := range uniqueList {
alreadyKnown := false alreadyKnown := false
for _, srv := range s.knownRelayServers { for _, srv := range s.knownRelayServers {
if srv == newServer { if srv == newServer {
alreadyKnown = true alreadyKnown = true
}
}
if !alreadyKnown {
s.knownRelayServers = append(s.knownRelayServers, newServer)
} }
} }
if !alreadyKnown {
s.knownRelayServers = append(s.knownRelayServers, newServer)
}
} }
} }