Python cliente/servidor pregunta

Question

Estoy trabajando en un poco de un proyecto en python. Tengo un cliente y un servidor. El servidor escucha las conexiones y una vez que se recibe una conexión espera la entrada del cliente. La idea es que el cliente pueda conectarse al servidor y ejecutar comandos del sistema como ls y cat. Este es mi código de servidor:

import sys, os, socket 


host = ''     
port = 50105 

s = socket.socket(socket.AF_INET, socket.SOCK_STREAM) 
s.bind((host, port)) 
print("Server started on port: ", port) 

s.listen(5) 
print("Server listening\n") 
conn, addr = s.accept() 
print 'New connection from ', addr 
while (1): 
    rc = conn.recv(5) 
    pipe = os.popen(rc) 
    rl = pipe.readlines() 
    file = conn.makefile('w', 0) 
    file.writelines(rl[:-1]) 
    file.close() 
    conn.close() 

Y este es mi código de cliente:

import sys, socket 
s = socket.socket(socket.AF_INET, socket.SOCK_STREAM) 
host = 'localhost' 
port = input('Port: ') 
s.connect((host, port)) 
cmd = raw_input('$ ') 
s.send(cmd) 
file = s.makefile('r', 0) 
sys.stdout.writelines(file.readlines()) 

Cuando inicio el servidor consigo la salida derecha, diciendo que el servidor está escuchando. Pero cuando me conecto con mi cliente y escriba un comando que el servidor finaliza con este error:

Traceback (most recent call last): 
File "server.py", line 21, in <module> 
    rc = conn.recv(2) 
File "/usr/lib/python2.6/socket.py", line 165, in _dummy 
    raise error(EBADF, 'Bad file descriptor') 
socket.error: [Errno 9] Bad file descriptor 

En el lado del cliente, consigo la salida de ls pero el servidor se fastidien.

Answer 1

el código llama conn.close() y luego vuelve de nuevo en torno a conn.recv(), pero conn ya está cerrado.

Answer 2

Si desea que su cliente para repetir lo que está haciendo, sólo tiene que añadir un bucle de allí;)

s = socket.socket(socket.AF_INET, socket.SOCK_STREAM) 
host = 'localhost' 
port = input('Port: ') 
s.connect((host, port)) 
while True: 
    cmd = raw_input('$ ') 
    s.send(cmd) 
    file = s.makefile('r', 0) 
    sys.stdout.writelines(file.readlines()) 

probablemente debería estar más cerca de lo que desea.

Otros comentarios:

s.listen(1) 

Esta declaración probablemente se debe mover fuera del bucle while. Solo necesita llamar al listen una vez.

pipe = os.popen(rc) 

os.popen ya no se utiliza, utilice el subprocess module lugar.

file = s.makefile('r', 0) 

Está abriendo un archivo, pero nunca se cierra el archivo. Probablemente debería agregar un file.close() después de su llamada sys.stdout.writelines().

EDITAR: responder a las siguientes observaciones; hecho aquí debido a la longitud y formateo

Tal como está, se lee desde el socket una vez, y luego se cierra inmediatamente. Por lo tanto, cuando el cliente va a enviar el siguiente comando, ve que el servidor cerró el socket e indica un error.

La solución es cambiar el código del servidor para que pueda manejar la recepción de varios comandos. Tenga en cuenta que esto se resuelve introduciendo otro ciclo.

Usted necesita para envolver

rc = conn.recv(2) 
pipe = os.popen(rc) 
rl = pipe.readlines() 
fl = conn.makefile('w', 0) 
fl.writelines(rl[:-1]) 

en otro bucle while True: para que se repite hasta que el cliente se desconecta, y luego envolver en un intento de que, salvo el bloque que llama la IOError que se produce por conn.recv () cuando el cliente se desconecta.

el try-excepto bloque debe ser similar

try: 
    # the described above loop goes here 
except IOError: 
    conn.close() 
# execution continues on...