Commit682467e

committed

add comment on idle conn tuning

1 parent698e697 commit682467eCopy full SHA for 682467e

File tree

-1

lines changed

-1

lines changed

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -99,8 +99,14 @@ func NewServerTailnet(`
`99`	`99`	`transport:tailnetTransport.Clone(),`
`100`	`100`	`}`
`101`	`101`	`tn.transport.DialContext=tn.dialContext`
`102`		`-tn.transport.MaxIdleConnsPerHost=10`
	`102`	`+// These options are mostly just picked at random, and they can likely be`
	`103`	`+// fine tuned further. Generally, users are running applications in dev mode`
	`104`	`+// which can generate hundreds of requests per page load, so we increased`
	`105`	`+// MaxIdleConnsPerHost from 2 to 6 and removed the limit of total idle`
	`106`	`+// conns.`
	`107`	`+tn.transport.MaxIdleConnsPerHost=6`
`103`	`108`	`tn.transport.MaxIdleConns=0`
	`109`	`+tn.transport.IdleConnTimeout=10*time.Minute`
`104`	`110`	`// We intentionally don't verify the certificate chain here.`
`105`	`111`	`// The connection to the workspace is already established and most`
`106`	`112`	`// apps are already going to be accessed over plain HTTP, this config`

Comments

(0)