valyala/fasthttp

Recommendation on cleaning of HostClient.

Chiusa

#726 aperta il 12 gen 2020

 (2 commenti) (0 reazioni) (0 assegnatari)Go (1755 fork)batch import
help wantedquestion

Metriche repository

Star
 (21.741 stelle)
Metriche merge PR
 (Merge medio 3g) (44 PR mergiate in 30 g)

Descrizione

As of now the only way to override Host header is to use HostClient as per Issue 318. This means we have to manage reusing/cleanup of HostClients ourselves which fasthttp.Client provided out of the box.

We will be writing something very similar to fasthttp.Client to handle this. Is it ok to remove the HostClient if it is not being used from more than MaxIdleConnDuration*2 or something, as we are sure all goroutines for individual connection cleaning would have run? We don't have access to connsCount as it is an unexported field and was modified to use connsCount instead of time.

if t.Sub(v.LastUseTime()) > (MaxIdleConnDuration*2) {
				delete(m, k)
			}

If not, what is the recommendation here?

Guida contributor