Edición: Gracias a la respuesta de @abulafia me doy cuenta que aunque mi respuesta es perfectamente válida para comunicaciones UDP, es probable que la conexión sea TCP ya que el OP nunca respondió a mi pregunta ¿Podrías agregar la parte en la que creas y configuras el socket?, de modo que añadiré un apartado adicional para tratar las comunicaciones TCP.
Comunicación UDP
Si el tráfico de datos es UDP entonces el problema que sufres es que en UDP, a diferencia de TCP, si recibimos un paquete y no podemos obtener el contenido en su totalidad el resto se pierde, no se queda en ningún tipo de buffer esperando ser leído, pero si hay más paquetes UDP pendientes de ser leídos no se pierden y podrán ser leídos uno tras otro de manera compacta (sin mezclarse un mensaje con otro).
Una ventaja que tienes es que se conoce y es acotado el tamaño máximo que puede llegar a tener un paquete IP (216 - 1 = 65.535 bytes) a los que se debería restar el tamaño de la propia cabecera IP (20 bytes mínimo) y la UDP (8 bytes).
En definitiva, deberás definir un tamaño máximo de información de 65.507 bytes para asegurarte que no perderás contenido de ningún paquete UDP por haberlo leído incompleto, ahorrándote el bucle y la concatenación de resultados parciales (nunca los habrá):
def recibirTodo(sock):
return sock.recv(65507)
Comunicación TCP
En el caso de TCP se podría usar la misma solución que UDP ya que el paquete máximo que se podrá recibir de una tacada será similar al UDP, aunque el tamaño de una cabecera TCP tiene un tamaño mínimo de 20 bytes en vez de 8 bytes, dando un tamaño máximo de 65.495 bytes por paquete (en realidad se denomina datagrama).
Las principales diferencias con UDP son:
- Si no se leen todos los datos recibidos el resto se quedan en un búfer esperando su obtención.
- Si se recibe más de un paquete la información se concatena en el búfer de recepción, por lo que una llamada a
sock.recv()
podría leer el contenido de dos paquetes a la vez.
- Debido a este último punto, no hay garantía alguna que al usar
sock.recv(65495)
se obtenga el contenido de únicamente un paquete o incluso, dependiendo de la situación, podría llegarse a leer uno de los paquetes recibidos de manera parcial.
Por ello se suele implementar siempre un protocolo de comunicación en el que cada mensaje enviado usa delimitadores para marcar el comienzo y/o final de cada mensaje o informar de su tamaño en un campo para que tras leer ese número de bytes se analice el contenido del mensaje.
En tu código supones que si llenas con el tamaño máximo tu variable de recepción es porque aún quedan datos por leer, pero existe una probabilidad (aunque sea minúscula) de que tu aplicación haya recibido exactamente 4096 bytes y vuelvas a leer el socket quedándote bloqueado en la lectura.
Por cierto, el código correcto (como apuntan en la otra respuesta mencionada) sería:
if len(parte) < buff_size
Pero si tu objetivo es que se quede esperando hasta recibir datos y llenar la variable hasta haber recibido toda la información pendiente, entonces es mejor usar socket.setblocking()
para forzar una excepción cuando no haya más datos que leer y no se quede bloqueado esperando a recibir más:
def recibirTodo(sock):
datos = ""
buff_size = 4096
# En python3 puedes obtener el estado previo y luego restablecerlo
#bloqueo = sock.getblocking()
# La primera vez nos forzamos a esperar la llegada de datos
sock.setblocking(True)
try:
while True:
datos += sock.recv(buff_size)
# A partir de ahora si no hay datos que leer finalizamos el bucle
sock.setblocking(False)
except socket.error:
pass
# En python3 podremos recuperar el estado inicial del bloqueo
#sock.setblocking(bloqueo)
return datos