commit | e0086c0fee7fc134b70a817dbbfc6170c2d0778d | [log] [tgz] |
---|---|---|
author | Mike Yu <yumike@google.com> | Sat May 29 12:37:04 2021 +0800 |
committer | Mike Yu <yumike@google.com> | Tue Jun 08 08:40:23 2021 +0000 |
tree | 2bb84faef8d6ad36fbf9f50f5108b84890f3ae0f | |
parent | b977d1ac2ffb365f3b955651e3941e3f92164a0a [diff] |
Clean up DnsTlsDispatcher when a network is disconnected DnsTlsDispatcher used to keep Transport in the map for at least 5 minutes. Since Transport is no longer a stateless object, it needs to be removed as soon as the associated network, testing network particularly, is disconnected. This change actually fixes two known test bugs: - bug 189384775. How to reproduce it: run TlsServerRevalidation twice. $ ./resolv_integration_test64 --gtest_filter="*TlsServerRevalidation" $ ./resolv_integration_test64 --gtest_filter="*TlsServerRevalidation" - bug 189132684 How to reproduce it: run ConnectTlsServerTimeout and QueryTlsServerTimeout before before ConnectTlsServerTimeout_ConcurrentQueries. $ ./resolv_integration_test64 --gtest_filter="*ConnectTlsServerTimeout:*QueryTlsServerTimeout" $ ./resolv_integration_test64 --gtest_filter="*ConnectTlsServerTimeout_ConcurrentQueries" Bug: 189384775 Bug: 189132684 Bug: 189161918 Test: run atest on R platform with the dnsresolver built from master Test: run atest on Q platform with the dnsresolver built from mainline-prod Test: verified that the two mentioned bugs are not reproducible Test: run resolv_integration_test twice Change-Id: I3bf3b7ddec7818c4fcf3ffaa0f97173d876d3642
This code uses LOG(X) for logging. Log levels are VERBOSE,DEBUG,INFO,WARNING and ERROR. The default setting is WARNING and logs relate to WARNING and ERROR will be shown. If you want to enable the DEBUG level logs, using following command. adb shell service call dnsresolver 10 i32 1 VERBOSE 0 DEBUG 1 INFO 2 WARNING 3 ERROR 4 Verbose resolver logs could contain PII -- do NOT enable in production builds.