consul/agent/proxycfg
Matt Keeler 12acdd7481
Disable background cache refresh for Connect Leaf Certs
The rationale behind removing them is that all of our own code (xDS, builtin connect proxy) use the cache notification mechanism. This ensures that the blocking fetch behind the scenes is always executing. Therefore the only way you might go to get a certificate and have to wait is when 1) the request has never been made for that cert before or 2) you are using the v1/agent/connect/ca/leaf API for retrieving the cert yourself.

In the first case, the refresh change doesn’t alter the behavior. In the second case, it can be mitigated by using blocking queries with that API which just like normal cache notification mechanism will cause the blocking fetch to be initiated and to get leaf certs as soon as needed.

If you are not using blocking queries, or Envoy/xDS, or the builtin connect proxy but are retrieving the certs yourself then the HTTP endpoint might take a little longer to respond.

This also renames the RefreshTimeout field on the register options to QueryTimeout to more accurately reflect that it is used for any type that supports blocking queries.
2020-07-21 12:19:25 -04:00
..
manager.go Add service id context to the proxycfg logger 2020-05-18 09:08:05 -05:00
manager_test.go Fix a bunch of unparam lint issues 2020-06-24 13:00:14 -04:00
proxycfg.go
snapshot.go Enable gofmt simplify 2020-06-16 13:21:11 -04:00
state.go Merge pull request #8034 from hashicorp/dnephin/add-linter-staticcheck-4 2020-06-17 12:16:02 -04:00
state_test.go Enable gofmt simplify 2020-06-16 13:21:11 -04:00
testing.go Disable background cache refresh for Connect Leaf Certs 2020-07-21 12:19:25 -04:00