Gitiles
Code Review Sign In
review.blissroms.org / platform_packages_modules_DnsResolver / 166f60b45614dd6e20f6a078e455920672a8a8d5
commit166f60b45614dd6e20f6a078e455920672a8a8d5[log] [tgz]
authorMike Yu <yumike@google.com>Mon Jan 17 18:41:41 2022 +0800
committerMike Yu <yumike@google.com>Mon Jan 24 09:51:13 2022 +0000
tree3c60f731f1c8ba764a3608e5a71fd9c5c36b0c46
parent995c75fcb1b70581f2bad1f58b4f5d73b65fc113 [diff]
DoH: Enable session resumption

When a connection is closed gracefully, the connection's session
will be stored in the `Connection`. The `Network` that owns the
`Connection` will attempt to reuse the session on next connection.

This change should improve the performance in terms of reducing
traffic resources.

The example log of a reconnection to dns.google (w/o session resumption):
15:42:47.431  1017  1095 D doh::connection::driver: Sent 1200 bytes on network 100
15:42:47.454  1017  1095 D doh::connection::driver: Received 1200 bytes on network 100
15:42:47.455  1017  1095 D doh::connection::driver: Sent 1200 bytes on network 100
15:42:47.455  1017  1095 D doh::connection::driver: Received 1200 bytes on network 100
15:42:47.456  1017  1095 D doh::connection::driver: Sent 59 bytes on network 100
15:42:47.456  1017  1095 D doh::connection::driver: Received 1200 bytes on network 100
15:42:47.457  1017  1095 D doh::connection::driver: Sent 59 bytes on network 100
15:42:47.458  1017  1095 D doh::connection::driver: Received 1200 bytes on network 100
15:42:47.458  1017  1095 D doh::connection::driver: Sent 59 bytes on network 100
15:42:47.460  1017  1095 D doh::connection::driver: Received 1200 bytes on network 100
15:42:47.461  1017  1095 D doh::connection::driver: Sent 60 bytes on network 100
15:42:47.469  1017  1095 D doh::connection::driver: Received 397 bytes on network 100
15:42:47.469  1017  1095 D doh::connection::driver: Sent 132 bytes on network 100
15:42:47.470  1017  1095 D doh::connection::driver: Handling DNS request on network 100, ...

The example log of a reconnection to dns.google (w/ session resumption)
15:33:18.702  1021  1066 D doh::connection::driver: Sent 1200 bytes on network 101
15:33:18.756  1021  1066 D doh::connection::driver: Received 1200 bytes on network 101
15:33:18.757  1021  1066 D doh::connection::driver: Sent 1200 bytes on network 101
15:33:18.758  1021  1066 D doh::connection::driver: Handling DNS request on network 101, ...

Bug: 205922811
Test: cd packages/modules/DnsResolver && atest
Test: manually tested every non-testing DoH provider listed
      in mAvailableDoHProviders
Change-Id: I5ec02970d03a40a7fea05688e9a42fa50ceb4859
11 files changed
tree: 3c60f731f1c8ba764a3608e5a71fd9c5c36b0c46
  1. aidl_api/
  2. apex/
  3. binder/
  4. doh/
  5. include/
  6. tests/
  7. .clang-format ⇨ ../../../build/soong/scripts/system-clang-format
  8. rustfmt.toml ⇨ ../../../build/soong/scripts/rustfmt.toml
  9. .editorconfig
  10. Android.bp
  11. cbindgen.toml
  12. Dns64Configuration.cpp
  13. Dns64Configuration.h
  14. DnsProxyListener.cpp
  15. DnsProxyListener.h
  16. DnsQueryLog.cpp
  17. DnsQueryLog.h
  18. DnsQueryLogTest.cpp
  19. DnsResolver.cpp
  20. DnsResolver.h
  21. DnsResolverService.cpp
  22. DnsResolverService.h
  23. DnsStats.cpp
  24. DnsStats.h
  25. DnsStatsTest.cpp
  26. DnsTlsDispatcher.cpp
  27. DnsTlsDispatcher.h
  28. DnsTlsQueryMap.cpp
  29. DnsTlsQueryMap.h
  30. DnsTlsServer.cpp
  31. DnsTlsServer.h
  32. DnsTlsSessionCache.cpp
  33. DnsTlsSessionCache.h
  34. DnsTlsSocket.cpp
  35. DnsTlsSocket.h
  36. DnsTlsSocketFactory.h
  37. DnsTlsTransport.cpp
  38. DnsTlsTransport.h
  39. doh.h
  40. Experiments.cpp
  41. Experiments.h
  42. ExperimentsTest.cpp
  43. getaddrinfo.cpp
  44. getaddrinfo.h
  45. gethnamaddr.cpp
  46. gethnamaddr.h
  47. hostent.h
  48. IDnsTlsSocket.h
  49. IDnsTlsSocketFactory.h
  50. IDnsTlsSocketObserver.h
  51. IPrivateDnsServer.h
  52. libnetd_resolv.map.txt
  53. LockedQueue.h
  54. NOTICE
  55. OperationLimiter.h
  56. OperationLimiterTest.cpp
  57. OWNERS
  58. params.h
  59. PREUPLOAD.cfg
  60. PrivateDnsCommon.h
  61. PrivateDnsConfiguration.cpp
  62. PrivateDnsConfiguration.h
  63. PrivateDnsConfigurationTest.cpp
  64. PrivateDnsValidationObserver.h
  65. README-DoT.md
  66. README.md
  67. res_cache.cpp
  68. res_comp.cpp
  69. res_comp.h
  70. res_debug.cpp
  71. res_debug.h
  72. res_mkquery.cpp
  73. res_query.cpp
  74. res_send.cpp
  75. res_send.h
  76. res_stats.cpp
  77. resolv_cache.h
  78. resolv_private.h
  79. resolv_rust_test_config_template.xml
  80. resolv_test_config_template.xml
  81. ResolverController.cpp
  82. ResolverController.h
  83. ResolverEventReporter.cpp
  84. ResolverEventReporter.h
  85. ResolverStats.h
  86. sethostent.cpp
  87. stats.h
  88. stats.proto
  89. TEST_MAPPING
  90. util.cpp
  91. util.h
README.md

Logging

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.