Abstraer los servidores

[marcosrmartin]

Hay código que puede introducir a error introduciendo posibles casuísticas manejando servidores y dificultando la lógica. Es decir hay varias funcionalidades que se podrían mejorar subiendo el nivel de abstracción, sobre todo en métodos que "rotan". Por ejemplo:

En controlador de puertos #36:

MTD_Server/mass/port_controller.py

Lines 25 to 26 in deaef4a

	rest = [NGINX, HTTPD]
	rest.remove(running)

MTD_Server/mass/port_controller.py

Lines 54 to 66 in deaef4a

	def swap_to_container(self, container):
	if container == NGINX:
	self.__remove_port(HTTPD, UDP)
	self.__remove_port(HTTPD, TCP)
	self.__add_port(NGINX, UDP)
	self.__add_port(NGINX, TCP)
	logger.info(f"Forwardind port changed from {PORT[HTTPD]} -> {PORT[NGINX]}")
	else:
	self.__remove_port(NGINX, UDP)
	self.__remove_port(NGINX, TCP)
	self.__add_port(HTTPD, UDP)
	self.__add_port(HTTPD, TCP)
	logger.info(f"Forwardind port changed from {PORT[NGINX]} -> {PORT[HTTPD]}")

MTD_Server/mass/port_controller.py

Lines 87 to 89 in deaef4a

	ports = {NGINX:{}, HTTPD:{}}
	f_ports = self.firebus.getForwardPorts(self.zone)
	for container in (NGINX, HTTPD):

En controlador de contenedores #37:

MTD_Server/mass/container_controller.py

Lines 17 to 19 in deaef4a

	self.port_bindings_httpd = {PORT[PROXY]:PORT[HTTPD]}
	self.port_bindings_nginx = {PORT[PROXY]:PORT[NGINX]}

MTD_Server/mass/container_controller.py

Line 44 in deaef4a

ports = self.port_bindings_nginx if container_name == NGINX.lower() else self.port_bindings_httpd

En el controlador del MTD #35:

MTD_Server/mass/mtd_controller.py

Lines 23 to 39 in deaef4a

	def switch(self):
	old_container = self.container_controller.active_container
	if self.container_controller.active_container.name == NGINX.lower():
	logger.info("Rotating from NGINX -> HTTPD")
	self.container_controller.create_container(HTTPD.lower())
	self.port_controller.swap_to_container(HTTPD)
	self.container_controller.remove_container(old_container)
	container = HTTPD
	elif self.container_controller.active_container.name == HTTPD.lower():
	logger.info("Rotating from HTTPD -> NGINX")
	self.container_controller.create_container(NGINX.lower())
	self.port_controller.swap_to_container(NGINX)
	self.container_controller.remove_container(old_container)
	container = NGINX
	else:
	logger.error(f"MTD rotation failed cause of: {str(e)}")
	raise ValueError(f"MTD rotation failed cause of: {str(e)}")

[marcosrmartin]

Tendría sentido arreglar esto @JJ? (refiriéndome al tiempo). He tirado del hilo al implementar los test y pensar si hacer para swap_to_container, que comparte problema con el resto.

[JJ]

Bueno, habría que formular un poco mejor el problema. Ahora mismo tiene como título la solución, "abstraer los servidores".
Yo creo que la idea principal que había detrás es que se puedan usar diferentes servidores como parte del perfil.