search

marcosrmartin / MTD_Server

GNU General Public License v3.0
0 stars 1 forks source link

Abstraer los servidores #57

Open marcosrmartin opened 1 month ago

marcosrmartin commented 1 month ago

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:

https://github.com/marcosrmartin/MTD_Server/blob/deaef4a106069410a58730d8187b72ad47e19279/mass/port_controller.py#L25-L26 https://github.com/marcosrmartin/MTD_Server/blob/deaef4a106069410a58730d8187b72ad47e19279/mass/port_controller.py#L54-L66 https://github.com/marcosrmartin/MTD_Server/blob/deaef4a106069410a58730d8187b72ad47e19279/mass/port_controller.py#L87-L89

En controlador de contenedores #37:

https://github.com/marcosrmartin/MTD_Server/blob/deaef4a106069410a58730d8187b72ad47e19279/mass/container_controller.py#L17-L19 https://github.com/marcosrmartin/MTD_Server/blob/deaef4a106069410a58730d8187b72ad47e19279/mass/container_controller.py#L44

En el controlador del MTD #35:

https://github.com/marcosrmartin/MTD_Server/blob/deaef4a106069410a58730d8187b72ad47e19279/mass/mtd_controller.py#L23-L39

marcosrmartin commented 1 month ago

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 commented 5 days ago

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.