Search code examples
pythontcpservertelnet

Python TCP Server


I wrote a simple Python 3 TCP multithreaded server and telnet to it. All seems to work until I go to close the telnet connection. When I close the telnet connection ^] and then control+d to exit telnet the server repeatedly prints ''.

Is there something trivial I am missing here to prevent this behavior?

import socket 
from threading import Thread 
from socketserver import ThreadingMixIn 
 
# Multithreaded Python server : TCP Server Socket Thread Pool
class ClientThread(Thread): 
 
    def __init__(self,ip,port): 
        Thread.__init__(self) 
        self.ip = ip 
        self.port = port 
        print ("[+] New server socket thread started for " + ip + ":" + str(port)) 
 
    def run(self): 
        while True : 
            data = conn.recv(2048) 
            #data = "Echo data: " + data.decode()
            print ("Server received data: ", data)
            conn.send(data)  # echo 
 
# Multithreaded Python server : TCP Server Socket Program Stub
TCP_IP = '0.0.0.0' 
TCP_PORT = 2004 
BUFFER_SIZE = 1024  # Usually 1024, but we need quick response 
 
tcpServer = socket.socket(socket.AF_INET, socket.SOCK_STREAM) 
tcpServer.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1) 
tcpServer.bind((TCP_IP, TCP_PORT)) 
threads = [] 
 
while True: 
    tcpServer.listen(4) 
    print ("Multithreaded Python server : Waiting for connections from TCP clients...") 
    (conn, (ip,port)) = tcpServer.accept() 
    newthread = ClientThread(ip,port) 
    newthread.start() 
    threads.append(newthread) 
 
for t in threads: 
    t.join() 



Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received data:  b''
Server received da

Solution

  • Your client thread seems to loop indefinitely. It will happily continue to (attempt to) read from the closed connection. You should add a check that the data received from the read actually contains bytes. If not, the connection is closed.

    So, after

    data = conn.recv(2048)
    

    add:

    if not data: break