miracle091/dnscrypt-proxy
miracle091
/
dnscrypt-proxy
mirror of https://github.com/DNSCrypt/dnscrypt-proxy.git
1
0
Fork 0
Code Issues Releases Wiki Activity

fix: serversInfo: use lb strategy always even with estimator 

I don't see why estimator should select which candidate to use.
This commit is contained in:
Markus Linnala 2019-10-20 18:17:21 +03:00 committed by Frank Denis
parent bc9a37e550
commit 1b4f873026
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
dnscrypt-proxy/serversInfo.go
View File

@ -157,7 +157,11 @@ func (serversInfo *ServersInfo) refresh(proxy *Proxy) (int, error) {
return liveServers, err
}
func (serversInfo *ServersInfo) estimatorUpdate(candidate int) {
func (serversInfo *ServersInfo) estimatorUpdate() {
candidate := rand.Intn(len(serversInfo.inner))
if candidate == 0 {
return
}
candidateRtt, currentBestRtt := serversInfo.inner[candidate].rtt.Value(), serversInfo.inner[0].rtt.Value()
if currentBestRtt < 0 {
currentBestRtt = candidateRtt
@ -193,11 +197,7 @@ func (serversInfo *ServersInfo) getOne() *ServerInfo {
return nil
}
if serversInfo.lbEstimator {
candidate := rand.Intn(serversCount)
if candidate == 0 {
return serversInfo.inner[candidate]
}
serversInfo.estimatorUpdate(candidate)
serversInfo.estimatorUpdate()
}
var candidate int
switch serversInfo.lbStrategy {