platform_packages_modules_DnsResolver/PrivateDnsConfiguration.cpp, branch master Unnamed repository; edit this file 'description' to name the repository. DoH: Refactor FFI 2021-10-05T23:52:41+00:00 Matthew Maurer mmaurer@google.com 2021-09-17T00:08:51+00:00 6b17842f445b68b075f247b841285a36c502adda * Factor out constants to level enum conversion * Document property that doh_init_logger will have no effect on future calls * Change `doh_set_log_level` to default to `Error` logging on an invalid value rather than disabling all logging. This matches the behavior of `doh_init_logger`. * Prefix constants with `DOH_`. `cbindgen`-exported constants are not in a namespace; they are raw C constants. Prepending the library name will help avoid future collisions. * `android_logger::init_once` explicitly states in the documentation that calling it more than once is safe, and will simply not change the logger on future calls. Bug: 202081046 Change-Id: I91a8a7f40363ee2dcf6211fb595e1c6e7f6084a8 
* Factor out constants to level enum conversion
* Document property that doh_init_logger will have no effect on future calls
* Change `doh_set_log_level` to default to `Error` logging on an invalid
  value rather than disabling all logging. This matches the behavior of
  `doh_init_logger`.
* Prefix constants with `DOH_`. `cbindgen`-exported constants are not in
  a namespace; they are raw C constants. Prepending the library name
  will help avoid future collisions.
* `android_logger::init_once` explicitly states in the documentation that
  calling it more than once is safe, and will simply not change the logger
  on future calls.

Bug: 202081046
Change-Id: I91a8a7f40363ee2dcf6211fb595e1c6e7f6084a8
Cleanup DoH statistics when DoH is not supported in the private DNS mode. 2021-10-01T15:10:46+00:00 waynema waynema@google.com 2021-10-01T06:56:34+00:00 ae5fe72c344017e4fecfef661ba4391dfde89e97 Test steps: a. Default enable strict mode. b. Turn on WiFi and Mobile data. c. Swtich private DNS mode from strict mode to automatic mode. d. Expect to see DoH statistics only on WiFi. e. Switch private DNS mode from automatic mode to off mode. f. Expect to see no DoH statistics showing up on both WiFi and Mobile data. Tests will be done in a follow-up CL. Bug: 201735918 Test: Check output in dumpsys dnsresolver Change-Id: Ie4e67c3967778ada480cb2b5f9222cde5d2c705e 
Test steps:
a. Default enable strict mode.
b. Turn on WiFi and Mobile data.
c. Swtich private DNS mode from strict mode to automatic mode.
d. Expect to see DoH statistics only on WiFi.
e. Switch private DNS mode from automatic mode to off mode.
f. Expect to see no DoH statistics showing up on both WiFi and Mobile
data.

Tests will be done in a follow-up CL.

Bug: 201735918
Test: Check output in dumpsys dnsresolver
Change-Id: Ie4e67c3967778ada480cb2b5f9222cde5d2c705e
Tags the socket for DoH3 2021-09-29T04:56:39+00:00 Luke Huang huangluke@google.com 2021-09-29T04:11:30+00:00 10eed9da6e8470205a517e732b010d4188e1173a Test: atest Bug: 200807019 Change-Id: I7ec6f7100e2f8a511ea2c03933637af09a6e67a1 
Test: atest
Bug: 200807019

Change-Id: I7ec6f7100e2f8a511ea2c03933637af09a6e67a1
Merge "DoH statistics is now available from DnsStats and is shown in dumpsys dnsresolver." 2021-09-28T08:51:55+00:00 Treehugger Robot treehugger-gerrit@google.com 2021-09-28T08:51:55+00:00 7821d2fa75f4c875bd67d28e954a5a9a33483ebe 






DoH statistics is now available from DnsStats and is shown in dumpsys dnsresolver.
2021-09-28T06:28:02+00:00

waynema
waynema@google.com

2021-09-13T08:28:12+00:00

356e4c60bc6dd3a318a54911b81a2654a1c39a02

