blob: 3a189cce52a07f2787642cc0ee8a2ddf9c1ed9e5 [file] [log] [blame]
Ben Craig1684c422015-04-24 08:52:44 -05001/*
2 * Licensed to the Apache Software Foundation (ASF) under one
3 * or more contributor license agreements. See the NOTICE file
4 * distributed with this work for additional information
5 * regarding copyright ownership. The ASF licenses this file
6 * to you under the Apache License, Version 2.0 (the
7 * "License"); you may not use this file except in compliance
8 * with the License. You may obtain a copy of the License at
9 *
10 * http://www.apache.org/licenses/LICENSE-2.0
11 *
12 * Unless required by applicable law or agreed to in writing,
13 * software distributed under the License is distributed on an
14 * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15 * KIND, either express or implied. See the License for the
16 * specific language governing permissions and limitations
17 * under the License.
18 */
19
20#define BOOST_TEST_MODULE TSocketInterruptTest
21#include <boost/test/auto_unit_test.hpp>
22
Ben Craig1684c422015-04-24 08:52:44 -050023#include <boost/chrono/duration.hpp>
24#include <boost/date_time/posix_time/posix_time_duration.hpp>
25#include <boost/thread/thread.hpp>
26#include <thrift/transport/TSocket.h>
27#include <thrift/transport/TServerSocket.h>
James E. King, III82ae9572017-08-05 12:23:54 -040028#include <thrift/stdcxx.h>
Ben Craig1684c422015-04-24 08:52:44 -050029
30using apache::thrift::transport::TServerSocket;
31using apache::thrift::transport::TSocket;
32using apache::thrift::transport::TTransport;
33using apache::thrift::transport::TTransportException;
James E. King, III82ae9572017-08-05 12:23:54 -040034using namespace apache::thrift;
Ben Craig1684c422015-04-24 08:52:44 -050035
John Sirois9bd35302016-02-12 21:07:49 -070036BOOST_AUTO_TEST_SUITE(TSocketInterruptTest)
Ben Craig1684c422015-04-24 08:52:44 -050037
James E. King, III82ae9572017-08-05 12:23:54 -040038void readerWorker(stdcxx::shared_ptr<TTransport> tt, uint32_t expectedResult) {
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020039 uint8_t buf[4];
40 BOOST_CHECK_EQUAL(expectedResult, tt->read(buf, 4));
41}
42
James E. King, III82ae9572017-08-05 12:23:54 -040043void readerWorkerMustThrow(stdcxx::shared_ptr<TTransport> tt) {
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020044 try {
Ben Craig1684c422015-04-24 08:52:44 -050045 uint8_t buf[4];
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020046 tt->read(buf, 4);
47 BOOST_ERROR("should not have gotten here");
48 } catch (const TTransportException& tx) {
49 BOOST_CHECK_EQUAL(TTransportException::INTERRUPTED, tx.getType());
50 }
Ben Craig1684c422015-04-24 08:52:44 -050051}
52
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020053BOOST_AUTO_TEST_CASE(test_interruptable_child_read) {
John Sirois9bd35302016-02-12 21:07:49 -070054 TServerSocket sock1("localhost", 0);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020055 sock1.listen();
John Sirois9bd35302016-02-12 21:07:49 -070056 int port = sock1.getPort();
57 TSocket clientSock("localhost", port);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020058 clientSock.open();
James E. King, III82ae9572017-08-05 12:23:54 -040059 stdcxx::shared_ptr<TTransport> accepted = sock1.accept();
60 boost::thread readThread(stdcxx::bind(readerWorkerMustThrow, accepted));
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020061 boost::this_thread::sleep(boost::posix_time::milliseconds(50));
62 // readThread is practically guaranteed to be blocking now
63 sock1.interruptChildren();
64 BOOST_CHECK_MESSAGE(readThread.try_join_for(boost::chrono::milliseconds(200)),
65 "server socket interruptChildren did not interrupt child read");
66 clientSock.close();
67 accepted->close();
68 sock1.close();
Ben Craig1684c422015-04-24 08:52:44 -050069}
70
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020071BOOST_AUTO_TEST_CASE(test_non_interruptable_child_read) {
John Sirois9bd35302016-02-12 21:07:49 -070072 TServerSocket sock1("localhost", 0);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020073 sock1.setInterruptableChildren(false); // returns to pre-THRIFT-2441 behavior
74 sock1.listen();
John Sirois9bd35302016-02-12 21:07:49 -070075 int port = sock1.getPort();
76 TSocket clientSock("localhost", port);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020077 clientSock.open();
James E. King, III82ae9572017-08-05 12:23:54 -040078 stdcxx::shared_ptr<TTransport> accepted = sock1.accept();
79 boost::thread readThread(stdcxx::bind(readerWorker, accepted, 0));
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020080 boost::this_thread::sleep(boost::posix_time::milliseconds(50));
81 // readThread is practically guaranteed to be blocking here
82 sock1.interruptChildren();
83 BOOST_CHECK_MESSAGE(!readThread.try_join_for(boost::chrono::milliseconds(200)),
84 "server socket interruptChildren interrupted child read");
85
86 // only way to proceed is to have the client disconnect
87 clientSock.close();
88 readThread.join();
89 accepted->close();
90 sock1.close();
Ben Craig1684c422015-04-24 08:52:44 -050091}
92
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020093BOOST_AUTO_TEST_CASE(test_cannot_change_after_listen) {
John Sirois9bd35302016-02-12 21:07:49 -070094 TServerSocket sock1("localhost", 0);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +020095 sock1.listen();
96 BOOST_CHECK_THROW(sock1.setInterruptableChildren(false), std::logic_error);
97 sock1.close();
Ben Craig1684c422015-04-24 08:52:44 -050098}
99
James E. King, III82ae9572017-08-05 12:23:54 -0400100void peekerWorker(stdcxx::shared_ptr<TTransport> tt, bool expectedResult) {
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200101 BOOST_CHECK_EQUAL(expectedResult, tt->peek());
Ben Craig1684c422015-04-24 08:52:44 -0500102}
103
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200104BOOST_AUTO_TEST_CASE(test_interruptable_child_peek) {
John Sirois9bd35302016-02-12 21:07:49 -0700105 TServerSocket sock1("localhost", 0);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200106 sock1.listen();
John Sirois9bd35302016-02-12 21:07:49 -0700107 int port = sock1.getPort();
108 TSocket clientSock("localhost", port);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200109 clientSock.open();
James E. King, III82ae9572017-08-05 12:23:54 -0400110 stdcxx::shared_ptr<TTransport> accepted = sock1.accept();
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200111 // peek() will return false if child is interrupted
James E. King, III82ae9572017-08-05 12:23:54 -0400112 boost::thread peekThread(stdcxx::bind(peekerWorker, accepted, false));
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200113 boost::this_thread::sleep(boost::posix_time::milliseconds(50));
114 // peekThread is practically guaranteed to be blocking now
115 sock1.interruptChildren();
116 BOOST_CHECK_MESSAGE(peekThread.try_join_for(boost::chrono::milliseconds(200)),
117 "server socket interruptChildren did not interrupt child peek");
118 clientSock.close();
119 accepted->close();
120 sock1.close();
Ben Craig1684c422015-04-24 08:52:44 -0500121}
122
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200123BOOST_AUTO_TEST_CASE(test_non_interruptable_child_peek) {
John Sirois9bd35302016-02-12 21:07:49 -0700124 TServerSocket sock1("localhost", 0);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200125 sock1.setInterruptableChildren(false); // returns to pre-THRIFT-2441 behavior
126 sock1.listen();
John Sirois9bd35302016-02-12 21:07:49 -0700127 int port = sock1.getPort();
128 TSocket clientSock("localhost", port);
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200129 clientSock.open();
James E. King, III82ae9572017-08-05 12:23:54 -0400130 stdcxx::shared_ptr<TTransport> accepted = sock1.accept();
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200131 // peek() will return false when remote side is closed
James E. King, III82ae9572017-08-05 12:23:54 -0400132 boost::thread peekThread(stdcxx::bind(peekerWorker, accepted, false));
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200133 boost::this_thread::sleep(boost::posix_time::milliseconds(50));
134 // peekThread is practically guaranteed to be blocking now
135 sock1.interruptChildren();
136 BOOST_CHECK_MESSAGE(!peekThread.try_join_for(boost::chrono::milliseconds(200)),
137 "server socket interruptChildren interrupted child peek");
Ben Craig1684c422015-04-24 08:52:44 -0500138
Konrad Grochowski1f6e3802015-05-18 18:10:06 +0200139 // only way to proceed is to have the client disconnect
140 clientSock.close();
141 peekThread.join();
142 accepted->close();
143 sock1.close();
Ben Craig1684c422015-04-24 08:52:44 -0500144}
145
146BOOST_AUTO_TEST_SUITE_END()