Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1 | """Test script for ftplib module.""" |
| 2 | |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 3 | # Modified by Giampaolo Rodola' to test FTP class, IPv6 and TLS |
| 4 | # environment |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 5 | |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 6 | import ftplib |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 7 | import asyncore |
| 8 | import asynchat |
| 9 | import socket |
| 10 | import io |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 11 | import errno |
| 12 | import os |
Antoine Pitrou | a6a4dc8 | 2017-09-07 18:56:24 +0200 | [diff] [blame] | 13 | import threading |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 14 | import time |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 15 | try: |
| 16 | import ssl |
| 17 | except ImportError: |
| 18 | ssl = None |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 19 | |
Serhiy Storchaka | 4376763 | 2013-11-03 21:31:38 +0200 | [diff] [blame] | 20 | from unittest import TestCase, skipUnless |
Benjamin Peterson | ee8712c | 2008-05-20 21:35:26 +0000 | [diff] [blame] | 21 | from test import support |
Hai Shi | e80697d | 2020-05-28 06:10:27 +0800 | [diff] [blame] | 22 | from test.support import threading_helper |
Serhiy Storchaka | 1699491 | 2020-04-25 10:06:29 +0300 | [diff] [blame] | 23 | from test.support import socket_helper |
Hai Shi | 4660597 | 2020-08-04 00:49:18 +0800 | [diff] [blame] | 24 | from test.support import warnings_helper |
Serhiy Storchaka | 1699491 | 2020-04-25 10:06:29 +0300 | [diff] [blame] | 25 | from test.support.socket_helper import HOST, HOSTv6 |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 26 | |
Victor Stinner | 07871b2 | 2019-12-10 20:32:59 +0100 | [diff] [blame] | 27 | TIMEOUT = support.LOOPBACK_TIMEOUT |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 28 | DEFAULT_ENCODING = 'utf-8' |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 29 | # the dummy data returned by server over the data channel when |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 30 | # RETR, LIST, NLST, MLSD commands are issued |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 31 | RETR_DATA = 'abcde12345\r\n' * 1000 + 'non-ascii char \xAE\r\n' |
| 32 | LIST_DATA = 'foo\r\nbar\r\n non-ascii char \xAE\r\n' |
| 33 | NLST_DATA = 'foo\r\nbar\r\n non-ascii char \xAE\r\n' |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 34 | MLSD_DATA = ("type=cdir;perm=el;unique==keVO1+ZF4; test\r\n" |
| 35 | "type=pdir;perm=e;unique==keVO1+d?3; ..\r\n" |
| 36 | "type=OS.unix=slink:/foobar;perm=;unique==keVO1+4G4; foobar\r\n" |
| 37 | "type=OS.unix=chr-13/29;perm=;unique==keVO1+5G4; device\r\n" |
| 38 | "type=OS.unix=blk-11/108;perm=;unique==keVO1+6G4; block\r\n" |
| 39 | "type=file;perm=awr;unique==keVO1+8G4; writable\r\n" |
| 40 | "type=dir;perm=cpmel;unique==keVO1+7G4; promiscuous\r\n" |
| 41 | "type=dir;perm=;unique==keVO1+1t2; no-exec\r\n" |
| 42 | "type=file;perm=r;unique==keVO1+EG4; two words\r\n" |
| 43 | "type=file;perm=r;unique==keVO1+IH4; leading space\r\n" |
| 44 | "type=file;perm=r;unique==keVO1+1G4; file1\r\n" |
| 45 | "type=dir;perm=cpmel;unique==keVO1+7G4; incoming\r\n" |
| 46 | "type=file;perm=r;unique==keVO1+1G4; file2\r\n" |
| 47 | "type=file;perm=r;unique==keVO1+1G4; file3\r\n" |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 48 | "type=file;perm=r;unique==keVO1+1G4; file4\r\n" |
| 49 | "type=dir;perm=cpmel;unique==SGP1; dir \xAE non-ascii char\r\n" |
| 50 | "type=file;perm=r;unique==SGP2; file \xAE non-ascii char\r\n") |
Christian Heimes | 836baa5 | 2008-02-26 08:18:30 +0000 | [diff] [blame] | 51 | |
Christian Heimes | 836baa5 | 2008-02-26 08:18:30 +0000 | [diff] [blame] | 52 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 53 | class DummyDTPHandler(asynchat.async_chat): |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 54 | dtp_conn_closed = False |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 55 | |
| 56 | def __init__(self, conn, baseclass): |
| 57 | asynchat.async_chat.__init__(self, conn) |
| 58 | self.baseclass = baseclass |
| 59 | self.baseclass.last_received_data = '' |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 60 | self.encoding = baseclass.encoding |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 61 | |
| 62 | def handle_read(self): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 63 | new_data = self.recv(1024).decode(self.encoding, 'replace') |
| 64 | self.baseclass.last_received_data += new_data |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 65 | |
| 66 | def handle_close(self): |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 67 | # XXX: this method can be called many times in a row for a single |
| 68 | # connection, including in clear-text (non-TLS) mode. |
| 69 | # (behaviour witnessed with test_data_connection) |
| 70 | if not self.dtp_conn_closed: |
| 71 | self.baseclass.push('226 transfer complete') |
| 72 | self.close() |
| 73 | self.dtp_conn_closed = True |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 74 | |
| 75 | def push(self, what): |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 76 | if self.baseclass.next_data is not None: |
| 77 | what = self.baseclass.next_data |
| 78 | self.baseclass.next_data = None |
| 79 | if not what: |
| 80 | return self.close_when_done() |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 81 | super(DummyDTPHandler, self).push(what.encode(self.encoding)) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 82 | |
Giampaolo Rodolà | d930b63 | 2010-05-06 20:21:57 +0000 | [diff] [blame] | 83 | def handle_error(self): |
Berker Peksag | 8f791d3 | 2014-11-01 10:45:57 +0200 | [diff] [blame] | 84 | raise Exception |
Giampaolo Rodolà | d930b63 | 2010-05-06 20:21:57 +0000 | [diff] [blame] | 85 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 86 | |
| 87 | class DummyFTPHandler(asynchat.async_chat): |
| 88 | |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 89 | dtp_handler = DummyDTPHandler |
| 90 | |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 91 | def __init__(self, conn, encoding=DEFAULT_ENCODING): |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 92 | asynchat.async_chat.__init__(self, conn) |
Giampaolo Rodola' | 0b5c21f | 2011-05-07 19:03:47 +0200 | [diff] [blame] | 93 | # tells the socket to handle urgent data inline (ABOR command) |
| 94 | self.socket.setsockopt(socket.SOL_SOCKET, socket.SO_OOBINLINE, 1) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 95 | self.set_terminator(b"\r\n") |
| 96 | self.in_buffer = [] |
| 97 | self.dtp = None |
| 98 | self.last_received_cmd = None |
| 99 | self.last_received_data = '' |
| 100 | self.next_response = '' |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 101 | self.next_data = None |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 102 | self.rest = None |
Serhiy Storchaka | c30b178 | 2013-10-20 16:58:27 +0300 | [diff] [blame] | 103 | self.next_retr_data = RETR_DATA |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 104 | self.push('220 welcome') |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 105 | self.encoding = encoding |
Gregory P. Smith | 0ab152c | 2021-03-15 11:39:31 -0700 | [diff] [blame] | 106 | # We use this as the string IPv4 address to direct the client |
| 107 | # to in response to a PASV command. To test security behavior. |
| 108 | # https://bugs.python.org/issue43285/. |
| 109 | self.fake_pasv_server_ip = '252.253.254.255' |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 110 | |
| 111 | def collect_incoming_data(self, data): |
| 112 | self.in_buffer.append(data) |
| 113 | |
| 114 | def found_terminator(self): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 115 | line = b''.join(self.in_buffer).decode(self.encoding) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 116 | self.in_buffer = [] |
| 117 | if self.next_response: |
| 118 | self.push(self.next_response) |
| 119 | self.next_response = '' |
| 120 | cmd = line.split(' ')[0].lower() |
| 121 | self.last_received_cmd = cmd |
| 122 | space = line.find(' ') |
| 123 | if space != -1: |
| 124 | arg = line[space + 1:] |
| 125 | else: |
| 126 | arg = "" |
| 127 | if hasattr(self, 'cmd_' + cmd): |
| 128 | method = getattr(self, 'cmd_' + cmd) |
| 129 | method(arg) |
| 130 | else: |
| 131 | self.push('550 command "%s" not understood.' %cmd) |
| 132 | |
| 133 | def handle_error(self): |
Berker Peksag | 8f791d3 | 2014-11-01 10:45:57 +0200 | [diff] [blame] | 134 | raise Exception |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 135 | |
| 136 | def push(self, data): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 137 | asynchat.async_chat.push(self, data.encode(self.encoding) + b'\r\n') |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 138 | |
| 139 | def cmd_port(self, arg): |
| 140 | addr = list(map(int, arg.split(','))) |
| 141 | ip = '%d.%d.%d.%d' %tuple(addr[:4]) |
| 142 | port = (addr[4] * 256) + addr[5] |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 143 | s = socket.create_connection((ip, port), timeout=TIMEOUT) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 144 | self.dtp = self.dtp_handler(s, baseclass=self) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 145 | self.push('200 active data connection established') |
| 146 | |
| 147 | def cmd_pasv(self, arg): |
Giampaolo Rodola | eb7e29f | 2019-04-09 00:34:02 +0200 | [diff] [blame] | 148 | with socket.create_server((self.socket.getsockname()[0], 0)) as sock: |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 149 | sock.settimeout(TIMEOUT) |
Gregory P. Smith | 0ab152c | 2021-03-15 11:39:31 -0700 | [diff] [blame] | 150 | port = sock.getsockname()[1] |
| 151 | ip = self.fake_pasv_server_ip |
Brett Cannon | 918e2d4 | 2010-10-29 23:26:25 +0000 | [diff] [blame] | 152 | ip = ip.replace('.', ','); p1 = port / 256; p2 = port % 256 |
| 153 | self.push('227 entering passive mode (%s,%d,%d)' %(ip, p1, p2)) |
| 154 | conn, addr = sock.accept() |
| 155 | self.dtp = self.dtp_handler(conn, baseclass=self) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 156 | |
| 157 | def cmd_eprt(self, arg): |
| 158 | af, ip, port = arg.split(arg[0])[1:-1] |
| 159 | port = int(port) |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 160 | s = socket.create_connection((ip, port), timeout=TIMEOUT) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 161 | self.dtp = self.dtp_handler(s, baseclass=self) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 162 | self.push('200 active data connection established') |
| 163 | |
| 164 | def cmd_epsv(self, arg): |
Giampaolo Rodola | eb7e29f | 2019-04-09 00:34:02 +0200 | [diff] [blame] | 165 | with socket.create_server((self.socket.getsockname()[0], 0), |
| 166 | family=socket.AF_INET6) as sock: |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 167 | sock.settimeout(TIMEOUT) |
Brett Cannon | 918e2d4 | 2010-10-29 23:26:25 +0000 | [diff] [blame] | 168 | port = sock.getsockname()[1] |
| 169 | self.push('229 entering extended passive mode (|||%d|)' %port) |
| 170 | conn, addr = sock.accept() |
| 171 | self.dtp = self.dtp_handler(conn, baseclass=self) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 172 | |
| 173 | def cmd_echo(self, arg): |
| 174 | # sends back the received string (used by the test suite) |
| 175 | self.push(arg) |
| 176 | |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 177 | def cmd_noop(self, arg): |
| 178 | self.push('200 noop ok') |
| 179 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 180 | def cmd_user(self, arg): |
| 181 | self.push('331 username ok') |
| 182 | |
| 183 | def cmd_pass(self, arg): |
| 184 | self.push('230 password ok') |
| 185 | |
| 186 | def cmd_acct(self, arg): |
| 187 | self.push('230 acct ok') |
| 188 | |
| 189 | def cmd_rnfr(self, arg): |
| 190 | self.push('350 rnfr ok') |
| 191 | |
| 192 | def cmd_rnto(self, arg): |
| 193 | self.push('250 rnto ok') |
| 194 | |
| 195 | def cmd_dele(self, arg): |
| 196 | self.push('250 dele ok') |
| 197 | |
| 198 | def cmd_cwd(self, arg): |
| 199 | self.push('250 cwd ok') |
| 200 | |
| 201 | def cmd_size(self, arg): |
| 202 | self.push('250 1000') |
| 203 | |
| 204 | def cmd_mkd(self, arg): |
| 205 | self.push('257 "%s"' %arg) |
| 206 | |
| 207 | def cmd_rmd(self, arg): |
| 208 | self.push('250 rmd ok') |
| 209 | |
| 210 | def cmd_pwd(self, arg): |
| 211 | self.push('257 "pwd ok"') |
| 212 | |
| 213 | def cmd_type(self, arg): |
Giampaolo Rodolà | f96482e | 2010-08-04 10:36:18 +0000 | [diff] [blame] | 214 | self.push('200 type ok') |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 215 | |
| 216 | def cmd_quit(self, arg): |
| 217 | self.push('221 quit ok') |
| 218 | self.close() |
| 219 | |
Giampaolo Rodola' | 0b5c21f | 2011-05-07 19:03:47 +0200 | [diff] [blame] | 220 | def cmd_abor(self, arg): |
| 221 | self.push('226 abor ok') |
| 222 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 223 | def cmd_stor(self, arg): |
| 224 | self.push('125 stor ok') |
| 225 | |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 226 | def cmd_rest(self, arg): |
| 227 | self.rest = arg |
| 228 | self.push('350 rest ok') |
| 229 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 230 | def cmd_retr(self, arg): |
| 231 | self.push('125 retr ok') |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 232 | if self.rest is not None: |
| 233 | offset = int(self.rest) |
| 234 | else: |
| 235 | offset = 0 |
Serhiy Storchaka | c30b178 | 2013-10-20 16:58:27 +0300 | [diff] [blame] | 236 | self.dtp.push(self.next_retr_data[offset:]) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 237 | self.dtp.close_when_done() |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 238 | self.rest = None |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 239 | |
| 240 | def cmd_list(self, arg): |
| 241 | self.push('125 list ok') |
| 242 | self.dtp.push(LIST_DATA) |
| 243 | self.dtp.close_when_done() |
| 244 | |
| 245 | def cmd_nlst(self, arg): |
| 246 | self.push('125 nlst ok') |
| 247 | self.dtp.push(NLST_DATA) |
| 248 | self.dtp.close_when_done() |
| 249 | |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 250 | def cmd_opts(self, arg): |
| 251 | self.push('200 opts ok') |
| 252 | |
| 253 | def cmd_mlsd(self, arg): |
| 254 | self.push('125 mlsd ok') |
| 255 | self.dtp.push(MLSD_DATA) |
| 256 | self.dtp.close_when_done() |
| 257 | |
Serhiy Storchaka | c30b178 | 2013-10-20 16:58:27 +0300 | [diff] [blame] | 258 | def cmd_setlongretr(self, arg): |
| 259 | # For testing. Next RETR will return long line. |
| 260 | self.next_retr_data = 'x' * int(arg) |
| 261 | self.push('125 setlongretr ok') |
| 262 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 263 | |
| 264 | class DummyFTPServer(asyncore.dispatcher, threading.Thread): |
| 265 | |
| 266 | handler = DummyFTPHandler |
| 267 | |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 268 | def __init__(self, address, af=socket.AF_INET, encoding=DEFAULT_ENCODING): |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 269 | threading.Thread.__init__(self) |
| 270 | asyncore.dispatcher.__init__(self) |
Christian Heimes | 529525f | 2018-05-23 22:24:45 +0200 | [diff] [blame] | 271 | self.daemon = True |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 272 | self.create_socket(af, socket.SOCK_STREAM) |
| 273 | self.bind(address) |
| 274 | self.listen(5) |
| 275 | self.active = False |
| 276 | self.active_lock = threading.Lock() |
| 277 | self.host, self.port = self.socket.getsockname()[:2] |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 278 | self.handler_instance = None |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 279 | self.encoding = encoding |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 280 | |
| 281 | def start(self): |
| 282 | assert not self.active |
| 283 | self.__flag = threading.Event() |
| 284 | threading.Thread.start(self) |
| 285 | self.__flag.wait() |
| 286 | |
| 287 | def run(self): |
| 288 | self.active = True |
| 289 | self.__flag.set() |
| 290 | while self.active and asyncore.socket_map: |
| 291 | self.active_lock.acquire() |
| 292 | asyncore.loop(timeout=0.1, count=1) |
| 293 | self.active_lock.release() |
| 294 | asyncore.close_all(ignore_all=True) |
| 295 | |
| 296 | def stop(self): |
| 297 | assert self.active |
| 298 | self.active = False |
| 299 | self.join() |
| 300 | |
Giampaolo Rodolà | 977c707 | 2010-10-04 21:08:36 +0000 | [diff] [blame] | 301 | def handle_accepted(self, conn, addr): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 302 | self.handler_instance = self.handler(conn, encoding=self.encoding) |
Benjamin Peterson | d06e3b0 | 2008-09-28 21:00:42 +0000 | [diff] [blame] | 303 | |
| 304 | def handle_connect(self): |
| 305 | self.close() |
| 306 | handle_read = handle_connect |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 307 | |
| 308 | def writable(self): |
| 309 | return 0 |
| 310 | |
| 311 | def handle_error(self): |
Berker Peksag | 8f791d3 | 2014-11-01 10:45:57 +0200 | [diff] [blame] | 312 | raise Exception |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 313 | |
| 314 | |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 315 | if ssl is not None: |
| 316 | |
Christian Heimes | e5b5edf | 2013-12-02 02:56:02 +0100 | [diff] [blame] | 317 | CERTFILE = os.path.join(os.path.dirname(__file__), "keycert3.pem") |
| 318 | CAFILE = os.path.join(os.path.dirname(__file__), "pycacert.pem") |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 319 | |
| 320 | class SSLConnection(asyncore.dispatcher): |
| 321 | """An asyncore.dispatcher subclass supporting TLS/SSL.""" |
| 322 | |
| 323 | _ssl_accepting = False |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 324 | _ssl_closing = False |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 325 | |
| 326 | def secure_connection(self): |
Christian Heimes | 2875c60 | 2021-04-19 07:27:10 +0200 | [diff] [blame] | 327 | context = ssl.SSLContext(ssl.PROTOCOL_TLS_SERVER) |
Christian Heimes | d048637 | 2016-09-10 23:23:33 +0200 | [diff] [blame] | 328 | context.load_cert_chain(CERTFILE) |
| 329 | socket = context.wrap_socket(self.socket, |
| 330 | suppress_ragged_eofs=False, |
| 331 | server_side=True, |
| 332 | do_handshake_on_connect=False) |
Giampaolo Rodola' | 096dcb1 | 2011-06-27 11:17:51 +0200 | [diff] [blame] | 333 | self.del_channel() |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 334 | self.set_socket(socket) |
| 335 | self._ssl_accepting = True |
| 336 | |
| 337 | def _do_ssl_handshake(self): |
| 338 | try: |
| 339 | self.socket.do_handshake() |
| 340 | except ssl.SSLError as err: |
| 341 | if err.args[0] in (ssl.SSL_ERROR_WANT_READ, |
| 342 | ssl.SSL_ERROR_WANT_WRITE): |
| 343 | return |
| 344 | elif err.args[0] == ssl.SSL_ERROR_EOF: |
| 345 | return self.handle_close() |
Christian Heimes | 61d478c | 2018-01-27 15:51:38 +0100 | [diff] [blame] | 346 | # TODO: SSLError does not expose alert information |
| 347 | elif "SSLV3_ALERT_BAD_CERTIFICATE" in err.args[1]: |
| 348 | return self.handle_close() |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 349 | raise |
Andrew Svetlov | 0832af6 | 2012-12-18 23:10:48 +0200 | [diff] [blame] | 350 | except OSError as err: |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 351 | if err.args[0] == errno.ECONNABORTED: |
| 352 | return self.handle_close() |
| 353 | else: |
| 354 | self._ssl_accepting = False |
| 355 | |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 356 | def _do_ssl_shutdown(self): |
| 357 | self._ssl_closing = True |
| 358 | try: |
| 359 | self.socket = self.socket.unwrap() |
| 360 | except ssl.SSLError as err: |
| 361 | if err.args[0] in (ssl.SSL_ERROR_WANT_READ, |
| 362 | ssl.SSL_ERROR_WANT_WRITE): |
| 363 | return |
Pablo Galindo | 293dd23 | 2019-11-19 21:34:03 +0000 | [diff] [blame] | 364 | except OSError: |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 365 | # Any "socket error" corresponds to a SSL_ERROR_SYSCALL return |
| 366 | # from OpenSSL's SSL_shutdown(), corresponding to a |
| 367 | # closed socket condition. See also: |
| 368 | # http://www.mail-archive.com/openssl-users@openssl.org/msg60710.html |
| 369 | pass |
| 370 | self._ssl_closing = False |
Benjamin Peterson | b29614e | 2012-10-09 11:16:03 -0400 | [diff] [blame] | 371 | if getattr(self, '_ccc', False) is False: |
Giampaolo Rodola' | 096dcb1 | 2011-06-27 11:17:51 +0200 | [diff] [blame] | 372 | super(SSLConnection, self).close() |
| 373 | else: |
| 374 | pass |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 375 | |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 376 | def handle_read_event(self): |
| 377 | if self._ssl_accepting: |
| 378 | self._do_ssl_handshake() |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 379 | elif self._ssl_closing: |
| 380 | self._do_ssl_shutdown() |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 381 | else: |
| 382 | super(SSLConnection, self).handle_read_event() |
| 383 | |
| 384 | def handle_write_event(self): |
| 385 | if self._ssl_accepting: |
| 386 | self._do_ssl_handshake() |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 387 | elif self._ssl_closing: |
| 388 | self._do_ssl_shutdown() |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 389 | else: |
| 390 | super(SSLConnection, self).handle_write_event() |
| 391 | |
| 392 | def send(self, data): |
| 393 | try: |
| 394 | return super(SSLConnection, self).send(data) |
| 395 | except ssl.SSLError as err: |
Antoine Pitrou | 5733c08 | 2010-03-22 14:49:10 +0000 | [diff] [blame] | 396 | if err.args[0] in (ssl.SSL_ERROR_EOF, ssl.SSL_ERROR_ZERO_RETURN, |
| 397 | ssl.SSL_ERROR_WANT_READ, |
| 398 | ssl.SSL_ERROR_WANT_WRITE): |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 399 | return 0 |
| 400 | raise |
| 401 | |
| 402 | def recv(self, buffer_size): |
| 403 | try: |
| 404 | return super(SSLConnection, self).recv(buffer_size) |
| 405 | except ssl.SSLError as err: |
Antoine Pitrou | 5733c08 | 2010-03-22 14:49:10 +0000 | [diff] [blame] | 406 | if err.args[0] in (ssl.SSL_ERROR_WANT_READ, |
| 407 | ssl.SSL_ERROR_WANT_WRITE): |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 408 | return b'' |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 409 | if err.args[0] in (ssl.SSL_ERROR_EOF, ssl.SSL_ERROR_ZERO_RETURN): |
| 410 | self.handle_close() |
| 411 | return b'' |
| 412 | raise |
| 413 | |
| 414 | def handle_error(self): |
Berker Peksag | 8f791d3 | 2014-11-01 10:45:57 +0200 | [diff] [blame] | 415 | raise Exception |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 416 | |
| 417 | def close(self): |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 418 | if (isinstance(self.socket, ssl.SSLSocket) and |
Christian Heimes | 529525f | 2018-05-23 22:24:45 +0200 | [diff] [blame] | 419 | self.socket._sslobj is not None): |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 420 | self._do_ssl_shutdown() |
Benjamin Peterson | 1bd93a7 | 2010-10-31 19:58:07 +0000 | [diff] [blame] | 421 | else: |
| 422 | super(SSLConnection, self).close() |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 423 | |
| 424 | |
| 425 | class DummyTLS_DTPHandler(SSLConnection, DummyDTPHandler): |
| 426 | """A DummyDTPHandler subclass supporting TLS/SSL.""" |
| 427 | |
| 428 | def __init__(self, conn, baseclass): |
| 429 | DummyDTPHandler.__init__(self, conn, baseclass) |
| 430 | if self.baseclass.secure_data_channel: |
| 431 | self.secure_connection() |
| 432 | |
| 433 | |
| 434 | class DummyTLS_FTPHandler(SSLConnection, DummyFTPHandler): |
| 435 | """A DummyFTPHandler subclass supporting TLS/SSL.""" |
| 436 | |
| 437 | dtp_handler = DummyTLS_DTPHandler |
| 438 | |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 439 | def __init__(self, conn, encoding=DEFAULT_ENCODING): |
| 440 | DummyFTPHandler.__init__(self, conn, encoding=encoding) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 441 | self.secure_data_channel = False |
Giampaolo Rodola' | 096dcb1 | 2011-06-27 11:17:51 +0200 | [diff] [blame] | 442 | self._ccc = False |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 443 | |
| 444 | def cmd_auth(self, line): |
| 445 | """Set up secure control channel.""" |
| 446 | self.push('234 AUTH TLS successful') |
| 447 | self.secure_connection() |
| 448 | |
Giampaolo Rodola' | 096dcb1 | 2011-06-27 11:17:51 +0200 | [diff] [blame] | 449 | def cmd_ccc(self, line): |
| 450 | self.push('220 Reverting back to clear-text') |
| 451 | self._ccc = True |
| 452 | self._do_ssl_shutdown() |
| 453 | |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 454 | def cmd_pbsz(self, line): |
| 455 | """Negotiate size of buffer for secure data transfer. |
| 456 | For TLS/SSL the only valid value for the parameter is '0'. |
| 457 | Any other value is accepted but ignored. |
| 458 | """ |
| 459 | self.push('200 PBSZ=0 successful.') |
| 460 | |
| 461 | def cmd_prot(self, line): |
| 462 | """Setup un/secure data channel.""" |
| 463 | arg = line.upper() |
| 464 | if arg == 'C': |
| 465 | self.push('200 Protection set to Clear') |
| 466 | self.secure_data_channel = False |
| 467 | elif arg == 'P': |
| 468 | self.push('200 Protection set to Private') |
| 469 | self.secure_data_channel = True |
| 470 | else: |
| 471 | self.push("502 Unrecognized PROT type (use C or P).") |
| 472 | |
| 473 | |
| 474 | class DummyTLS_FTPServer(DummyFTPServer): |
| 475 | handler = DummyTLS_FTPHandler |
| 476 | |
| 477 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 478 | class TestFTPClass(TestCase): |
| 479 | |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 480 | def setUp(self, encoding=DEFAULT_ENCODING): |
| 481 | self.server = DummyFTPServer((HOST, 0), encoding=encoding) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 482 | self.server.start() |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 483 | self.client = ftplib.FTP(timeout=TIMEOUT, encoding=encoding) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 484 | self.client.connect(self.server.host, self.server.port) |
| 485 | |
| 486 | def tearDown(self): |
| 487 | self.client.close() |
| 488 | self.server.stop() |
Victor Stinner | d403a29 | 2017-09-13 03:58:25 -0700 | [diff] [blame] | 489 | # Explicitly clear the attribute to prevent dangling thread |
| 490 | self.server = None |
Victor Stinner | 7352864 | 2017-06-30 17:36:57 +0200 | [diff] [blame] | 491 | asyncore.close_all(ignore_all=True) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 492 | |
Giampaolo Rodola' | 8bc8585 | 2012-01-09 17:10:10 +0100 | [diff] [blame] | 493 | def check_data(self, received, expected): |
| 494 | self.assertEqual(len(received), len(expected)) |
| 495 | self.assertEqual(received, expected) |
| 496 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 497 | def test_getwelcome(self): |
| 498 | self.assertEqual(self.client.getwelcome(), '220 welcome') |
| 499 | |
| 500 | def test_sanitize(self): |
| 501 | self.assertEqual(self.client.sanitize('foo'), repr('foo')) |
| 502 | self.assertEqual(self.client.sanitize('pass 12345'), repr('pass *****')) |
| 503 | self.assertEqual(self.client.sanitize('PASS 12345'), repr('PASS *****')) |
| 504 | |
| 505 | def test_exceptions(self): |
Dong-hee Na | 2b1e6e9 | 2017-07-23 02:20:22 +0900 | [diff] [blame] | 506 | self.assertRaises(ValueError, self.client.sendcmd, 'echo 40\r\n0') |
| 507 | self.assertRaises(ValueError, self.client.sendcmd, 'echo 40\n0') |
| 508 | self.assertRaises(ValueError, self.client.sendcmd, 'echo 40\r0') |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 509 | self.assertRaises(ftplib.error_temp, self.client.sendcmd, 'echo 400') |
| 510 | self.assertRaises(ftplib.error_temp, self.client.sendcmd, 'echo 499') |
| 511 | self.assertRaises(ftplib.error_perm, self.client.sendcmd, 'echo 500') |
| 512 | self.assertRaises(ftplib.error_perm, self.client.sendcmd, 'echo 599') |
| 513 | self.assertRaises(ftplib.error_proto, self.client.sendcmd, 'echo 999') |
| 514 | |
| 515 | def test_all_errors(self): |
| 516 | exceptions = (ftplib.error_reply, ftplib.error_temp, ftplib.error_perm, |
Dong-hee Na | 2b1e6e9 | 2017-07-23 02:20:22 +0900 | [diff] [blame] | 517 | ftplib.error_proto, ftplib.Error, OSError, |
| 518 | EOFError) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 519 | for x in exceptions: |
| 520 | try: |
| 521 | raise x('exception not included in all_errors set') |
| 522 | except ftplib.all_errors: |
| 523 | pass |
| 524 | |
| 525 | def test_set_pasv(self): |
| 526 | # passive mode is supposed to be enabled by default |
| 527 | self.assertTrue(self.client.passiveserver) |
| 528 | self.client.set_pasv(True) |
| 529 | self.assertTrue(self.client.passiveserver) |
| 530 | self.client.set_pasv(False) |
| 531 | self.assertFalse(self.client.passiveserver) |
| 532 | |
| 533 | def test_voidcmd(self): |
| 534 | self.client.voidcmd('echo 200') |
| 535 | self.client.voidcmd('echo 299') |
| 536 | self.assertRaises(ftplib.error_reply, self.client.voidcmd, 'echo 199') |
| 537 | self.assertRaises(ftplib.error_reply, self.client.voidcmd, 'echo 300') |
| 538 | |
| 539 | def test_login(self): |
| 540 | self.client.login() |
| 541 | |
| 542 | def test_acct(self): |
| 543 | self.client.acct('passwd') |
| 544 | |
| 545 | def test_rename(self): |
| 546 | self.client.rename('a', 'b') |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 547 | self.server.handler_instance.next_response = '200' |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 548 | self.assertRaises(ftplib.error_reply, self.client.rename, 'a', 'b') |
| 549 | |
| 550 | def test_delete(self): |
| 551 | self.client.delete('foo') |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 552 | self.server.handler_instance.next_response = '199' |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 553 | self.assertRaises(ftplib.error_reply, self.client.delete, 'foo') |
| 554 | |
| 555 | def test_size(self): |
| 556 | self.client.size('foo') |
| 557 | |
| 558 | def test_mkd(self): |
| 559 | dir = self.client.mkd('/foo') |
| 560 | self.assertEqual(dir, '/foo') |
| 561 | |
| 562 | def test_rmd(self): |
| 563 | self.client.rmd('foo') |
| 564 | |
Senthil Kumaran | 0d53860 | 2013-08-12 22:25:27 -0700 | [diff] [blame] | 565 | def test_cwd(self): |
| 566 | dir = self.client.cwd('/foo') |
| 567 | self.assertEqual(dir, '250 cwd ok') |
| 568 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 569 | def test_pwd(self): |
| 570 | dir = self.client.pwd() |
| 571 | self.assertEqual(dir, 'pwd ok') |
| 572 | |
| 573 | def test_quit(self): |
| 574 | self.assertEqual(self.client.quit(), '221 quit ok') |
| 575 | # Ensure the connection gets closed; sock attribute should be None |
| 576 | self.assertEqual(self.client.sock, None) |
| 577 | |
Giampaolo Rodola' | 0b5c21f | 2011-05-07 19:03:47 +0200 | [diff] [blame] | 578 | def test_abort(self): |
| 579 | self.client.abort() |
| 580 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 581 | def test_retrbinary(self): |
| 582 | def callback(data): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 583 | received.append(data.decode(self.client.encoding)) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 584 | received = [] |
| 585 | self.client.retrbinary('retr', callback) |
Giampaolo Rodola' | 8bc8585 | 2012-01-09 17:10:10 +0100 | [diff] [blame] | 586 | self.check_data(''.join(received), RETR_DATA) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 587 | |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 588 | def test_retrbinary_rest(self): |
| 589 | def callback(data): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 590 | received.append(data.decode(self.client.encoding)) |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 591 | for rest in (0, 10, 20): |
| 592 | received = [] |
| 593 | self.client.retrbinary('retr', callback, rest=rest) |
Giampaolo Rodola' | 8bc8585 | 2012-01-09 17:10:10 +0100 | [diff] [blame] | 594 | self.check_data(''.join(received), RETR_DATA[rest:]) |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 595 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 596 | def test_retrlines(self): |
| 597 | received = [] |
| 598 | self.client.retrlines('retr', received.append) |
Giampaolo Rodola' | 8bc8585 | 2012-01-09 17:10:10 +0100 | [diff] [blame] | 599 | self.check_data(''.join(received), RETR_DATA.replace('\r\n', '')) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 600 | |
| 601 | def test_storbinary(self): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 602 | f = io.BytesIO(RETR_DATA.encode(self.client.encoding)) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 603 | self.client.storbinary('stor', f) |
Giampaolo Rodola' | 8bc8585 | 2012-01-09 17:10:10 +0100 | [diff] [blame] | 604 | self.check_data(self.server.handler_instance.last_received_data, RETR_DATA) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 605 | # test new callback arg |
| 606 | flag = [] |
| 607 | f.seek(0) |
| 608 | self.client.storbinary('stor', f, callback=lambda x: flag.append(None)) |
| 609 | self.assertTrue(flag) |
| 610 | |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 611 | def test_storbinary_rest(self): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 612 | data = RETR_DATA.replace('\r\n', '\n').encode(self.client.encoding) |
| 613 | f = io.BytesIO(data) |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 614 | for r in (30, '30'): |
| 615 | f.seek(0) |
| 616 | self.client.storbinary('stor', f, rest=r) |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 617 | self.assertEqual(self.server.handler_instance.rest, str(r)) |
Antoine Pitrou | 648bcd7 | 2009-11-27 13:23:26 +0000 | [diff] [blame] | 618 | |
Giampaolo Rodolà | f96482e | 2010-08-04 10:36:18 +0000 | [diff] [blame] | 619 | def test_storlines(self): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 620 | data = RETR_DATA.replace('\r\n', '\n').encode(self.client.encoding) |
| 621 | f = io.BytesIO(data) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 622 | self.client.storlines('stor', f) |
Giampaolo Rodola' | 8bc8585 | 2012-01-09 17:10:10 +0100 | [diff] [blame] | 623 | self.check_data(self.server.handler_instance.last_received_data, RETR_DATA) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 624 | # test new callback arg |
| 625 | flag = [] |
| 626 | f.seek(0) |
| 627 | self.client.storlines('stor foo', f, callback=lambda x: flag.append(None)) |
| 628 | self.assertTrue(flag) |
| 629 | |
Victor Stinner | ed3a303 | 2013-04-02 22:13:27 +0200 | [diff] [blame] | 630 | f = io.StringIO(RETR_DATA.replace('\r\n', '\n')) |
| 631 | # storlines() expects a binary file, not a text file |
Hai Shi | 4660597 | 2020-08-04 00:49:18 +0800 | [diff] [blame] | 632 | with warnings_helper.check_warnings(('', BytesWarning), quiet=True): |
Florent Xicluna | 5f3fef3 | 2013-07-06 15:08:21 +0200 | [diff] [blame] | 633 | self.assertRaises(TypeError, self.client.storlines, 'stor foo', f) |
Victor Stinner | ed3a303 | 2013-04-02 22:13:27 +0200 | [diff] [blame] | 634 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 635 | def test_nlst(self): |
| 636 | self.client.nlst() |
| 637 | self.assertEqual(self.client.nlst(), NLST_DATA.split('\r\n')[:-1]) |
| 638 | |
| 639 | def test_dir(self): |
| 640 | l = [] |
| 641 | self.client.dir(lambda x: l.append(x)) |
| 642 | self.assertEqual(''.join(l), LIST_DATA.replace('\r\n', '')) |
| 643 | |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 644 | def test_mlsd(self): |
| 645 | list(self.client.mlsd()) |
| 646 | list(self.client.mlsd(path='/')) |
| 647 | list(self.client.mlsd(path='/', facts=['size', 'type'])) |
| 648 | |
| 649 | ls = list(self.client.mlsd()) |
| 650 | for name, facts in ls: |
Giampaolo Rodola' | a55efb3 | 2011-05-07 16:06:59 +0200 | [diff] [blame] | 651 | self.assertIsInstance(name, str) |
| 652 | self.assertIsInstance(facts, dict) |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 653 | self.assertTrue(name) |
Giampaolo Rodola' | a55efb3 | 2011-05-07 16:06:59 +0200 | [diff] [blame] | 654 | self.assertIn('type', facts) |
| 655 | self.assertIn('perm', facts) |
| 656 | self.assertIn('unique', facts) |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 657 | |
| 658 | def set_data(data): |
| 659 | self.server.handler_instance.next_data = data |
| 660 | |
| 661 | def test_entry(line, type=None, perm=None, unique=None, name=None): |
| 662 | type = 'type' if type is None else type |
| 663 | perm = 'perm' if perm is None else perm |
| 664 | unique = 'unique' if unique is None else unique |
| 665 | name = 'name' if name is None else name |
| 666 | set_data(line) |
| 667 | _name, facts = next(self.client.mlsd()) |
| 668 | self.assertEqual(_name, name) |
| 669 | self.assertEqual(facts['type'], type) |
| 670 | self.assertEqual(facts['perm'], perm) |
| 671 | self.assertEqual(facts['unique'], unique) |
| 672 | |
| 673 | # plain |
| 674 | test_entry('type=type;perm=perm;unique=unique; name\r\n') |
| 675 | # "=" in fact value |
| 676 | test_entry('type=ty=pe;perm=perm;unique=unique; name\r\n', type="ty=pe") |
| 677 | test_entry('type==type;perm=perm;unique=unique; name\r\n', type="=type") |
| 678 | test_entry('type=t=y=pe;perm=perm;unique=unique; name\r\n', type="t=y=pe") |
| 679 | test_entry('type=====;perm=perm;unique=unique; name\r\n', type="====") |
| 680 | # spaces in name |
| 681 | test_entry('type=type;perm=perm;unique=unique; na me\r\n', name="na me") |
| 682 | test_entry('type=type;perm=perm;unique=unique; name \r\n', name="name ") |
| 683 | test_entry('type=type;perm=perm;unique=unique; name\r\n', name=" name") |
| 684 | test_entry('type=type;perm=perm;unique=unique; n am e\r\n', name="n am e") |
| 685 | # ";" in name |
| 686 | test_entry('type=type;perm=perm;unique=unique; na;me\r\n', name="na;me") |
| 687 | test_entry('type=type;perm=perm;unique=unique; ;name\r\n', name=";name") |
| 688 | test_entry('type=type;perm=perm;unique=unique; ;name;\r\n', name=";name;") |
| 689 | test_entry('type=type;perm=perm;unique=unique; ;;;;\r\n', name=";;;;") |
| 690 | # case sensitiveness |
| 691 | set_data('Type=type;TyPe=perm;UNIQUE=unique; name\r\n') |
| 692 | _name, facts = next(self.client.mlsd()) |
Giampaolo Rodola' | a55efb3 | 2011-05-07 16:06:59 +0200 | [diff] [blame] | 693 | for x in facts: |
| 694 | self.assertTrue(x.islower()) |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 695 | # no data (directory empty) |
| 696 | set_data('') |
| 697 | self.assertRaises(StopIteration, next, self.client.mlsd()) |
| 698 | set_data('') |
| 699 | for x in self.client.mlsd(): |
Berker Peksag | 8f791d3 | 2014-11-01 10:45:57 +0200 | [diff] [blame] | 700 | self.fail("unexpected data %s" % x) |
Giampaolo Rodola' | d78def9 | 2011-05-06 19:49:08 +0200 | [diff] [blame] | 701 | |
Benjamin Peterson | 3a53fbb | 2008-09-27 22:04:16 +0000 | [diff] [blame] | 702 | def test_makeport(self): |
Brett Cannon | 918e2d4 | 2010-10-29 23:26:25 +0000 | [diff] [blame] | 703 | with self.client.makeport(): |
| 704 | # IPv4 is in use, just make sure send_eprt has not been used |
| 705 | self.assertEqual(self.server.handler_instance.last_received_cmd, |
| 706 | 'port') |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 707 | |
| 708 | def test_makepasv(self): |
| 709 | host, port = self.client.makepasv() |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 710 | conn = socket.create_connection((host, port), timeout=TIMEOUT) |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 711 | conn.close() |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 712 | # IPv4 is in use, just make sure send_epsv has not been used |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 713 | self.assertEqual(self.server.handler_instance.last_received_cmd, 'pasv') |
| 714 | |
Gregory P. Smith | 0ab152c | 2021-03-15 11:39:31 -0700 | [diff] [blame] | 715 | def test_makepasv_issue43285_security_disabled(self): |
| 716 | """Test the opt-in to the old vulnerable behavior.""" |
| 717 | self.client.trust_server_pasv_ipv4_address = True |
| 718 | bad_host, port = self.client.makepasv() |
| 719 | self.assertEqual( |
| 720 | bad_host, self.server.handler_instance.fake_pasv_server_ip) |
| 721 | # Opening and closing a connection keeps the dummy server happy |
| 722 | # instead of timing out on accept. |
| 723 | socket.create_connection((self.client.sock.getpeername()[0], port), |
| 724 | timeout=TIMEOUT).close() |
| 725 | |
| 726 | def test_makepasv_issue43285_security_enabled_default(self): |
| 727 | self.assertFalse(self.client.trust_server_pasv_ipv4_address) |
| 728 | trusted_host, port = self.client.makepasv() |
| 729 | self.assertNotEqual( |
| 730 | trusted_host, self.server.handler_instance.fake_pasv_server_ip) |
| 731 | # Opening and closing a connection keeps the dummy server happy |
| 732 | # instead of timing out on accept. |
| 733 | socket.create_connection((trusted_host, port), timeout=TIMEOUT).close() |
| 734 | |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 735 | def test_with_statement(self): |
| 736 | self.client.quit() |
| 737 | |
| 738 | def is_client_connected(): |
| 739 | if self.client.sock is None: |
| 740 | return False |
| 741 | try: |
| 742 | self.client.sendcmd('noop') |
Andrew Svetlov | 0832af6 | 2012-12-18 23:10:48 +0200 | [diff] [blame] | 743 | except (OSError, EOFError): |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 744 | return False |
| 745 | return True |
| 746 | |
| 747 | # base test |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 748 | with ftplib.FTP(timeout=TIMEOUT) as self.client: |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 749 | self.client.connect(self.server.host, self.server.port) |
| 750 | self.client.sendcmd('noop') |
| 751 | self.assertTrue(is_client_connected()) |
| 752 | self.assertEqual(self.server.handler_instance.last_received_cmd, 'quit') |
| 753 | self.assertFalse(is_client_connected()) |
| 754 | |
| 755 | # QUIT sent inside the with block |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 756 | with ftplib.FTP(timeout=TIMEOUT) as self.client: |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 757 | self.client.connect(self.server.host, self.server.port) |
| 758 | self.client.sendcmd('noop') |
| 759 | self.client.quit() |
| 760 | self.assertEqual(self.server.handler_instance.last_received_cmd, 'quit') |
| 761 | self.assertFalse(is_client_connected()) |
| 762 | |
| 763 | # force a wrong response code to be sent on QUIT: error_perm |
| 764 | # is expected and the connection is supposed to be closed |
| 765 | try: |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 766 | with ftplib.FTP(timeout=TIMEOUT) as self.client: |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 767 | self.client.connect(self.server.host, self.server.port) |
| 768 | self.client.sendcmd('noop') |
| 769 | self.server.handler_instance.next_response = '550 error on quit' |
| 770 | except ftplib.error_perm as err: |
| 771 | self.assertEqual(str(err), '550 error on quit') |
| 772 | else: |
| 773 | self.fail('Exception not raised') |
| 774 | # needed to give the threaded server some time to set the attribute |
| 775 | # which otherwise would still be == 'noop' |
| 776 | time.sleep(0.1) |
| 777 | self.assertEqual(self.server.handler_instance.last_received_cmd, 'quit') |
| 778 | self.assertFalse(is_client_connected()) |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 779 | |
Giampaolo Rodolà | 396ff06 | 2011-02-28 19:19:51 +0000 | [diff] [blame] | 780 | def test_source_address(self): |
| 781 | self.client.quit() |
Serhiy Storchaka | 1699491 | 2020-04-25 10:06:29 +0300 | [diff] [blame] | 782 | port = socket_helper.find_unused_port() |
Antoine Pitrou | 6dca527 | 2011-04-03 18:29:45 +0200 | [diff] [blame] | 783 | try: |
| 784 | self.client.connect(self.server.host, self.server.port, |
| 785 | source_address=(HOST, port)) |
| 786 | self.assertEqual(self.client.sock.getsockname()[1], port) |
| 787 | self.client.quit() |
Andrew Svetlov | f7a17b4 | 2012-12-25 16:47:37 +0200 | [diff] [blame] | 788 | except OSError as e: |
Antoine Pitrou | 6dca527 | 2011-04-03 18:29:45 +0200 | [diff] [blame] | 789 | if e.errno == errno.EADDRINUSE: |
| 790 | self.skipTest("couldn't bind to port %d" % port) |
| 791 | raise |
Giampaolo Rodolà | 396ff06 | 2011-02-28 19:19:51 +0000 | [diff] [blame] | 792 | |
| 793 | def test_source_address_passive_connection(self): |
Serhiy Storchaka | 1699491 | 2020-04-25 10:06:29 +0300 | [diff] [blame] | 794 | port = socket_helper.find_unused_port() |
Giampaolo Rodolà | 396ff06 | 2011-02-28 19:19:51 +0000 | [diff] [blame] | 795 | self.client.source_address = (HOST, port) |
Antoine Pitrou | 6dca527 | 2011-04-03 18:29:45 +0200 | [diff] [blame] | 796 | try: |
| 797 | with self.client.transfercmd('list') as sock: |
| 798 | self.assertEqual(sock.getsockname()[1], port) |
Andrew Svetlov | f7a17b4 | 2012-12-25 16:47:37 +0200 | [diff] [blame] | 799 | except OSError as e: |
Antoine Pitrou | 6dca527 | 2011-04-03 18:29:45 +0200 | [diff] [blame] | 800 | if e.errno == errno.EADDRINUSE: |
| 801 | self.skipTest("couldn't bind to port %d" % port) |
| 802 | raise |
Giampaolo Rodolà | 396ff06 | 2011-02-28 19:19:51 +0000 | [diff] [blame] | 803 | |
Giampaolo Rodolà | bbc4782 | 2010-08-23 22:10:32 +0000 | [diff] [blame] | 804 | def test_parse257(self): |
| 805 | self.assertEqual(ftplib.parse257('257 "/foo/bar"'), '/foo/bar') |
| 806 | self.assertEqual(ftplib.parse257('257 "/foo/bar" created'), '/foo/bar') |
| 807 | self.assertEqual(ftplib.parse257('257 ""'), '') |
| 808 | self.assertEqual(ftplib.parse257('257 "" created'), '') |
| 809 | self.assertRaises(ftplib.error_reply, ftplib.parse257, '250 "/foo/bar"') |
| 810 | # The 257 response is supposed to include the directory |
| 811 | # name and in case it contains embedded double-quotes |
| 812 | # they must be doubled (see RFC-959, chapter 7, appendix 2). |
| 813 | self.assertEqual(ftplib.parse257('257 "/foo/b""ar"'), '/foo/b"ar') |
| 814 | self.assertEqual(ftplib.parse257('257 "/foo/b""ar" created'), '/foo/b"ar') |
| 815 | |
Serhiy Storchaka | c30b178 | 2013-10-20 16:58:27 +0300 | [diff] [blame] | 816 | def test_line_too_long(self): |
| 817 | self.assertRaises(ftplib.Error, self.client.sendcmd, |
| 818 | 'x' * self.client.maxline * 2) |
| 819 | |
| 820 | def test_retrlines_too_long(self): |
| 821 | self.client.sendcmd('SETLONGRETR %d' % (self.client.maxline * 2)) |
| 822 | received = [] |
| 823 | self.assertRaises(ftplib.Error, |
| 824 | self.client.retrlines, 'retr', received.append) |
| 825 | |
| 826 | def test_storlines_too_long(self): |
| 827 | f = io.BytesIO(b'x' * self.client.maxline * 2) |
| 828 | self.assertRaises(ftplib.Error, self.client.storlines, 'stor', f) |
| 829 | |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 830 | def test_encoding_param(self): |
| 831 | encodings = ['latin-1', 'utf-8'] |
| 832 | for encoding in encodings: |
| 833 | with self.subTest(encoding=encoding): |
| 834 | self.tearDown() |
| 835 | self.setUp(encoding=encoding) |
| 836 | self.assertEqual(encoding, self.client.encoding) |
| 837 | self.test_retrbinary() |
| 838 | self.test_storbinary() |
| 839 | self.test_retrlines() |
| 840 | new_dir = self.client.mkd('/non-ascii dir \xAE') |
| 841 | self.check_data(new_dir, '/non-ascii dir \xAE') |
| 842 | # Check default encoding |
| 843 | client = ftplib.FTP(timeout=TIMEOUT) |
| 844 | self.assertEqual(DEFAULT_ENCODING, client.encoding) |
| 845 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 846 | |
Serhiy Storchaka | 1699491 | 2020-04-25 10:06:29 +0300 | [diff] [blame] | 847 | @skipUnless(socket_helper.IPV6_ENABLED, "IPv6 not enabled") |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 848 | class TestIPv6Environment(TestCase): |
| 849 | |
| 850 | def setUp(self): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 851 | self.server = DummyFTPServer((HOSTv6, 0), |
| 852 | af=socket.AF_INET6, |
| 853 | encoding=DEFAULT_ENCODING) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 854 | self.server.start() |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 855 | self.client = ftplib.FTP(timeout=TIMEOUT, encoding=DEFAULT_ENCODING) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 856 | self.client.connect(self.server.host, self.server.port) |
| 857 | |
| 858 | def tearDown(self): |
| 859 | self.client.close() |
| 860 | self.server.stop() |
Victor Stinner | d403a29 | 2017-09-13 03:58:25 -0700 | [diff] [blame] | 861 | # Explicitly clear the attribute to prevent dangling thread |
| 862 | self.server = None |
Victor Stinner | 7352864 | 2017-06-30 17:36:57 +0200 | [diff] [blame] | 863 | asyncore.close_all(ignore_all=True) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 864 | |
| 865 | def test_af(self): |
| 866 | self.assertEqual(self.client.af, socket.AF_INET6) |
| 867 | |
| 868 | def test_makeport(self): |
Brett Cannon | 918e2d4 | 2010-10-29 23:26:25 +0000 | [diff] [blame] | 869 | with self.client.makeport(): |
| 870 | self.assertEqual(self.server.handler_instance.last_received_cmd, |
| 871 | 'eprt') |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 872 | |
| 873 | def test_makepasv(self): |
| 874 | host, port = self.client.makepasv() |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 875 | conn = socket.create_connection((host, port), timeout=TIMEOUT) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 876 | conn.close() |
Giampaolo Rodolà | bd576b7 | 2010-05-10 14:53:29 +0000 | [diff] [blame] | 877 | self.assertEqual(self.server.handler_instance.last_received_cmd, 'epsv') |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 878 | |
| 879 | def test_transfer(self): |
| 880 | def retr(): |
| 881 | def callback(data): |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 882 | received.append(data.decode(self.client.encoding)) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 883 | received = [] |
| 884 | self.client.retrbinary('retr', callback) |
Giampaolo Rodola' | 8bc8585 | 2012-01-09 17:10:10 +0100 | [diff] [blame] | 885 | self.assertEqual(len(''.join(received)), len(RETR_DATA)) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 886 | self.assertEqual(''.join(received), RETR_DATA) |
| 887 | self.client.set_pasv(True) |
| 888 | retr() |
| 889 | self.client.set_pasv(False) |
| 890 | retr() |
| 891 | |
| 892 | |
Serhiy Storchaka | 4376763 | 2013-11-03 21:31:38 +0200 | [diff] [blame] | 893 | @skipUnless(ssl, "SSL not available") |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 894 | class TestTLS_FTPClassMixin(TestFTPClass): |
| 895 | """Repeat TestFTPClass tests starting the TLS layer for both control |
| 896 | and data connections first. |
| 897 | """ |
| 898 | |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 899 | def setUp(self, encoding=DEFAULT_ENCODING): |
| 900 | self.server = DummyTLS_FTPServer((HOST, 0), encoding=encoding) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 901 | self.server.start() |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 902 | self.client = ftplib.FTP_TLS(timeout=TIMEOUT, encoding=encoding) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 903 | self.client.connect(self.server.host, self.server.port) |
| 904 | # enable TLS |
| 905 | self.client.auth() |
| 906 | self.client.prot_p() |
| 907 | |
| 908 | |
Serhiy Storchaka | 4376763 | 2013-11-03 21:31:38 +0200 | [diff] [blame] | 909 | @skipUnless(ssl, "SSL not available") |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 910 | class TestTLS_FTPClass(TestCase): |
| 911 | """Specific TLS_FTP class tests.""" |
| 912 | |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 913 | def setUp(self, encoding=DEFAULT_ENCODING): |
| 914 | self.server = DummyTLS_FTPServer((HOST, 0), encoding=encoding) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 915 | self.server.start() |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 916 | self.client = ftplib.FTP_TLS(timeout=TIMEOUT) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 917 | self.client.connect(self.server.host, self.server.port) |
| 918 | |
| 919 | def tearDown(self): |
| 920 | self.client.close() |
| 921 | self.server.stop() |
Victor Stinner | d403a29 | 2017-09-13 03:58:25 -0700 | [diff] [blame] | 922 | # Explicitly clear the attribute to prevent dangling thread |
| 923 | self.server = None |
Victor Stinner | 7352864 | 2017-06-30 17:36:57 +0200 | [diff] [blame] | 924 | asyncore.close_all(ignore_all=True) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 925 | |
| 926 | def test_control_connection(self): |
Ezio Melotti | e961593 | 2010-01-24 19:26:24 +0000 | [diff] [blame] | 927 | self.assertNotIsInstance(self.client.sock, ssl.SSLSocket) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 928 | self.client.auth() |
Ezio Melotti | e961593 | 2010-01-24 19:26:24 +0000 | [diff] [blame] | 929 | self.assertIsInstance(self.client.sock, ssl.SSLSocket) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 930 | |
| 931 | def test_data_connection(self): |
| 932 | # clear text |
Brett Cannon | 918e2d4 | 2010-10-29 23:26:25 +0000 | [diff] [blame] | 933 | with self.client.transfercmd('list') as sock: |
| 934 | self.assertNotIsInstance(sock, ssl.SSLSocket) |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 935 | self.assertEqual(sock.recv(1024), |
| 936 | LIST_DATA.encode(self.client.encoding)) |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 937 | self.assertEqual(self.client.voidresp(), "226 transfer complete") |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 938 | |
| 939 | # secured, after PROT P |
| 940 | self.client.prot_p() |
Brett Cannon | 918e2d4 | 2010-10-29 23:26:25 +0000 | [diff] [blame] | 941 | with self.client.transfercmd('list') as sock: |
| 942 | self.assertIsInstance(sock, ssl.SSLSocket) |
Christian Heimes | 1590c39 | 2018-08-16 19:43:44 +0200 | [diff] [blame] | 943 | # consume from SSL socket to finalize handshake and avoid |
| 944 | # "SSLError [SSL] shutdown while in init" |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 945 | self.assertEqual(sock.recv(1024), |
| 946 | LIST_DATA.encode(self.client.encoding)) |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 947 | self.assertEqual(self.client.voidresp(), "226 transfer complete") |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 948 | |
| 949 | # PROT C is issued, the connection must be in cleartext again |
| 950 | self.client.prot_c() |
Brett Cannon | 918e2d4 | 2010-10-29 23:26:25 +0000 | [diff] [blame] | 951 | with self.client.transfercmd('list') as sock: |
| 952 | self.assertNotIsInstance(sock, ssl.SSLSocket) |
Sebastian Pedersen | a1a0eb4 | 2020-04-14 01:07:56 +0200 | [diff] [blame] | 953 | self.assertEqual(sock.recv(1024), |
| 954 | LIST_DATA.encode(self.client.encoding)) |
Antoine Pitrou | 2c4f98b | 2010-04-23 00:16:21 +0000 | [diff] [blame] | 955 | self.assertEqual(self.client.voidresp(), "226 transfer complete") |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 956 | |
| 957 | def test_login(self): |
| 958 | # login() is supposed to implicitly secure the control connection |
Ezio Melotti | e961593 | 2010-01-24 19:26:24 +0000 | [diff] [blame] | 959 | self.assertNotIsInstance(self.client.sock, ssl.SSLSocket) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 960 | self.client.login() |
Ezio Melotti | e961593 | 2010-01-24 19:26:24 +0000 | [diff] [blame] | 961 | self.assertIsInstance(self.client.sock, ssl.SSLSocket) |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 962 | # make sure that AUTH TLS doesn't get issued again |
| 963 | self.client.login() |
| 964 | |
| 965 | def test_auth_issued_twice(self): |
| 966 | self.client.auth() |
| 967 | self.assertRaises(ValueError, self.client.auth) |
| 968 | |
Giampaolo Rodolà | a67299e | 2010-05-26 18:06:04 +0000 | [diff] [blame] | 969 | def test_context(self): |
| 970 | self.client.quit() |
Christian Heimes | a170fa1 | 2017-09-15 20:27:30 +0200 | [diff] [blame] | 971 | ctx = ssl.SSLContext(ssl.PROTOCOL_TLS_CLIENT) |
| 972 | ctx.check_hostname = False |
| 973 | ctx.verify_mode = ssl.CERT_NONE |
Giampaolo Rodolà | a67299e | 2010-05-26 18:06:04 +0000 | [diff] [blame] | 974 | self.assertRaises(ValueError, ftplib.FTP_TLS, keyfile=CERTFILE, |
| 975 | context=ctx) |
| 976 | self.assertRaises(ValueError, ftplib.FTP_TLS, certfile=CERTFILE, |
| 977 | context=ctx) |
| 978 | self.assertRaises(ValueError, ftplib.FTP_TLS, certfile=CERTFILE, |
| 979 | keyfile=CERTFILE, context=ctx) |
| 980 | |
Giampaolo Rodola' | 0d4f08c | 2013-05-16 15:12:01 +0200 | [diff] [blame] | 981 | self.client = ftplib.FTP_TLS(context=ctx, timeout=TIMEOUT) |
Giampaolo Rodolà | a67299e | 2010-05-26 18:06:04 +0000 | [diff] [blame] | 982 | self.client.connect(self.server.host, self.server.port) |
| 983 | self.assertNotIsInstance(self.client.sock, ssl.SSLSocket) |
| 984 | self.client.auth() |
| 985 | self.assertIs(self.client.sock.context, ctx) |
| 986 | self.assertIsInstance(self.client.sock, ssl.SSLSocket) |
| 987 | |
| 988 | self.client.prot_p() |
Brett Cannon | 918e2d4 | 2010-10-29 23:26:25 +0000 | [diff] [blame] | 989 | with self.client.transfercmd('list') as sock: |
| 990 | self.assertIs(sock.context, ctx) |
| 991 | self.assertIsInstance(sock, ssl.SSLSocket) |
Giampaolo Rodolà | a67299e | 2010-05-26 18:06:04 +0000 | [diff] [blame] | 992 | |
Giampaolo Rodola' | 096dcb1 | 2011-06-27 11:17:51 +0200 | [diff] [blame] | 993 | def test_ccc(self): |
| 994 | self.assertRaises(ValueError, self.client.ccc) |
| 995 | self.client.login(secure=True) |
| 996 | self.assertIsInstance(self.client.sock, ssl.SSLSocket) |
| 997 | self.client.ccc() |
| 998 | self.assertRaises(ValueError, self.client.sock.unwrap) |
Giampaolo Rodola' | 096dcb1 | 2011-06-27 11:17:51 +0200 | [diff] [blame] | 999 | |
Victor Stinner | 51500f3 | 2018-01-29 13:21:34 +0100 | [diff] [blame] | 1000 | @skipUnless(False, "FIXME: bpo-32706") |
Christian Heimes | e5b5edf | 2013-12-02 02:56:02 +0100 | [diff] [blame] | 1001 | def test_check_hostname(self): |
| 1002 | self.client.quit() |
Christian Heimes | a170fa1 | 2017-09-15 20:27:30 +0200 | [diff] [blame] | 1003 | ctx = ssl.SSLContext(ssl.PROTOCOL_TLS_CLIENT) |
| 1004 | self.assertEqual(ctx.verify_mode, ssl.CERT_REQUIRED) |
| 1005 | self.assertEqual(ctx.check_hostname, True) |
Christian Heimes | e5b5edf | 2013-12-02 02:56:02 +0100 | [diff] [blame] | 1006 | ctx.load_verify_locations(CAFILE) |
| 1007 | self.client = ftplib.FTP_TLS(context=ctx, timeout=TIMEOUT) |
| 1008 | |
| 1009 | # 127.0.0.1 doesn't match SAN |
| 1010 | self.client.connect(self.server.host, self.server.port) |
| 1011 | with self.assertRaises(ssl.CertificateError): |
| 1012 | self.client.auth() |
| 1013 | # exception quits connection |
| 1014 | |
| 1015 | self.client.connect(self.server.host, self.server.port) |
| 1016 | self.client.prot_p() |
| 1017 | with self.assertRaises(ssl.CertificateError): |
| 1018 | with self.client.transfercmd("list") as sock: |
| 1019 | pass |
| 1020 | self.client.quit() |
| 1021 | |
| 1022 | self.client.connect("localhost", self.server.port) |
| 1023 | self.client.auth() |
| 1024 | self.client.quit() |
| 1025 | |
| 1026 | self.client.connect("localhost", self.server.port) |
| 1027 | self.client.prot_p() |
| 1028 | with self.client.transfercmd("list") as sock: |
| 1029 | pass |
| 1030 | |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 1031 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1032 | class TestTimeouts(TestCase): |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1033 | |
| 1034 | def setUp(self): |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1035 | self.evt = threading.Event() |
Christian Heimes | 5e69685 | 2008-04-09 08:37:03 +0000 | [diff] [blame] | 1036 | self.sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM) |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1037 | self.sock.settimeout(20) |
Serhiy Storchaka | 1699491 | 2020-04-25 10:06:29 +0300 | [diff] [blame] | 1038 | self.port = socket_helper.bind_port(self.sock) |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1039 | self.server_thread = threading.Thread(target=self.server) |
Christian Heimes | 529525f | 2018-05-23 22:24:45 +0200 | [diff] [blame] | 1040 | self.server_thread.daemon = True |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1041 | self.server_thread.start() |
Christian Heimes | 836baa5 | 2008-02-26 08:18:30 +0000 | [diff] [blame] | 1042 | # Wait for the server to be ready. |
| 1043 | self.evt.wait() |
| 1044 | self.evt.clear() |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1045 | self.old_port = ftplib.FTP.port |
Christian Heimes | 5e69685 | 2008-04-09 08:37:03 +0000 | [diff] [blame] | 1046 | ftplib.FTP.port = self.port |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1047 | |
| 1048 | def tearDown(self): |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1049 | ftplib.FTP.port = self.old_port |
| 1050 | self.server_thread.join() |
Victor Stinner | b157ce1 | 2017-09-13 06:43:58 -0700 | [diff] [blame] | 1051 | # Explicitly clear the attribute to prevent dangling thread |
| 1052 | self.server_thread = None |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1053 | |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1054 | def server(self): |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1055 | # This method sets the evt 3 times: |
| 1056 | # 1) when the connection is ready to be accepted. |
| 1057 | # 2) when it is safe for the caller to close the connection |
| 1058 | # 3) when we have closed the socket |
Charles-François Natali | 6e20460 | 2014-07-23 19:28:13 +0100 | [diff] [blame] | 1059 | self.sock.listen() |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1060 | # (1) Signal the caller that we are ready to accept the connection. |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1061 | self.evt.set() |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1062 | try: |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1063 | conn, addr = self.sock.accept() |
Christian Heimes | 03c8ddd | 2020-11-20 09:26:07 +0100 | [diff] [blame] | 1064 | except TimeoutError: |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1065 | pass |
| 1066 | else: |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1067 | conn.sendall(b"1 Hola mundo\n") |
| 1068 | conn.shutdown(socket.SHUT_WR) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1069 | # (2) Signal the caller that it is safe to close the socket. |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1070 | self.evt.set() |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1071 | conn.close() |
| 1072 | finally: |
Antoine Pitrou | 08d0272 | 2012-12-19 20:44:02 +0100 | [diff] [blame] | 1073 | self.sock.close() |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1074 | |
| 1075 | def testTimeoutDefault(self): |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1076 | # default -- use global socket timeout |
Serhiy Storchaka | 25d8aea | 2014-02-08 14:50:08 +0200 | [diff] [blame] | 1077 | self.assertIsNone(socket.getdefaulttimeout()) |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1078 | socket.setdefaulttimeout(30) |
| 1079 | try: |
Antoine Pitrou | f6fbf56 | 2013-08-22 00:39:46 +0200 | [diff] [blame] | 1080 | ftp = ftplib.FTP(HOST) |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1081 | finally: |
| 1082 | socket.setdefaulttimeout(None) |
| 1083 | self.assertEqual(ftp.sock.gettimeout(), 30) |
| 1084 | self.evt.wait() |
| 1085 | ftp.close() |
| 1086 | |
| 1087 | def testTimeoutNone(self): |
| 1088 | # no timeout -- do not use global socket timeout |
Serhiy Storchaka | 25d8aea | 2014-02-08 14:50:08 +0200 | [diff] [blame] | 1089 | self.assertIsNone(socket.getdefaulttimeout()) |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1090 | socket.setdefaulttimeout(30) |
| 1091 | try: |
Antoine Pitrou | f6fbf56 | 2013-08-22 00:39:46 +0200 | [diff] [blame] | 1092 | ftp = ftplib.FTP(HOST, timeout=None) |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1093 | finally: |
| 1094 | socket.setdefaulttimeout(None) |
Serhiy Storchaka | 25d8aea | 2014-02-08 14:50:08 +0200 | [diff] [blame] | 1095 | self.assertIsNone(ftp.sock.gettimeout()) |
Christian Heimes | 836baa5 | 2008-02-26 08:18:30 +0000 | [diff] [blame] | 1096 | self.evt.wait() |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1097 | ftp.close() |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1098 | |
| 1099 | def testTimeoutValue(self): |
| 1100 | # a value |
Christian Heimes | 5e69685 | 2008-04-09 08:37:03 +0000 | [diff] [blame] | 1101 | ftp = ftplib.FTP(HOST, timeout=30) |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1102 | self.assertEqual(ftp.sock.gettimeout(), 30) |
Christian Heimes | 836baa5 | 2008-02-26 08:18:30 +0000 | [diff] [blame] | 1103 | self.evt.wait() |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1104 | ftp.close() |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1105 | |
Dong-hee Na | a190e2a | 2020-01-14 04:34:34 +0900 | [diff] [blame] | 1106 | # bpo-39259 |
| 1107 | with self.assertRaises(ValueError): |
| 1108 | ftplib.FTP(HOST, timeout=0) |
| 1109 | |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1110 | def testTimeoutConnect(self): |
| 1111 | ftp = ftplib.FTP() |
Christian Heimes | 5e69685 | 2008-04-09 08:37:03 +0000 | [diff] [blame] | 1112 | ftp.connect(HOST, timeout=30) |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1113 | self.assertEqual(ftp.sock.gettimeout(), 30) |
Christian Heimes | 836baa5 | 2008-02-26 08:18:30 +0000 | [diff] [blame] | 1114 | self.evt.wait() |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1115 | ftp.close() |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1116 | |
| 1117 | def testTimeoutDifferentOrder(self): |
| 1118 | ftp = ftplib.FTP(timeout=30) |
Christian Heimes | 5e69685 | 2008-04-09 08:37:03 +0000 | [diff] [blame] | 1119 | ftp.connect(HOST) |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1120 | self.assertEqual(ftp.sock.gettimeout(), 30) |
Christian Heimes | 836baa5 | 2008-02-26 08:18:30 +0000 | [diff] [blame] | 1121 | self.evt.wait() |
Georg Brandl | f78e02b | 2008-06-10 17:40:04 +0000 | [diff] [blame] | 1122 | ftp.close() |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1123 | |
| 1124 | def testTimeoutDirectAccess(self): |
| 1125 | ftp = ftplib.FTP() |
| 1126 | ftp.timeout = 30 |
Christian Heimes | 5e69685 | 2008-04-09 08:37:03 +0000 | [diff] [blame] | 1127 | ftp.connect(HOST) |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1128 | self.assertEqual(ftp.sock.gettimeout(), 30) |
Christian Heimes | 836baa5 | 2008-02-26 08:18:30 +0000 | [diff] [blame] | 1129 | self.evt.wait() |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1130 | ftp.close() |
| 1131 | |
| 1132 | |
Martin Panter | 19e69c5 | 2015-11-14 12:46:42 +0000 | [diff] [blame] | 1133 | class MiscTestCase(TestCase): |
| 1134 | def test__all__(self): |
Victor Stinner | fbf43f0 | 2020-08-17 07:20:40 +0200 | [diff] [blame] | 1135 | not_exported = { |
| 1136 | 'MSG_OOB', 'FTP_PORT', 'MAXLINE', 'CRLF', 'B_CRLF', 'Error', |
| 1137 | 'parse150', 'parse227', 'parse229', 'parse257', 'print_line', |
| 1138 | 'ftpcp', 'test'} |
| 1139 | support.check__all__(self, ftplib, not_exported=not_exported) |
Martin Panter | 19e69c5 | 2015-11-14 12:46:42 +0000 | [diff] [blame] | 1140 | |
| 1141 | |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1142 | def test_main(): |
Berker Peksag | 8f791d3 | 2014-11-01 10:45:57 +0200 | [diff] [blame] | 1143 | tests = [TestFTPClass, TestTimeouts, |
Serhiy Storchaka | 4376763 | 2013-11-03 21:31:38 +0200 | [diff] [blame] | 1144 | TestIPv6Environment, |
Martin Panter | 19e69c5 | 2015-11-14 12:46:42 +0000 | [diff] [blame] | 1145 | TestTLS_FTPClassMixin, TestTLS_FTPClass, |
| 1146 | MiscTestCase] |
Antoine Pitrou | f988cd0 | 2009-11-17 20:21:14 +0000 | [diff] [blame] | 1147 | |
Hai Shi | e80697d | 2020-05-28 06:10:27 +0800 | [diff] [blame] | 1148 | thread_info = threading_helper.threading_setup() |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1149 | try: |
| 1150 | support.run_unittest(*tests) |
| 1151 | finally: |
Hai Shi | e80697d | 2020-05-28 06:10:27 +0800 | [diff] [blame] | 1152 | threading_helper.threading_cleanup(*thread_info) |
Benjamin Peterson | be17a11 | 2008-09-27 21:49:47 +0000 | [diff] [blame] | 1153 | |
Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1154 | |
| 1155 | if __name__ == '__main__': |
| 1156 | test_main() |