Example of output from dumpsys dnsresolver:

  Server statistics: (total, RTT avg, {rcode:counts}, last update)
    over UDP
      8.8.4.4 (3, 2503ms, [TIMEOUT:3 ], 465s) score{71.0}
      8.8.8.8 (4, 1880ms, [NOERROR:1 TIMEOUT:3 ], 468s) score{77.3}
      2001:4860:4860::8844 (9, 15ms, [NOERROR:9 ], 78s) score{99.8}
      2001:4860:4860::8888 <no data> score{100.0}
    over DOH
      2001:4860:4860::8888 (89, 30ms, [NOERROR:89 ], 127s) score{99.7}
    over TLS
      8.8.4.4 <no data> score{100.0}
      8.8.8.8 <no data> score{100.0}
      2001:4860:4860::8844 <no data> score{100.0}
      2001:4860:4860::8888 <no data> score{100.0}
    over TCP
      8.8.4.4 <no data> score{100.0}
      8.8.8.8 <no data> score{100.0}
      2001:4860:4860::8844 <no data> score{100.0}
      2001:4860:4860::8888 <no data> score{100.0}
    over MDNS
      224.0.0.251 <no data> score{100.0}
      ff02::fb <no data> score{100.0}
  TC mode: default
  TransportType: WIFI

Test steps:
a. Default enable strict mode.
b. Turn on WiFi.
c. Turn on Mobile data.
d. Check the output from dumpsys dnsresolver.
e. Expect to see two different statistics with the same DNS server on different NetId.

NetId: 101
    over DOH
      2001:4860:4860::8888 (90, 29ms, [NOERROR:90 ], 7s) score{99.7}
  TransportType: WIFI

NetId: 102
    over DOH
      2001:4860:4860::8888 (2, 33ms, [NOERROR:2 ], 74s) score{99.7}
  TransportType: CELLULAR

Bug: 195092631
Test: checked output in dumpsys dnsresolver
Change-Id: I65c69a850bce352839314833d44c9f165c7ca142





Example of output from dumpsys dnsresolver:

  Server statistics: (total, RTT avg, {rcode:counts}, last update)
    over UDP
      8.8.4.4 (3, 2503ms, [TIMEOUT:3 ], 465s) score{71.0}
      8.8.8.8 (4, 1880ms, [NOERROR:1 TIMEOUT:3 ], 468s) score{77.3}
      2001:4860:4860::8844 (9, 15ms, [NOERROR:9 ], 78s) score{99.8}
      2001:4860:4860::8888 <no data> score{100.0}
    over DOH
      2001:4860:4860::8888 (89, 30ms, [NOERROR:89 ], 127s) score{99.7}
    over TLS
      8.8.4.4 <no data> score{100.0}
      8.8.8.8 <no data> score{100.0}
      2001:4860:4860::8844 <no data> score{100.0}
      2001:4860:4860::8888 <no data> score{100.0}
    over TCP
      8.8.4.4 <no data> score{100.0}
      8.8.8.8 <no data> score{100.0}
      2001:4860:4860::8844 <no data> score{100.0}
      2001:4860:4860::8888 <no data> score{100.0}
    over MDNS
      224.0.0.251 <no data> score{100.0}
      ff02::fb <no data> score{100.0}
  TC mode: default
  TransportType: WIFI

Test steps:
a. Default enable strict mode.
b. Turn on WiFi.
c. Turn on Mobile data.
d. Check the output from dumpsys dnsresolver.
e. Expect to see two different statistics with the same DNS server on different NetId.

NetId: 101
    over DOH
      2001:4860:4860::8888 (90, 29ms, [NOERROR:90 ], 7s) score{99.7}
  TransportType: WIFI

NetId: 102
    over DOH
      2001:4860:4860::8888 (2, 33ms, [NOERROR:2 ], 74s) score{99.7}
  TransportType: CELLULAR

Bug: 195092631
Test: checked output in dumpsys dnsresolver
Change-Id: I65c69a850bce352839314833d44c9f165c7ca142







Fix the bug while updating DoH servers
2021-09-23T06:00:38+00:00

