Sanity checks on dummy configs for integration tests
Open
area/quic
enhancement
no stalebot
- Dominant language
- C++
- Stars
- 28.9k
- Forks
- 5.6k
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 437
Description
This issue was discovered in #20259 which used MockClusterInfo for HTTP integration tests.
However the default value of the connect timeout would cause Quiche to complain and crash at [QuicConnection::SetNetworkTimeouts](https://github.com/google/quiche/blob/main/quic/core/quic_connection.cc#L4632).
I think default value like this doesn't make much sense.
I wonder if more of such cases exist in Envoy's integration test setups.
Contributor guide
Assessment
This issue has not been assessed yet.