| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2018 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
| Ken Chen | 5471dca | 2019-04-15 15:25:35 +0800 | [diff] [blame] | 17 | #define LOG_TAG "resolv" |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 18 | |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 19 | #include <arpa/inet.h> |
| 20 | |
| 21 | #include <chrono> |
| 22 | |
| 23 | #include <android-base/logging.h> |
| 24 | #include <android-base/macros.h> |
| Mike Yu | 1b9069c | 2020-08-25 15:17:29 +0800 | [diff] [blame] | 25 | #include <gmock/gmock.h> |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 26 | #include <gtest/gtest.h> |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 27 | #include <netdutils/Slice.h> |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 28 | |
| Bernie Innocenti | ec4219b | 2019-01-30 11:16:36 +0900 | [diff] [blame] | 29 | #include "DnsTlsDispatcher.h" |
| 30 | #include "DnsTlsQueryMap.h" |
| 31 | #include "DnsTlsServer.h" |
| 32 | #include "DnsTlsSessionCache.h" |
| 33 | #include "DnsTlsSocket.h" |
| 34 | #include "DnsTlsTransport.h" |
| Mike Yu | bb49909 | 2020-08-28 19:18:42 +0800 | [diff] [blame] | 35 | #include "Experiments.h" |
| Bernie Innocenti | ec4219b | 2019-01-30 11:16:36 +0900 | [diff] [blame] | 36 | #include "IDnsTlsSocket.h" |
| 37 | #include "IDnsTlsSocketFactory.h" |
| 38 | #include "IDnsTlsSocketObserver.h" |
| chenbruce | b43ec75 | 2019-07-24 20:19:41 +0800 | [diff] [blame] | 39 | #include "tests/dns_responder/dns_tls_frontend.h" |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 40 | |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 41 | namespace android { |
| 42 | namespace net { |
| 43 | |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 44 | using netdutils::makeSlice; |
| Mike Yu | 1b9069c | 2020-08-25 15:17:29 +0800 | [diff] [blame] | 45 | using netdutils::Slice; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 46 | |
| Mike Yu | bb49909 | 2020-08-28 19:18:42 +0800 | [diff] [blame] | 47 | static const std::string DOT_MAXTRIES_FLAG = "dot_maxtries"; |
| 48 | |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 49 | typedef std::vector<uint8_t> bytevec; |
| 50 | |
| 51 | static void parseServer(const char* server, in_port_t port, sockaddr_storage* parsed) { |
| 52 | sockaddr_in* sin = reinterpret_cast<sockaddr_in*>(parsed); |
| 53 | if (inet_pton(AF_INET, server, &(sin->sin_addr)) == 1) { |
| 54 | // IPv4 parse succeeded, so it's IPv4 |
| 55 | sin->sin_family = AF_INET; |
| 56 | sin->sin_port = htons(port); |
| 57 | return; |
| 58 | } |
| 59 | sockaddr_in6* sin6 = reinterpret_cast<sockaddr_in6*>(parsed); |
| 60 | if (inet_pton(AF_INET6, server, &(sin6->sin6_addr)) == 1){ |
| 61 | // IPv6 parse succeeded, so it's IPv6. |
| 62 | sin6->sin6_family = AF_INET6; |
| 63 | sin6->sin6_port = htons(port); |
| 64 | return; |
| 65 | } |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 66 | LOG(ERROR) << "Failed to parse server address: " << server; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 67 | } |
| 68 | |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 69 | std::string SERVERNAME1 = "dns.example.com"; |
| 70 | std::string SERVERNAME2 = "dns.example.org"; |
| 71 | |
| 72 | // BaseTest just provides constants that are useful for the tests. |
| 73 | class BaseTest : public ::testing::Test { |
| 74 | protected: |
| 75 | BaseTest() { |
| 76 | parseServer("192.0.2.1", 853, &V4ADDR1); |
| 77 | parseServer("192.0.2.2", 853, &V4ADDR2); |
| 78 | parseServer("2001:db8::1", 853, &V6ADDR1); |
| 79 | parseServer("2001:db8::2", 853, &V6ADDR2); |
| 80 | |
| 81 | SERVER1 = DnsTlsServer(V4ADDR1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 82 | SERVER1.name = SERVERNAME1; |
| 83 | } |
| 84 | |
| 85 | sockaddr_storage V4ADDR1; |
| 86 | sockaddr_storage V4ADDR2; |
| 87 | sockaddr_storage V6ADDR1; |
| 88 | sockaddr_storage V6ADDR2; |
| 89 | |
| 90 | DnsTlsServer SERVER1; |
| 91 | }; |
| 92 | |
| 93 | bytevec make_query(uint16_t id, size_t size) { |
| 94 | bytevec vec(size); |
| 95 | vec[0] = id >> 8; |
| 96 | vec[1] = id; |
| 97 | // Arbitrarily fill the query body with unique data. |
| 98 | for (size_t i = 2; i < size; ++i) { |
| 99 | vec[i] = id + i; |
| 100 | } |
| 101 | return vec; |
| 102 | } |
| 103 | |
| 104 | // Query constants |
| 105 | const unsigned MARK = 123; |
| 106 | const uint16_t ID = 52; |
| 107 | const uint16_t SIZE = 22; |
| 108 | const bytevec QUERY = make_query(ID, SIZE); |
| 109 | |
| 110 | template <class T> |
| 111 | class FakeSocketFactory : public IDnsTlsSocketFactory { |
| 112 | public: |
| 113 | FakeSocketFactory() {} |
| 114 | std::unique_ptr<IDnsTlsSocket> createDnsTlsSocket( |
| 115 | const DnsTlsServer& server ATTRIBUTE_UNUSED, |
| 116 | unsigned mark ATTRIBUTE_UNUSED, |
| 117 | IDnsTlsSocketObserver* observer, |
| 118 | DnsTlsSessionCache* cache ATTRIBUTE_UNUSED) override { |
| 119 | return std::make_unique<T>(observer); |
| 120 | } |
| 121 | }; |
| 122 | |
| 123 | bytevec make_echo(uint16_t id, const Slice query) { |
| 124 | bytevec response(query.size() + 2); |
| 125 | response[0] = id >> 8; |
| 126 | response[1] = id; |
| 127 | // Echo the query as the fake response. |
| 128 | memcpy(response.data() + 2, query.base(), query.size()); |
| 129 | return response; |
| 130 | } |
| 131 | |
| 132 | // Simplest possible fake server. This just echoes the query as the response. |
| 133 | class FakeSocketEcho : public IDnsTlsSocket { |
| 134 | public: |
| 135 | explicit FakeSocketEcho(IDnsTlsSocketObserver* observer) : mObserver(observer) {} |
| 136 | bool query(uint16_t id, const Slice query) override { |
| 137 | // Return the response immediately (asynchronously). |
| 138 | std::thread(&IDnsTlsSocketObserver::onResponse, mObserver, make_echo(id, query)).detach(); |
| 139 | return true; |
| 140 | } |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 141 | bool startHandshake() override { return true; } |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 142 | |
| 143 | private: |
| 144 | IDnsTlsSocketObserver* const mObserver; |
| 145 | }; |
| 146 | |
| 147 | class TransportTest : public BaseTest {}; |
| 148 | |
| 149 | TEST_F(TransportTest, Query) { |
| 150 | FakeSocketFactory<FakeSocketEcho> factory; |
| 151 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 152 | auto r = transport.query(makeSlice(QUERY)).get(); |
| 153 | |
| 154 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 155 | EXPECT_EQ(QUERY, r.response); |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 156 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 157 | } |
| 158 | |
| 159 | // Fake Socket that echoes the observed query ID as the response body. |
| 160 | class FakeSocketId : public IDnsTlsSocket { |
| 161 | public: |
| 162 | explicit FakeSocketId(IDnsTlsSocketObserver* observer) : mObserver(observer) {} |
| 163 | bool query(uint16_t id, const Slice query ATTRIBUTE_UNUSED) override { |
| 164 | // Return the response immediately (asynchronously). |
| 165 | bytevec response(4); |
| 166 | // Echo the ID in the header to match the response to the query. |
| 167 | // This will be overwritten by DnsTlsQueryMap. |
| 168 | response[0] = id >> 8; |
| 169 | response[1] = id; |
| 170 | // Echo the ID in the body, so that the test can verify which ID was used by |
| 171 | // DnsTlsQueryMap. |
| 172 | response[2] = id >> 8; |
| 173 | response[3] = id; |
| 174 | std::thread(&IDnsTlsSocketObserver::onResponse, mObserver, response).detach(); |
| 175 | return true; |
| 176 | } |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 177 | bool startHandshake() override { return true; } |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 178 | |
| 179 | private: |
| 180 | IDnsTlsSocketObserver* const mObserver; |
| 181 | }; |
| 182 | |
| 183 | // Test that IDs are properly reused |
| 184 | TEST_F(TransportTest, IdReuse) { |
| 185 | FakeSocketFactory<FakeSocketId> factory; |
| 186 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 187 | for (int i = 0; i < 100; ++i) { |
| 188 | // Send a query. |
| Mike Yu | bd13699 | 2019-12-04 15:01:07 +0800 | [diff] [blame] | 189 | std::future<DnsTlsTransport::Result> f = transport.query(makeSlice(QUERY)); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 190 | // Wait for the response. |
| Mike Yu | bd13699 | 2019-12-04 15:01:07 +0800 | [diff] [blame] | 191 | DnsTlsTransport::Result r = f.get(); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 192 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 193 | |
| 194 | // All queries should have an observed ID of zero, because it is returned to the ID pool |
| 195 | // after each use. |
| 196 | EXPECT_EQ(0, (r.response[2] << 8) | r.response[3]); |
| 197 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 198 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 199 | } |
| 200 | |
| 201 | // These queries might be handled in serial or parallel as they race the |
| 202 | // responses. |
| 203 | TEST_F(TransportTest, RacingQueries_10000) { |
| 204 | FakeSocketFactory<FakeSocketEcho> factory; |
| 205 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 206 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 207 | // Fewer than 65536 queries to avoid ID exhaustion. |
| 208 | const int num_queries = 10000; |
| 209 | results.reserve(num_queries); |
| 210 | for (int i = 0; i < num_queries; ++i) { |
| 211 | results.push_back(transport.query(makeSlice(QUERY))); |
| 212 | } |
| 213 | for (auto& result : results) { |
| 214 | auto r = result.get(); |
| 215 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 216 | EXPECT_EQ(QUERY, r.response); |
| 217 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 218 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 219 | } |
| 220 | |
| 221 | // A server that waits until sDelay queries are queued before responding. |
| 222 | class FakeSocketDelay : public IDnsTlsSocket { |
| 223 | public: |
| 224 | explicit FakeSocketDelay(IDnsTlsSocketObserver* observer) : mObserver(observer) {} |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 225 | ~FakeSocketDelay() { |
| 226 | std::lock_guard guard(mLock); |
| 227 | sDelay = 1; |
| 228 | sReverse = false; |
| 229 | sConnectable = true; |
| 230 | } |
| 231 | inline static size_t sDelay = 1; |
| 232 | inline static bool sReverse = false; |
| 233 | inline static bool sConnectable = true; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 234 | |
| 235 | bool query(uint16_t id, const Slice query) override { |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 236 | LOG(DEBUG) << "FakeSocketDelay got query with ID " << int(id); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 237 | std::lock_guard guard(mLock); |
| 238 | // Check for duplicate IDs. |
| 239 | EXPECT_EQ(0U, mIds.count(id)); |
| 240 | mIds.insert(id); |
| 241 | |
| 242 | // Store response. |
| 243 | mResponses.push_back(make_echo(id, query)); |
| 244 | |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 245 | LOG(DEBUG) << "Up to " << mResponses.size() << " out of " << sDelay << " queries"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 246 | if (mResponses.size() == sDelay) { |
| 247 | std::thread(&FakeSocketDelay::sendResponses, this).detach(); |
| 248 | } |
| 249 | return true; |
| 250 | } |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 251 | bool startHandshake() override { return sConnectable; } |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 252 | |
| 253 | private: |
| 254 | void sendResponses() { |
| 255 | std::lock_guard guard(mLock); |
| 256 | if (sReverse) { |
| 257 | std::reverse(std::begin(mResponses), std::end(mResponses)); |
| 258 | } |
| 259 | for (auto& response : mResponses) { |
| 260 | mObserver->onResponse(response); |
| 261 | } |
| 262 | mIds.clear(); |
| 263 | mResponses.clear(); |
| 264 | } |
| 265 | |
| 266 | std::mutex mLock; |
| 267 | IDnsTlsSocketObserver* const mObserver; |
| 268 | std::set<uint16_t> mIds GUARDED_BY(mLock); |
| 269 | std::vector<bytevec> mResponses GUARDED_BY(mLock); |
| 270 | }; |
| 271 | |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 272 | TEST_F(TransportTest, ParallelColliding) { |
| 273 | FakeSocketDelay::sDelay = 10; |
| 274 | FakeSocketDelay::sReverse = false; |
| 275 | FakeSocketFactory<FakeSocketDelay> factory; |
| 276 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 277 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 278 | // Fewer than 65536 queries to avoid ID exhaustion. |
| 279 | results.reserve(FakeSocketDelay::sDelay); |
| 280 | for (size_t i = 0; i < FakeSocketDelay::sDelay; ++i) { |
| 281 | results.push_back(transport.query(makeSlice(QUERY))); |
| 282 | } |
| 283 | for (auto& result : results) { |
| 284 | auto r = result.get(); |
| 285 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 286 | EXPECT_EQ(QUERY, r.response); |
| 287 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 288 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 289 | } |
| 290 | |
| 291 | TEST_F(TransportTest, ParallelColliding_Max) { |
| 292 | FakeSocketDelay::sDelay = 65536; |
| 293 | FakeSocketDelay::sReverse = false; |
| 294 | FakeSocketFactory<FakeSocketDelay> factory; |
| 295 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 296 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 297 | // Exactly 65536 queries should still be possible in parallel, |
| 298 | // even if they all have the same original ID. |
| 299 | results.reserve(FakeSocketDelay::sDelay); |
| 300 | for (size_t i = 0; i < FakeSocketDelay::sDelay; ++i) { |
| 301 | results.push_back(transport.query(makeSlice(QUERY))); |
| 302 | } |
| 303 | for (auto& result : results) { |
| 304 | auto r = result.get(); |
| 305 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 306 | EXPECT_EQ(QUERY, r.response); |
| 307 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 308 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 309 | } |
| 310 | |
| 311 | TEST_F(TransportTest, ParallelUnique) { |
| 312 | FakeSocketDelay::sDelay = 10; |
| 313 | FakeSocketDelay::sReverse = false; |
| 314 | FakeSocketFactory<FakeSocketDelay> factory; |
| 315 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 316 | std::vector<bytevec> queries(FakeSocketDelay::sDelay); |
| 317 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 318 | results.reserve(FakeSocketDelay::sDelay); |
| 319 | for (size_t i = 0; i < FakeSocketDelay::sDelay; ++i) { |
| 320 | queries[i] = make_query(i, SIZE); |
| 321 | results.push_back(transport.query(makeSlice(queries[i]))); |
| 322 | } |
| 323 | for (size_t i = 0 ; i < FakeSocketDelay::sDelay; ++i) { |
| 324 | auto r = results[i].get(); |
| 325 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 326 | EXPECT_EQ(queries[i], r.response); |
| 327 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 328 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 329 | } |
| 330 | |
| 331 | TEST_F(TransportTest, ParallelUnique_Max) { |
| 332 | FakeSocketDelay::sDelay = 65536; |
| 333 | FakeSocketDelay::sReverse = false; |
| 334 | FakeSocketFactory<FakeSocketDelay> factory; |
| 335 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 336 | std::vector<bytevec> queries(FakeSocketDelay::sDelay); |
| 337 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 338 | // Exactly 65536 queries should still be possible in parallel, |
| 339 | // and they should all be mapped correctly back to the original ID. |
| 340 | results.reserve(FakeSocketDelay::sDelay); |
| 341 | for (size_t i = 0; i < FakeSocketDelay::sDelay; ++i) { |
| 342 | queries[i] = make_query(i, SIZE); |
| 343 | results.push_back(transport.query(makeSlice(queries[i]))); |
| 344 | } |
| 345 | for (size_t i = 0 ; i < FakeSocketDelay::sDelay; ++i) { |
| 346 | auto r = results[i].get(); |
| 347 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 348 | EXPECT_EQ(queries[i], r.response); |
| 349 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 350 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 351 | } |
| 352 | |
| 353 | TEST_F(TransportTest, IdExhaustion) { |
| 354 | const int num_queries = 65536; |
| 355 | // A delay of 65537 is unreachable, because the maximum number |
| 356 | // of outstanding queries is 65536. |
| 357 | FakeSocketDelay::sDelay = num_queries + 1; |
| 358 | FakeSocketDelay::sReverse = false; |
| 359 | FakeSocketFactory<FakeSocketDelay> factory; |
| 360 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 361 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 362 | // Issue the maximum number of queries. |
| 363 | results.reserve(num_queries); |
| 364 | for (int i = 0; i < num_queries; ++i) { |
| 365 | results.push_back(transport.query(makeSlice(QUERY))); |
| 366 | } |
| 367 | |
| 368 | // The ID space is now full, so subsequent queries should fail immediately. |
| 369 | auto r = transport.query(makeSlice(QUERY)).get(); |
| 370 | EXPECT_EQ(DnsTlsTransport::Response::internal_error, r.code); |
| 371 | EXPECT_TRUE(r.response.empty()); |
| 372 | |
| 373 | for (auto& result : results) { |
| 374 | // All other queries should remain outstanding. |
| 375 | EXPECT_EQ(std::future_status::timeout, |
| 376 | result.wait_for(std::chrono::duration<int>::zero())); |
| 377 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 378 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 379 | } |
| 380 | |
| 381 | // Responses can come back from the server in any order. This should have no |
| 382 | // effect on Transport's observed behavior. |
| 383 | TEST_F(TransportTest, ReverseOrder) { |
| 384 | FakeSocketDelay::sDelay = 10; |
| 385 | FakeSocketDelay::sReverse = true; |
| 386 | FakeSocketFactory<FakeSocketDelay> factory; |
| 387 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 388 | std::vector<bytevec> queries(FakeSocketDelay::sDelay); |
| 389 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 390 | results.reserve(FakeSocketDelay::sDelay); |
| 391 | for (size_t i = 0; i < FakeSocketDelay::sDelay; ++i) { |
| 392 | queries[i] = make_query(i, SIZE); |
| 393 | results.push_back(transport.query(makeSlice(queries[i]))); |
| 394 | } |
| 395 | for (size_t i = 0 ; i < FakeSocketDelay::sDelay; ++i) { |
| 396 | auto r = results[i].get(); |
| 397 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 398 | EXPECT_EQ(queries[i], r.response); |
| 399 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 400 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 401 | } |
| 402 | |
| 403 | TEST_F(TransportTest, ReverseOrder_Max) { |
| 404 | FakeSocketDelay::sDelay = 65536; |
| 405 | FakeSocketDelay::sReverse = true; |
| 406 | FakeSocketFactory<FakeSocketDelay> factory; |
| 407 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 408 | std::vector<bytevec> queries(FakeSocketDelay::sDelay); |
| 409 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 410 | results.reserve(FakeSocketDelay::sDelay); |
| 411 | for (size_t i = 0; i < FakeSocketDelay::sDelay; ++i) { |
| 412 | queries[i] = make_query(i, SIZE); |
| 413 | results.push_back(transport.query(makeSlice(queries[i]))); |
| 414 | } |
| 415 | for (size_t i = 0 ; i < FakeSocketDelay::sDelay; ++i) { |
| 416 | auto r = results[i].get(); |
| 417 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 418 | EXPECT_EQ(queries[i], r.response); |
| 419 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 420 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 421 | } |
| 422 | |
| 423 | // Returning null from the factory indicates a connection failure. |
| 424 | class NullSocketFactory : public IDnsTlsSocketFactory { |
| 425 | public: |
| 426 | NullSocketFactory() {} |
| 427 | std::unique_ptr<IDnsTlsSocket> createDnsTlsSocket( |
| 428 | const DnsTlsServer& server ATTRIBUTE_UNUSED, |
| 429 | unsigned mark ATTRIBUTE_UNUSED, |
| 430 | IDnsTlsSocketObserver* observer ATTRIBUTE_UNUSED, |
| 431 | DnsTlsSessionCache* cache ATTRIBUTE_UNUSED) override { |
| 432 | return nullptr; |
| 433 | } |
| 434 | }; |
| 435 | |
| 436 | TEST_F(TransportTest, ConnectFail) { |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 437 | // Failure on creating socket. |
| 438 | NullSocketFactory factory1; |
| 439 | DnsTlsTransport transport1(SERVER1, MARK, &factory1); |
| 440 | auto r = transport1.query(makeSlice(QUERY)).get(); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 441 | |
| 442 | EXPECT_EQ(DnsTlsTransport::Response::network_error, r.code); |
| 443 | EXPECT_TRUE(r.response.empty()); |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 444 | EXPECT_EQ(transport1.getConnectCounter(), 1); |
| 445 | |
| 446 | // Failure on handshaking. |
| 447 | FakeSocketDelay::sConnectable = false; |
| 448 | FakeSocketFactory<FakeSocketDelay> factory2; |
| 449 | DnsTlsTransport transport2(SERVER1, MARK, &factory2); |
| 450 | r = transport2.query(makeSlice(QUERY)).get(); |
| 451 | |
| 452 | EXPECT_EQ(DnsTlsTransport::Response::network_error, r.code); |
| 453 | EXPECT_TRUE(r.response.empty()); |
| 454 | EXPECT_EQ(transport2.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 455 | } |
| 456 | |
| 457 | // Simulate a socket that connects but then immediately receives a server |
| 458 | // close notification. |
| 459 | class FakeSocketClose : public IDnsTlsSocket { |
| 460 | public: |
| 461 | explicit FakeSocketClose(IDnsTlsSocketObserver* observer) |
| 462 | : mCloser(&IDnsTlsSocketObserver::onClosed, observer) {} |
| 463 | ~FakeSocketClose() { mCloser.join(); } |
| 464 | bool query(uint16_t id ATTRIBUTE_UNUSED, |
| 465 | const Slice query ATTRIBUTE_UNUSED) override { |
| 466 | return true; |
| 467 | } |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 468 | bool startHandshake() override { return true; } |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 469 | |
| 470 | private: |
| 471 | std::thread mCloser; |
| 472 | }; |
| 473 | |
| 474 | TEST_F(TransportTest, CloseRetryFail) { |
| 475 | FakeSocketFactory<FakeSocketClose> factory; |
| 476 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 477 | auto r = transport.query(makeSlice(QUERY)).get(); |
| 478 | |
| 479 | EXPECT_EQ(DnsTlsTransport::Response::network_error, r.code); |
| 480 | EXPECT_TRUE(r.response.empty()); |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 481 | |
| Mike Yu | bb49909 | 2020-08-28 19:18:42 +0800 | [diff] [blame] | 482 | // Reconnections might be triggered depending on the flag. |
| 483 | EXPECT_EQ(transport.getConnectCounter(), |
| 484 | Experiments::getInstance()->getFlag(DOT_MAXTRIES_FLAG, DnsTlsQueryMap::kMaxTries)); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 485 | } |
| 486 | |
| 487 | // Simulate a server that occasionally closes the connection and silently |
| 488 | // drops some queries. |
| 489 | class FakeSocketLimited : public IDnsTlsSocket { |
| 490 | public: |
| 491 | static int sLimit; // Number of queries to answer per socket. |
| 492 | static size_t sMaxSize; // Silently discard queries greater than this size. |
| 493 | explicit FakeSocketLimited(IDnsTlsSocketObserver* observer) |
| 494 | : mObserver(observer), mQueries(0) {} |
| 495 | ~FakeSocketLimited() { |
| 496 | { |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 497 | LOG(DEBUG) << "~FakeSocketLimited acquiring mLock"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 498 | std::lock_guard guard(mLock); |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 499 | LOG(DEBUG) << "~FakeSocketLimited acquired mLock"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 500 | for (auto& thread : mThreads) { |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 501 | LOG(DEBUG) << "~FakeSocketLimited joining response thread"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 502 | thread.join(); |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 503 | LOG(DEBUG) << "~FakeSocketLimited joined response thread"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 504 | } |
| 505 | mThreads.clear(); |
| 506 | } |
| 507 | |
| 508 | if (mCloser) { |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 509 | LOG(DEBUG) << "~FakeSocketLimited joining closer thread"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 510 | mCloser->join(); |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 511 | LOG(DEBUG) << "~FakeSocketLimited joined closer thread"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 512 | } |
| 513 | } |
| 514 | bool query(uint16_t id, const Slice query) override { |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 515 | LOG(DEBUG) << "FakeSocketLimited::query acquiring mLock"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 516 | std::lock_guard guard(mLock); |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 517 | LOG(DEBUG) << "FakeSocketLimited::query acquired mLock"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 518 | ++mQueries; |
| 519 | |
| 520 | if (mQueries <= sLimit) { |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 521 | LOG(DEBUG) << "size " << query.size() << " vs. limit of " << sMaxSize; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 522 | if (query.size() <= sMaxSize) { |
| 523 | // Return the response immediately (asynchronously). |
| 524 | mThreads.emplace_back(&IDnsTlsSocketObserver::onResponse, mObserver, make_echo(id, query)); |
| 525 | } |
| 526 | } |
| 527 | if (mQueries == sLimit) { |
| 528 | mCloser = std::make_unique<std::thread>(&FakeSocketLimited::sendClose, this); |
| 529 | } |
| 530 | return mQueries <= sLimit; |
| 531 | } |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 532 | bool startHandshake() override { return true; } |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 533 | |
| 534 | private: |
| 535 | void sendClose() { |
| 536 | { |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 537 | LOG(DEBUG) << "FakeSocketLimited::sendClose acquiring mLock"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 538 | std::lock_guard guard(mLock); |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 539 | LOG(DEBUG) << "FakeSocketLimited::sendClose acquired mLock"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 540 | for (auto& thread : mThreads) { |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 541 | LOG(DEBUG) << "FakeSocketLimited::sendClose joining response thread"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 542 | thread.join(); |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 543 | LOG(DEBUG) << "FakeSocketLimited::sendClose joined response thread"; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 544 | } |
| 545 | mThreads.clear(); |
| 546 | } |
| 547 | mObserver->onClosed(); |
| 548 | } |
| 549 | std::mutex mLock; |
| 550 | IDnsTlsSocketObserver* const mObserver; |
| 551 | int mQueries GUARDED_BY(mLock); |
| 552 | std::vector<std::thread> mThreads GUARDED_BY(mLock); |
| 553 | std::unique_ptr<std::thread> mCloser GUARDED_BY(mLock); |
| 554 | }; |
| 555 | |
| 556 | int FakeSocketLimited::sLimit; |
| 557 | size_t FakeSocketLimited::sMaxSize; |
| 558 | |
| 559 | TEST_F(TransportTest, SilentDrop) { |
| 560 | FakeSocketLimited::sLimit = 10; // Close the socket after 10 queries. |
| 561 | FakeSocketLimited::sMaxSize = 0; // Silently drop all queries |
| 562 | FakeSocketFactory<FakeSocketLimited> factory; |
| 563 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 564 | |
| 565 | // Queue up 10 queries. They will all be ignored, and after the 10th, |
| 566 | // the socket will close. Transport will retry them all, until they |
| 567 | // all hit the retry limit and expire. |
| 568 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 569 | results.reserve(FakeSocketLimited::sLimit); |
| 570 | for (int i = 0; i < FakeSocketLimited::sLimit; ++i) { |
| 571 | results.push_back(transport.query(makeSlice(QUERY))); |
| 572 | } |
| 573 | for (auto& result : results) { |
| 574 | auto r = result.get(); |
| 575 | EXPECT_EQ(DnsTlsTransport::Response::network_error, r.code); |
| 576 | EXPECT_TRUE(r.response.empty()); |
| 577 | } |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 578 | |
| Mike Yu | bb49909 | 2020-08-28 19:18:42 +0800 | [diff] [blame] | 579 | // Reconnections might be triggered depending on the flag. |
| 580 | EXPECT_EQ(transport.getConnectCounter(), |
| 581 | Experiments::getInstance()->getFlag(DOT_MAXTRIES_FLAG, DnsTlsQueryMap::kMaxTries)); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 582 | } |
| 583 | |
| 584 | TEST_F(TransportTest, PartialDrop) { |
| 585 | FakeSocketLimited::sLimit = 10; // Close the socket after 10 queries. |
| 586 | FakeSocketLimited::sMaxSize = SIZE - 2; // Silently drop "long" queries |
| 587 | FakeSocketFactory<FakeSocketLimited> factory; |
| 588 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 589 | |
| 590 | // Queue up 100 queries, alternating "short" which will be served and "long" |
| 591 | // which will be dropped. |
| 592 | const int num_queries = 10 * FakeSocketLimited::sLimit; |
| 593 | std::vector<bytevec> queries(num_queries); |
| 594 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 595 | results.reserve(num_queries); |
| 596 | for (int i = 0; i < num_queries; ++i) { |
| 597 | queries[i] = make_query(i, SIZE + (i % 2)); |
| 598 | results.push_back(transport.query(makeSlice(queries[i]))); |
| 599 | } |
| 600 | // Just check the short queries, which are at the even indices. |
| 601 | for (int i = 0; i < num_queries; i += 2) { |
| 602 | auto r = results[i].get(); |
| 603 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 604 | EXPECT_EQ(queries[i], r.response); |
| 605 | } |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 606 | |
| 607 | // TODO: transport.getConnectCounter() seems not stable in this test. Find how to check the |
| 608 | // connect attempts for this test. |
| 609 | } |
| 610 | |
| 611 | TEST_F(TransportTest, ConnectCounter) { |
| 612 | FakeSocketLimited::sLimit = 2; // Close the socket after 2 queries. |
| 613 | FakeSocketLimited::sMaxSize = SIZE; // No query drops. |
| 614 | FakeSocketFactory<FakeSocketLimited> factory; |
| 615 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 616 | |
| 617 | // Connecting on demand. |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 618 | EXPECT_EQ(transport.getConnectCounter(), 0); |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 619 | |
| 620 | const int num_queries = 10; |
| 621 | std::vector<std::future<DnsTlsTransport::Result>> results; |
| 622 | results.reserve(num_queries); |
| 623 | for (int i = 0; i < num_queries; i++) { |
| 624 | // Reconnections take place every two queries. |
| 625 | results.push_back(transport.query(makeSlice(QUERY))); |
| 626 | } |
| 627 | for (int i = 0; i < num_queries; i++) { |
| 628 | auto r = results[i].get(); |
| 629 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 630 | } |
| 631 | |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 632 | EXPECT_EQ(transport.getConnectCounter(), num_queries / FakeSocketLimited::sLimit); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 633 | } |
| 634 | |
| 635 | // Simulate a malfunctioning server that injects extra miscellaneous |
| 636 | // responses to queries that were not asked. This will cause wrong answers but |
| 637 | // must not crash the Transport. |
| 638 | class FakeSocketGarbage : public IDnsTlsSocket { |
| 639 | public: |
| 640 | explicit FakeSocketGarbage(IDnsTlsSocketObserver* observer) : mObserver(observer) { |
| 641 | // Inject a garbage event. |
| 642 | mThreads.emplace_back(&IDnsTlsSocketObserver::onResponse, mObserver, make_query(ID + 1, SIZE)); |
| 643 | } |
| 644 | ~FakeSocketGarbage() { |
| 645 | std::lock_guard guard(mLock); |
| 646 | for (auto& thread : mThreads) { |
| 647 | thread.join(); |
| 648 | } |
| 649 | } |
| 650 | bool query(uint16_t id, const Slice query) override { |
| 651 | std::lock_guard guard(mLock); |
| 652 | // Return the response twice. |
| 653 | auto echo = make_echo(id, query); |
| 654 | mThreads.emplace_back(&IDnsTlsSocketObserver::onResponse, mObserver, echo); |
| 655 | mThreads.emplace_back(&IDnsTlsSocketObserver::onResponse, mObserver, echo); |
| 656 | // Also return some other garbage |
| 657 | mThreads.emplace_back(&IDnsTlsSocketObserver::onResponse, mObserver, make_query(id + 1, query.size() + 2)); |
| 658 | return true; |
| 659 | } |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 660 | bool startHandshake() override { return true; } |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 661 | |
| 662 | private: |
| 663 | std::mutex mLock; |
| 664 | std::vector<std::thread> mThreads GUARDED_BY(mLock); |
| 665 | IDnsTlsSocketObserver* const mObserver; |
| 666 | }; |
| 667 | |
| 668 | TEST_F(TransportTest, IgnoringGarbage) { |
| 669 | FakeSocketFactory<FakeSocketGarbage> factory; |
| 670 | DnsTlsTransport transport(SERVER1, MARK, &factory); |
| 671 | for (int i = 0; i < 10; ++i) { |
| 672 | auto r = transport.query(makeSlice(QUERY)).get(); |
| 673 | |
| 674 | EXPECT_EQ(DnsTlsTransport::Response::success, r.code); |
| 675 | // Don't check the response because this server is malfunctioning. |
| 676 | } |
| Mike Yu | 1fea18c | 2019-12-06 10:59:17 +0800 | [diff] [blame] | 677 | EXPECT_EQ(transport.getConnectCounter(), 1); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 678 | } |
| 679 | |
| 680 | // Dispatcher tests |
| 681 | class DispatcherTest : public BaseTest {}; |
| 682 | |
| 683 | TEST_F(DispatcherTest, Query) { |
| 684 | bytevec ans(4096); |
| 685 | int resplen = 0; |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 686 | bool connectTriggered = false; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 687 | |
| 688 | auto factory = std::make_unique<FakeSocketFactory<FakeSocketEcho>>(); |
| 689 | DnsTlsDispatcher dispatcher(std::move(factory)); |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 690 | auto r = dispatcher.query(SERVER1, MARK, makeSlice(QUERY), makeSlice(ans), &resplen, |
| 691 | &connectTriggered); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 692 | |
| 693 | EXPECT_EQ(DnsTlsTransport::Response::success, r); |
| 694 | EXPECT_EQ(int(QUERY.size()), resplen); |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 695 | EXPECT_TRUE(connectTriggered); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 696 | ans.resize(resplen); |
| 697 | EXPECT_EQ(QUERY, ans); |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 698 | |
| 699 | // Expect to reuse the connection. |
| 700 | r = dispatcher.query(SERVER1, MARK, makeSlice(QUERY), makeSlice(ans), &resplen, |
| 701 | &connectTriggered); |
| 702 | EXPECT_EQ(DnsTlsTransport::Response::success, r); |
| 703 | EXPECT_FALSE(connectTriggered); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 704 | } |
| 705 | |
| 706 | TEST_F(DispatcherTest, AnswerTooLarge) { |
| 707 | bytevec ans(SIZE - 1); // Too small to hold the answer |
| 708 | int resplen = 0; |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 709 | bool connectTriggered = false; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 710 | |
| 711 | auto factory = std::make_unique<FakeSocketFactory<FakeSocketEcho>>(); |
| 712 | DnsTlsDispatcher dispatcher(std::move(factory)); |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 713 | auto r = dispatcher.query(SERVER1, MARK, makeSlice(QUERY), makeSlice(ans), &resplen, |
| 714 | &connectTriggered); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 715 | |
| 716 | EXPECT_EQ(DnsTlsTransport::Response::limit_error, r); |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 717 | EXPECT_TRUE(connectTriggered); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 718 | } |
| 719 | |
| 720 | template<class T> |
| 721 | class TrackingFakeSocketFactory : public IDnsTlsSocketFactory { |
| 722 | public: |
| 723 | TrackingFakeSocketFactory() {} |
| 724 | std::unique_ptr<IDnsTlsSocket> createDnsTlsSocket( |
| 725 | const DnsTlsServer& server, |
| 726 | unsigned mark, |
| 727 | IDnsTlsSocketObserver* observer, |
| 728 | DnsTlsSessionCache* cache ATTRIBUTE_UNUSED) override { |
| 729 | std::lock_guard guard(mLock); |
| 730 | keys.emplace(mark, server); |
| 731 | return std::make_unique<T>(observer); |
| 732 | } |
| 733 | std::multiset<std::pair<unsigned, DnsTlsServer>> keys; |
| 734 | |
| 735 | private: |
| 736 | std::mutex mLock; |
| 737 | }; |
| 738 | |
| 739 | TEST_F(DispatcherTest, Dispatching) { |
| 740 | FakeSocketDelay::sDelay = 5; |
| 741 | FakeSocketDelay::sReverse = true; |
| 742 | auto factory = std::make_unique<TrackingFakeSocketFactory<FakeSocketDelay>>(); |
| 743 | auto* weak_factory = factory.get(); // Valid as long as dispatcher is in scope. |
| 744 | DnsTlsDispatcher dispatcher(std::move(factory)); |
| 745 | |
| 746 | // Populate a vector of two servers and two socket marks, four combinations |
| 747 | // in total. |
| 748 | std::vector<std::pair<unsigned, DnsTlsServer>> keys; |
| 749 | keys.emplace_back(MARK, SERVER1); |
| 750 | keys.emplace_back(MARK + 1, SERVER1); |
| 751 | keys.emplace_back(MARK, V4ADDR2); |
| 752 | keys.emplace_back(MARK + 1, V4ADDR2); |
| 753 | |
| 754 | // Do several queries on each server. They should all succeed. |
| 755 | std::vector<std::thread> threads; |
| 756 | for (size_t i = 0; i < FakeSocketDelay::sDelay * keys.size(); ++i) { |
| 757 | auto key = keys[i % keys.size()]; |
| 758 | threads.emplace_back([key, i] (DnsTlsDispatcher* dispatcher) { |
| 759 | auto q = make_query(i, SIZE); |
| 760 | bytevec ans(4096); |
| 761 | int resplen = 0; |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 762 | bool connectTriggered = false; |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 763 | unsigned mark = key.first; |
| 764 | const DnsTlsServer& server = key.second; |
| Mike Yu | cb2bb7c | 2019-11-22 20:42:13 +0800 | [diff] [blame] | 765 | auto r = dispatcher->query(server, mark, makeSlice(q), makeSlice(ans), &resplen, |
| 766 | &connectTriggered); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 767 | EXPECT_EQ(DnsTlsTransport::Response::success, r); |
| 768 | EXPECT_EQ(int(q.size()), resplen); |
| 769 | ans.resize(resplen); |
| 770 | EXPECT_EQ(q, ans); |
| 771 | }, &dispatcher); |
| 772 | } |
| 773 | for (auto& thread : threads) { |
| 774 | thread.join(); |
| 775 | } |
| 776 | // We expect that the factory created one socket for each key. |
| 777 | EXPECT_EQ(keys.size(), weak_factory->keys.size()); |
| 778 | for (auto& key : keys) { |
| 779 | EXPECT_EQ(1U, weak_factory->keys.count(key)); |
| 780 | } |
| 781 | } |
| 782 | |
| 783 | // Check DnsTlsServer's comparison logic. |
| 784 | AddressComparator ADDRESS_COMPARATOR; |
| 785 | bool isAddressEqual(const DnsTlsServer& s1, const DnsTlsServer& s2) { |
| 786 | bool cmp1 = ADDRESS_COMPARATOR(s1, s2); |
| 787 | bool cmp2 = ADDRESS_COMPARATOR(s2, s1); |
| 788 | EXPECT_FALSE(cmp1 && cmp2); |
| 789 | return !cmp1 && !cmp2; |
| 790 | } |
| 791 | |
| 792 | void checkUnequal(const DnsTlsServer& s1, const DnsTlsServer& s2) { |
| 793 | EXPECT_TRUE(s1 == s1); |
| 794 | EXPECT_TRUE(s2 == s2); |
| 795 | EXPECT_TRUE(isAddressEqual(s1, s1)); |
| 796 | EXPECT_TRUE(isAddressEqual(s2, s2)); |
| 797 | |
| 798 | EXPECT_TRUE(s1 < s2 ^ s2 < s1); |
| 799 | EXPECT_FALSE(s1 == s2); |
| 800 | EXPECT_FALSE(s2 == s1); |
| 801 | } |
| 802 | |
| 803 | class ServerTest : public BaseTest {}; |
| 804 | |
| 805 | TEST_F(ServerTest, IPv4) { |
| 806 | checkUnequal(V4ADDR1, V4ADDR2); |
| 807 | EXPECT_FALSE(isAddressEqual(V4ADDR1, V4ADDR2)); |
| 808 | } |
| 809 | |
| 810 | TEST_F(ServerTest, IPv6) { |
| 811 | checkUnequal(V6ADDR1, V6ADDR2); |
| 812 | EXPECT_FALSE(isAddressEqual(V6ADDR1, V6ADDR2)); |
| 813 | } |
| 814 | |
| 815 | TEST_F(ServerTest, MixedAddressFamily) { |
| 816 | checkUnequal(V6ADDR1, V4ADDR1); |
| 817 | EXPECT_FALSE(isAddressEqual(V6ADDR1, V4ADDR1)); |
| 818 | } |
| 819 | |
| 820 | TEST_F(ServerTest, IPv6ScopeId) { |
| 821 | DnsTlsServer s1(V6ADDR1), s2(V6ADDR1); |
| 822 | sockaddr_in6* addr1 = reinterpret_cast<sockaddr_in6*>(&s1.ss); |
| 823 | addr1->sin6_scope_id = 1; |
| 824 | sockaddr_in6* addr2 = reinterpret_cast<sockaddr_in6*>(&s2.ss); |
| 825 | addr2->sin6_scope_id = 2; |
| 826 | checkUnequal(s1, s2); |
| 827 | EXPECT_FALSE(isAddressEqual(s1, s2)); |
| 828 | |
| 829 | EXPECT_FALSE(s1.wasExplicitlyConfigured()); |
| 830 | EXPECT_FALSE(s2.wasExplicitlyConfigured()); |
| 831 | } |
| 832 | |
| 833 | TEST_F(ServerTest, IPv6FlowInfo) { |
| 834 | DnsTlsServer s1(V6ADDR1), s2(V6ADDR1); |
| 835 | sockaddr_in6* addr1 = reinterpret_cast<sockaddr_in6*>(&s1.ss); |
| 836 | addr1->sin6_flowinfo = 1; |
| 837 | sockaddr_in6* addr2 = reinterpret_cast<sockaddr_in6*>(&s2.ss); |
| 838 | addr2->sin6_flowinfo = 2; |
| 839 | // All comparisons ignore flowinfo. |
| 840 | EXPECT_EQ(s1, s2); |
| 841 | EXPECT_TRUE(isAddressEqual(s1, s2)); |
| 842 | |
| 843 | EXPECT_FALSE(s1.wasExplicitlyConfigured()); |
| 844 | EXPECT_FALSE(s2.wasExplicitlyConfigured()); |
| 845 | } |
| 846 | |
| 847 | TEST_F(ServerTest, Port) { |
| 848 | DnsTlsServer s1, s2; |
| 849 | parseServer("192.0.2.1", 853, &s1.ss); |
| 850 | parseServer("192.0.2.1", 854, &s2.ss); |
| 851 | checkUnequal(s1, s2); |
| 852 | EXPECT_TRUE(isAddressEqual(s1, s2)); |
| 853 | |
| 854 | DnsTlsServer s3, s4; |
| 855 | parseServer("2001:db8::1", 853, &s3.ss); |
| 856 | parseServer("2001:db8::1", 852, &s4.ss); |
| 857 | checkUnequal(s3, s4); |
| 858 | EXPECT_TRUE(isAddressEqual(s3, s4)); |
| 859 | |
| 860 | EXPECT_FALSE(s1.wasExplicitlyConfigured()); |
| 861 | EXPECT_FALSE(s2.wasExplicitlyConfigured()); |
| 862 | } |
| 863 | |
| 864 | TEST_F(ServerTest, Name) { |
| 865 | DnsTlsServer s1(V4ADDR1), s2(V4ADDR1); |
| 866 | s1.name = SERVERNAME1; |
| 867 | checkUnequal(s1, s2); |
| 868 | s2.name = SERVERNAME2; |
| 869 | checkUnequal(s1, s2); |
| 870 | EXPECT_TRUE(isAddressEqual(s1, s2)); |
| 871 | |
| 872 | EXPECT_TRUE(s1.wasExplicitlyConfigured()); |
| 873 | EXPECT_TRUE(s2.wasExplicitlyConfigured()); |
| 874 | } |
| 875 | |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 876 | TEST(QueryMapTest, Basic) { |
| 877 | DnsTlsQueryMap map; |
| 878 | |
| 879 | EXPECT_TRUE(map.empty()); |
| 880 | |
| 881 | bytevec q0 = make_query(999, SIZE); |
| 882 | bytevec q1 = make_query(888, SIZE); |
| 883 | bytevec q2 = make_query(777, SIZE); |
| 884 | |
| 885 | auto f0 = map.recordQuery(makeSlice(q0)); |
| 886 | auto f1 = map.recordQuery(makeSlice(q1)); |
| 887 | auto f2 = map.recordQuery(makeSlice(q2)); |
| 888 | |
| 889 | // Check return values of recordQuery |
| 890 | EXPECT_EQ(0, f0->query.newId); |
| 891 | EXPECT_EQ(1, f1->query.newId); |
| 892 | EXPECT_EQ(2, f2->query.newId); |
| 893 | |
| 894 | // Check side effects of recordQuery |
| 895 | EXPECT_FALSE(map.empty()); |
| 896 | |
| 897 | auto all = map.getAll(); |
| 898 | EXPECT_EQ(3U, all.size()); |
| 899 | |
| 900 | EXPECT_EQ(0, all[0].newId); |
| 901 | EXPECT_EQ(1, all[1].newId); |
| 902 | EXPECT_EQ(2, all[2].newId); |
| 903 | |
| Mike Yu | 7e08b85 | 2019-10-18 18:27:43 +0800 | [diff] [blame] | 904 | EXPECT_EQ(q0, all[0].query); |
| 905 | EXPECT_EQ(q1, all[1].query); |
| 906 | EXPECT_EQ(q2, all[2].query); |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 907 | |
| 908 | bytevec a0 = make_query(0, SIZE); |
| 909 | bytevec a1 = make_query(1, SIZE); |
| 910 | bytevec a2 = make_query(2, SIZE); |
| 911 | |
| 912 | // Return responses out of order |
| 913 | map.onResponse(a2); |
| 914 | map.onResponse(a0); |
| 915 | map.onResponse(a1); |
| 916 | |
| 917 | EXPECT_TRUE(map.empty()); |
| 918 | |
| 919 | auto r0 = f0->result.get(); |
| 920 | auto r1 = f1->result.get(); |
| 921 | auto r2 = f2->result.get(); |
| 922 | |
| 923 | EXPECT_EQ(DnsTlsQueryMap::Response::success, r0.code); |
| 924 | EXPECT_EQ(DnsTlsQueryMap::Response::success, r1.code); |
| 925 | EXPECT_EQ(DnsTlsQueryMap::Response::success, r2.code); |
| 926 | |
| 927 | const bytevec& d0 = r0.response; |
| 928 | const bytevec& d1 = r1.response; |
| 929 | const bytevec& d2 = r2.response; |
| 930 | |
| 931 | // The ID should match the query |
| 932 | EXPECT_EQ(999, d0[0] << 8 | d0[1]); |
| 933 | EXPECT_EQ(888, d1[0] << 8 | d1[1]); |
| 934 | EXPECT_EQ(777, d2[0] << 8 | d2[1]); |
| 935 | // The body should match the answer |
| 936 | EXPECT_EQ(bytevec(a0.begin() + 2, a0.end()), bytevec(d0.begin() + 2, d0.end())); |
| 937 | EXPECT_EQ(bytevec(a1.begin() + 2, a1.end()), bytevec(d1.begin() + 2, d1.end())); |
| 938 | EXPECT_EQ(bytevec(a2.begin() + 2, a2.end()), bytevec(d2.begin() + 2, d2.end())); |
| 939 | } |
| 940 | |
| 941 | TEST(QueryMapTest, FillHole) { |
| 942 | DnsTlsQueryMap map; |
| 943 | std::vector<std::unique_ptr<DnsTlsQueryMap::QueryFuture>> futures(UINT16_MAX + 1); |
| 944 | for (uint32_t i = 0; i <= UINT16_MAX; ++i) { |
| 945 | futures[i] = map.recordQuery(makeSlice(QUERY)); |
| 946 | ASSERT_TRUE(futures[i]); // answers[i] should be nonnull. |
| 947 | EXPECT_EQ(i, futures[i]->query.newId); |
| 948 | } |
| 949 | |
| 950 | // The map should now be full. |
| 951 | EXPECT_EQ(size_t(UINT16_MAX + 1), map.getAll().size()); |
| 952 | |
| 953 | // Trying to add another query should fail because the map is full. |
| 954 | EXPECT_FALSE(map.recordQuery(makeSlice(QUERY))); |
| 955 | |
| 956 | // Send an answer to query 40000 |
| 957 | auto answer = make_query(40000, SIZE); |
| 958 | map.onResponse(answer); |
| 959 | auto result = futures[40000]->result.get(); |
| 960 | EXPECT_EQ(DnsTlsQueryMap::Response::success, result.code); |
| 961 | EXPECT_EQ(ID, result.response[0] << 8 | result.response[1]); |
| 962 | EXPECT_EQ(bytevec(answer.begin() + 2, answer.end()), |
| 963 | bytevec(result.response.begin() + 2, result.response.end())); |
| 964 | |
| 965 | // There should now be room in the map. |
| 966 | EXPECT_EQ(size_t(UINT16_MAX), map.getAll().size()); |
| 967 | auto f = map.recordQuery(makeSlice(QUERY)); |
| 968 | ASSERT_TRUE(f); |
| 969 | EXPECT_EQ(40000, f->query.newId); |
| 970 | |
| 971 | // The map should now be full again. |
| 972 | EXPECT_EQ(size_t(UINT16_MAX + 1), map.getAll().size()); |
| 973 | EXPECT_FALSE(map.recordQuery(makeSlice(QUERY))); |
| 974 | } |
| 975 | |
| Mike Yu | e93d9ae | 2020-08-25 19:09:51 +0800 | [diff] [blame] | 976 | class DnsTlsSocketTest : public ::testing::Test { |
| 977 | protected: |
| 978 | class MockDnsTlsSocketObserver : public IDnsTlsSocketObserver { |
| 979 | public: |
| 980 | MOCK_METHOD(void, onClosed, (), (override)); |
| 981 | MOCK_METHOD(void, onResponse, (std::vector<uint8_t>), (override)); |
| 982 | }; |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 983 | |
| Mike Yu | e93d9ae | 2020-08-25 19:09:51 +0800 | [diff] [blame] | 984 | DnsTlsSocketTest() { parseServer(kTlsAddr, std::stoi(kTlsPort), &server.ss); } |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 985 | |
| Mike Yu | e93d9ae | 2020-08-25 19:09:51 +0800 | [diff] [blame] | 986 | std::unique_ptr<DnsTlsSocket> makeDnsTlsSocket(IDnsTlsSocketObserver* observer) { |
| 987 | return std::make_unique<DnsTlsSocket>(this->server, MARK, observer, &this->cache); |
| 988 | } |
| 989 | |
| 990 | void enableAsyncHandshake(const std::unique_ptr<DnsTlsSocket>& socket) { |
| 991 | ASSERT_TRUE(socket); |
| 992 | DnsTlsSocket* delegate = socket.get(); |
| 993 | std::lock_guard guard(delegate->mLock); |
| 994 | delegate->mAsyncHandshake = true; |
| 995 | } |
| 996 | |
| 997 | static constexpr char kTlsAddr[] = "127.0.0.3"; |
| 998 | static constexpr char kTlsPort[] = "8530"; // High-numbered port so root isn't required. |
| 999 | static constexpr char kBackendAddr[] = "192.0.2.1"; |
| 1000 | static constexpr char kBackendPort[] = "8531"; // High-numbered port so root isn't required. |
| 1001 | |
| 1002 | test::DnsTlsFrontend tls{kTlsAddr, kTlsPort, kBackendAddr, kBackendPort}; |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 1003 | |
| 1004 | DnsTlsServer server; |
| Mike Yu | e93d9ae | 2020-08-25 19:09:51 +0800 | [diff] [blame] | 1005 | DnsTlsSessionCache cache; |
| 1006 | }; |
| 1007 | |
| 1008 | TEST_F(DnsTlsSocketTest, SlowDestructor) { |
| 1009 | ASSERT_TRUE(tls.startServer()); |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 1010 | |
| Mike Yu | 1b9069c | 2020-08-25 15:17:29 +0800 | [diff] [blame] | 1011 | MockDnsTlsSocketObserver observer; |
| Mike Yu | e93d9ae | 2020-08-25 19:09:51 +0800 | [diff] [blame] | 1012 | auto socket = makeDnsTlsSocket(&observer); |
| 1013 | |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 1014 | ASSERT_TRUE(socket->initialize()); |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 1015 | ASSERT_TRUE(socket->startHandshake()); |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 1016 | |
| 1017 | // Test: Time the socket destructor. This should be fast. |
| 1018 | auto before = std::chrono::steady_clock::now(); |
| Mike Yu | 1b9069c | 2020-08-25 15:17:29 +0800 | [diff] [blame] | 1019 | EXPECT_CALL(observer, onClosed); |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 1020 | socket.reset(); |
| 1021 | auto after = std::chrono::steady_clock::now(); |
| 1022 | auto delay = after - before; |
| chenbruce | aff8584 | 2019-05-31 15:46:42 +0800 | [diff] [blame] | 1023 | LOG(DEBUG) << "Shutdown took " << delay / std::chrono::nanoseconds{1} << "ns"; |
| Ben Schwartz | 62176fd | 2019-01-22 17:32:17 -0500 | [diff] [blame] | 1024 | // Shutdown should complete in milliseconds, but if the shutdown signal is lost |
| 1025 | // it will wait for the timeout, which is expected to take 20seconds. |
| 1026 | EXPECT_LT(delay, std::chrono::seconds{5}); |
| 1027 | } |
| 1028 | |
| Mike Yu | e93d9ae | 2020-08-25 19:09:51 +0800 | [diff] [blame] | 1029 | TEST_F(DnsTlsSocketTest, StartHandshake) { |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 1030 | ASSERT_TRUE(tls.startServer()); |
| 1031 | |
| Mike Yu | e93d9ae | 2020-08-25 19:09:51 +0800 | [diff] [blame] | 1032 | MockDnsTlsSocketObserver observer; |
| 1033 | auto socket = makeDnsTlsSocket(&observer); |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 1034 | |
| 1035 | // Call the function before the call to initialize(). |
| 1036 | EXPECT_FALSE(socket->startHandshake()); |
| 1037 | |
| 1038 | // Call the function after the call to initialize(). |
| 1039 | EXPECT_TRUE(socket->initialize()); |
| 1040 | EXPECT_TRUE(socket->startHandshake()); |
| 1041 | |
| 1042 | // Call both of them again. |
| 1043 | EXPECT_FALSE(socket->initialize()); |
| 1044 | EXPECT_FALSE(socket->startHandshake()); |
| Mike Yu | e93d9ae | 2020-08-25 19:09:51 +0800 | [diff] [blame] | 1045 | |
| 1046 | // Should happen when joining the loop thread in |socket| destruction. |
| 1047 | EXPECT_CALL(observer, onClosed); |
| 1048 | } |
| 1049 | |
| 1050 | TEST_F(DnsTlsSocketTest, ShutdownSignal) { |
| 1051 | ASSERT_TRUE(tls.startServer()); |
| 1052 | |
| 1053 | MockDnsTlsSocketObserver observer; |
| 1054 | std::unique_ptr<DnsTlsSocket> socket; |
| 1055 | |
| 1056 | const auto setupAndStartHandshake = [&]() { |
| 1057 | socket = makeDnsTlsSocket(&observer); |
| 1058 | EXPECT_TRUE(socket->initialize()); |
| 1059 | enableAsyncHandshake(socket); |
| 1060 | EXPECT_TRUE(socket->startHandshake()); |
| 1061 | }; |
| 1062 | const auto triggerShutdown = [&](const std::string& traceLog) { |
| 1063 | SCOPED_TRACE(traceLog); |
| 1064 | auto before = std::chrono::steady_clock::now(); |
| 1065 | EXPECT_CALL(observer, onClosed); |
| 1066 | socket.reset(); |
| 1067 | auto after = std::chrono::steady_clock::now(); |
| 1068 | auto delay = after - before; |
| 1069 | LOG(INFO) << "Shutdown took " << delay / std::chrono::nanoseconds{1} << "ns"; |
| 1070 | EXPECT_LT(delay, std::chrono::seconds{1}); |
| 1071 | }; |
| 1072 | |
| 1073 | tls.setHangOnHandshakeForTesting(true); |
| 1074 | |
| 1075 | // Test 1: Reset the DnsTlsSocket which is doing the handshake. |
| 1076 | setupAndStartHandshake(); |
| 1077 | triggerShutdown("Shutdown handshake w/o query requests"); |
| 1078 | |
| 1079 | // Test 2: Reset the DnsTlsSocket which is doing the handshake with some query requests. |
| 1080 | setupAndStartHandshake(); |
| 1081 | |
| 1082 | // DnsTlsSocket doesn't report the status of pending queries. The decision whether to mark |
| 1083 | // a query request as failed or not is made in DnsTlsTransport. |
| 1084 | EXPECT_CALL(observer, onResponse).Times(0); |
| 1085 | EXPECT_TRUE(socket->query(1, makeSlice(QUERY))); |
| 1086 | EXPECT_TRUE(socket->query(2, makeSlice(QUERY))); |
| 1087 | triggerShutdown("Shutdown handshake w/ query requests"); |
| Mike Yu | 441d937 | 2020-07-15 17:06:22 +0800 | [diff] [blame] | 1088 | } |
| 1089 | |
| Mike Yu | c52739e | 2018-10-19 21:06:32 +0800 | [diff] [blame] | 1090 | } // end of namespace net |
| 1091 | } // end of namespace android |