Luke Huang
huangluke@google.com

2021-09-22T10:25:01+00:00

072497e6e4652ff27dfc1ab8cc943ffb154556e1

Currently, DoH servers won't be updated correctly when the requested
private DNS server is empty or not in available DoH server list.

Bug: 200786579
Test: atest
Change-Id: Ic6b34e0173ac91924cde14512b1156c0e3fc2e23





Currently, DoH servers won't be updated correctly when the requested
private DNS server is empty or not in available DoH server list.

Bug: 200786579
Test: atest
Change-Id: Ic6b34e0173ac91924cde14512b1156c0e3fc2e23







Report private DNS protocols in onPrivateDnsValidationEvent
2021-09-10T09:57:08+00:00

Mike Yu
yumike@google.com

2021-09-09T14:21:09+00:00

03e4903f3759ff6a5e1060bcb04e3f9e7a23e31d

PrivateDnsDohTest.QueryFailover fails in waiting for the second
onPrivateDnsValidationEvent if DoT validation result and DoH
validation result are reported at the same time.

To fix the flaky test, this change makes the validation results
distinguishable by private DNS protocols.

Bug: 198111329
Test: PrivateDnsDohTest.QueryFailover 200 times passed on cuttlefish
Test: PrivateDnsDohTest.ValidationFail 500 times passed on cuttlefish
Change-Id: I8649b21bff176c6f2a1158f04812c8c8e96ad7d9





PrivateDnsDohTest.QueryFailover fails in waiting for the second
onPrivateDnsValidationEvent if DoT validation result and DoH
validation result are reported at the same time.

To fix the flaky test, this change makes the validation results
distinguishable by private DNS protocols.

Bug: 198111329
Test: PrivateDnsDohTest.QueryFailover 200 times passed on cuttlefish
Test: PrivateDnsDohTest.ValidationFail 500 times passed on cuttlefish
Change-Id: I8649b21bff176c6f2a1158f04812c8c8e96ad7d9







Replace StringPrintf() with fmt::format()
2021-08-20T06:45:36+00:00

chenbruce
chenbruce@google.com

2021-08-19T16:00:49+00:00

9b72daa3b7f3f33fcc5bf25269a9221cf296e904

Bug: 196298216
Test: cd packages/modules/DnsResolver && atest
Change-Id: I581c282bba1527afd2be9daf925e9ec5bd2e5cd6





Bug: 196298216
Test: cd packages/modules/DnsResolver && atest
Change-Id: I581c282bba1527afd2be9daf925e9ec5bd2e5cd6







Enlarge the DoH query timeout and the server probe timeout
2021-08-13T09:02:41+00:00

Luke Huang
huangluke@google.com

2021-08-13T06:53:37+00:00

dce6cbfab9ad5af7b03213f45fe75aa4dc5c0064

query timeout: from 2s to 30s
probe timeout: from 3s to 60s

Bug: 194048056
Test: atest
Change-Id: Ib11b17da7093b1d438ee8c7399694c2b19abeae9





query timeout: from 2s to 30s
probe timeout: from 3s to 60s

Bug: 194048056
Test: atest
Change-Id: Ib11b17da7093b1d438ee8c7399694c2b19abeae9







Add some DoH tests in resolv_integration_test
2021-08-02T09:39:57+00:00

Mike Yu
yumike@google.com

2021-07-06T13:01:17+00:00

5e406a34f7d870a0f8afaa3fdc17e2ca5ccfd894

The tests include:
[1] 1 test in TransportParameterizedTest
[2] 3 tests in PrivateDnsDohTest

Bug: 155855709
Bug: 181642979
Test: cd packages/modules/DnsResolver && atest
Change-Id: I8b5a85ad5556e7ff093656234069a81f29417c6a





The tests include:
[1] 1 test in TransportParameterizedTest
[2] 3 tests in PrivateDnsDohTest

Bug: 155855709
Bug: 181642979
Test: cd packages/modules/DnsResolver && atest
Change-Id: I8b5a85ad5556e7ff093656234069a81f29417c6a