| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |
|
|
|
|
|
|
|
|
| |
Inject a test Certificate authority for DNS-OVER-TLS test, and It
provides proper test coverage to TLS cerificate validation.
Test: built, flashed, booted
atest pass
Change-Id: I32b8c743d991ba1f113b37165ff01f4b2acc9da2
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Westworld requires the first field of all enums to be set to
UNKNOWN/UNSET, but this is awkward for the resolver code and
makes it hard to verify whether we have any PrivateDnsMode
variables initialized to UNKNOWN by accident.
So revert this change, and add a simple function to convert
the internal PrivateDnsMode enum to the other PrivateDnsMode
emun defined in the stats proto.
Bug: 119862317
Test: atest
Change-Id: Id146a16e41e0183867b3a4e2d87a2b0fc09df27d
Merged-In: I35908fac7f089c1fac507bae63b02ebb46ad50a9
Merged-In: I6054faa82f9a3cdc843fd6dcbd11d5cef75edd98
Merged-In: I74c7b4145d1469010d3e205183012a9b830f0a55
Merged-In: Id146a16e41e0183867b3a4e2d87a2b0fc09df27d
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, each device generates about 4000 queries per day.
Reduce the number of events by subsampling events based on
how interesting they are:
- if return_code == 0 -> log 1 in 100 events (random sampling)
- if return_code == EAI_NODATA -> log 1 in 10 events (random sampling)
- else -> log 100%
Also allow to use experiment flag to update sub-sampling denom.
Example for dumpsys dnsresolver:
NetId: 100
DnsEvent subsampling map: default:1 0:100 7:10
Bug: 119862317
Test: manual test with using setprop to change subsampling map
atest
Change-Id: Ie0a433a91f629dd32b1d0787a5b45926415faab0
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
This changes comprises:
[1] Convert resolv_set_nameservers_for_net to C++ containers.
[2] Unit tests for resolv_set_nameservers_for_net.
[3] Use std::vector<std::string> to store the DNS servers in struct
resolv_cache_info.
[4] Cleanup for libnetd_resolv_test.
Bug: 134897288
Test: cd systen/netd && atest
Change-Id: I4aebd22871a4ebbed08f03391e683cabe5dc2349
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1.
Drop the old C style used to store domains.
Previously, resolv is limited to use 6 search domains with total 255 length.
(including zero padding)
After this change, the length of each domain could exactly be at most 255. (rfc 1035)
Also, invalid or duplicate domains will be dropped.
2. rename resolv_set_nameservers_for_net to resolv_set_nameservers
Bug: 135506574
Test: cd system/netd && atest
Change-Id: I94129ea521522c817d087332a7b467f616cc4895
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Result dump
metric_id: 1111
event_metrics {
data {
elapsed_timestamp_nanos: 166844290056
atom {
network_dns_event_reported {
event_type: EVENT_GETADDRINFO
return_code: RC_EAI_NO_ERROR
latency_micros: 78961
hints_ai_flags: 1024
res_nsend_flags: -1
network_type: NT_UNKNOWN
private_dns_modes: PDM_STRICT
dns_query_events {
dns_query_event {
rcode: NS_R_NO_ERROR
type: NS_T_A
ip_version: IV_IPV4
protocol: PROTO_DOT
dns_server_index: 0
latency_micros: 77843
}
}
}
}
}
Bug: 119862317
Test: ./out/host/linux-x86/bin/statsd_testdrive 116
Change-Id: Ia9d2ca076cdd95df206f573531a17badb34fda62
|
| |
|
|
|
|
|
|
|
|
| |
include libnetd_client_headers explicitly
Bug: 135235267
Test: m
cd system/netd && atest
Change-Id: Iaeff2072ad0dcab8e42429cc4ae21f2dc3618004
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This is yet another preparatory change before enableing the ABI checker.
Internal functions are now declared in non-public headers paired with
the source files where they are declared.
A followup change could further split resolv_private.h.
Bug: 133117690
Test: No functionality changes, just run atest.
Change-Id: I66ab2f9856e7faa82c74a8ef3fbdeef579577c64
|
| |
|
|
|
|
|
|
|
|
|
| |
It is no longer necessary since ResolverController can directly
access the instance of PrivateDnsConfiguration and call its
method.
Bug: 130686826
Test: resolv_integration_test passed
Test: Re-enabled WiFi, and browsed websites
Change-Id: I340f60ef11415c9b83aa66884fffdfae3153d1d0
|
| |
|
|
|
|
|
|
|
|
|
| |
It is no longer necessary since ResolverController can directly
access the instance of PrivateDnsConfiguration and call its
method.
Bug: 130686826
Test: resolv_integration_test passed
Test: Switched to each private DNS mode and browsed websites
Change-Id: I944aa87c903c985d47466549c1b64034bdcaedd2
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This is a first pass at sanitizing the public headers of libnetd_resolv
by moving out things that are only used internally.
The main motivation is reducing noise in ABI dumps generated for the ABI
checker. It's also a nice opportunity to cut off the C wrappers that we
no longer need.
Test: cd system/netd && atest
Change-Id: I08689c263c4fd96363666a108ba08d768ad327df
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
1. Add binder call to support creating cache by netid.
2. revise call flow in res_cache.cpp, not create cache in
resolv_set_nameservers_for_net().
3. revise Log method in ResolverController.cpp
Bug: 129453995
Test: built, flashed, booted
system/netd/tests/runtests.sh pass
Change-Id: I31ab521027a08bce3c721b2a36418faca85853c7
|
| |
|
|
|
|
|
|
| |
setResolverConfiguration take a parcelable.
Test: built, flashed, booted
system/netd/tests/runtests.sh pass
Change-Id: I755b7832d395ab138d5440abdcb9d86fde0a00e3
|
| |
|
|
|
|
|
|
|
| |
Set LOG_TAG to "resolv" on DNS resolver. Then, the log can be filtered
by "adb logcat [filterspecs]", such as "adb logcat *:S resolv:*", or
'grep'.
Test: resolv_integration_test pass
Change-Id: Ic4f97685d487257560ec3dfe62bab0d187bf9d93
|
| |
|
|
|
|
|
|
|
|
|
| |
1. Make DnsProxyListener get DNS64 prefix internally in libnetd_resolv
2. resolver_test uses IDnsResolver instead of INetd for resolver cmds
Bug: 126141549
Test: built, flashed, booted
system/netd/tests/runtests.sh pass
Change-Id: Ic904a3efd42ac4011ddd94ff4344449c9e647ceb
|
|
|
In order to decrease API between libnetd_resolv and netd,
migrate resolver related commands to libnetd_resolv.
After migration done, netd's resolver related commands will be
removed in following commits.
Bug: 126141549
Test: N/A.
Note that this commit will cause build break,
it must be grouped with following clean-up commits.
Change-Id: Id0c2e20ce6a87b5c01ae5bcb998381099889306e
|