pryancito / zeusircd Goto Github PK

Hola! En el momento de la compilación me entrega un error algo curioso (Tengo todas las dependencias instaladas que pide).

[ 4%] Building amqp-client.o from amqp-client.cpp, please wait.
In file included from amqp-client.cpp:38:
../include/amqp.h:44:10: fatal error: proton/container.hpp: No such file or directory
44 | #include <proton/container.hpp>
| ^~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
gmake[1]: *** [Makefile:48: amqp-client.o] Error 1
gmake[1]: Leaving directory '/home/ircd/zeusircd/src'

Veo un problema imporante: no hay documentación interna:

• Ningún archivo (salvo los de terceros) tiene cabecera.
• Ningún archivo (salvo los de terceros) tiene LICENCIA.
• Ningún objeto (clase, método, función, variable...) tiene siquiera una descripción de qué hace.
• Se echa de menos algún esquema que explique cómo se relacionan las clases y los módulos, no sólo a nivel de herencia (aunque no he visto nada de herencia, lo que no es una crítica, ojo, si no es necesario pues no hay por qué heredar) sino también quién usa qué o compone cualo...
• En los archivos fuente (cpp) donde se implementa más de una clase, no hay una separación clara y visible entre dichas clases. Por ejemplo, en server.cpp ambas clases tienen nombres muy similares (Servidor y Servidores) así que es fácil confundirlas (ya sé, el "Buscar..." del IDE, pero aun así, si buscas "Servidor" en la zona de "Servidores" te puedes aburrir antes de encontrar la clase...). Debería haber algo que separe visualmente la implementación de cada una. Por ejemplo, yo he usado un comentario con el nombre de la cabecera y una línea de asteriscos para separar los archivos .h a los que hace referencia cada sección de código en el siguiente archivo: https://github.com/niuniomartinez/allegro-pas/blob/95d326c5632db02b4880a86b6fd18d3f529014c2/TRUNK/lib/allegro5.pas#L97).
• Otra cosa, hay cierta inconsistencia en los nombres. Por ejemplo: Server::handleAccept es semiCamelCase pero Server::handle_hanshake es under_score. Eso sí, bien que la mayoría de métodos/funciones sean verbos y las clases y propiedades sean nombres (aunque para estos últimos, en C++, prefiero que sean métodos "get..." y "set...", pero es una preferencia personal). Sí he visto alguna variable con verbo (is_flood, por ejemplo), y no debería ser (en este caso concreto, mejor flood_state ó flood_flag), claro que entonces el código, en vez de "if (is_flood) ..." debería ser "if (flood_flag == TRUE) ..." para que realmente estuviera bien escrito ("if flood flag" no tiene mucho sentido en inglés, pero "if flood flag is true" sí). Esto último es discutible, o sea, que puede que ser más cosa de estilo (el verbo "to be" es complejo, más que el ser/estar castellano, y casi tanto como "to have" y "to go" que también tienen lo suyo).

Lo de "La mejor documentación es un código bien escrito" deriva de un Argumento ad verecundiam; no he visto a ningún picacódigo que la defienda, sólo a ingenieros, y mi experiencia dicta que no suele ser cierto: dentro de nada volveréis al código y os pasaréis mucho más tiempo leyéndolo intentando saber qué narices hace o cómo funciona que modificándolo, da igual lo "bien" escrito que esté.

Deberíais, cómo mínimo, añadir licencias y una cabecera por cada archivo (no sólo .h, también .cpp) y cada clase y función (esto último vale con que se ponga en los .h) explicando aunque sea someramente qué es y cómo funciona; o sea, no poner únicamente que la clase Servidor representa a un servidor: también hay que dar alguna pista de cómo utilizarla y/o dónde se utiliza dentro del programa.

Ya sé, es mucha crítica, pero es lo que he visto. Para terminar, una cosa buena, muy buena: No he visto ni un constructor de conversión ni habéis sobrecargado ningún operador. Que siga así. En serio, los constructores de conversión y la sobrecarga de operadores es una de las cosas que más confusión y bugs difíciles de rastrear producen, y una de las principales razones por las que abandoné C++.

Falla /os kill /os gline

En los dos casos el comando no llega a ser ejecutado, el comando fue probado de la siguiente manera:

/os kill nick
Y el resultado en status fue: El nick xxx no esta conectado.

Al momento de ejecutar /os gline add IP Estoy probando! el proceso Zeus -Start cae graba el GLINE al momento de reiniciar el proceso Zeus el gline esta intacto.

Primero varias dependencias del install guide no han funcionado, segundo al momento de compilar tira error:

1. Error en guia de instalación:

[centos@vps-b678d646 ~]$ sudo yum -y install centos-release-scl
Extra Packages for Enterprise Linux Modular 8 - x86_64 152 kB/s | 117 kB 00:00
Extra Packages for Enterprise Linux 8 - x86_64 5.1 MB/s | 8.0 MB 00:01
No match for argument: centos-release-scl
Error: Unable to find a match: centos-release-scl
[centos@vps-b678d646 ~]$ sudo yum -y install devtoolset-7
Last metadata expiration check: 0:00:12 ago on Thu 17 Sep 2020 05:41:19 AM UTC.
No match for argument: devtoolset-7
Error: Unable to find a match: devtoolset-7
[centos@vps-b678d646 ~]$ scl enable devtoolset-7 bash
-bash: scl: command not found

No match for argument: icu-devtools
Package gettext-0.19.8.1-17.el8.x86_64 is already installed.
No match for argument: MariaDB-client
No match for argument: MariaDB-shared
Error: Unable to find a match: icu-devtools MariaDB-client MariaDB-shared

2. No instala:

[centos@vps-b678d646 zeusircd]$ make

************************ ,--.,---.,----.
**** ,---. {code} | | ¬_| ,--.
**** / .---., .,---.| | __, |
**** / |---' | ---.| | |\ '--'
**** '---'^---'^---''---'^--^--^ ^--^--'

*** { Innovating, Making The World } ***

make[1]: Entering directory '/home/centos/zeusircd/src'
Building ZeusIRCD core, this operation may take several minutes...
[ 4%] Building amqp-client.o from amqp-client.cpp, please wait.
In file included from ../boost/include/boost/asio/ssl/context_base.hpp:19,
from ../boost/include/boost/asio/ssl/context.hpp:23,
from ../boost/include/boost/asio/ssl.hpp:18,
from ../include/Server.h:27,
from ../include/amqp.h:41,
from amqp-client.cpp:38:
../boost/include/boost/asio/ssl/detail/openssl_types.hpp:23:10: fatal error: openssl/conf.h: No such file or directory
#include <openssl/conf.h>
^~~~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [Makefile:48: amqp-client.o] Error 1
make[1]: Leaving directory '/home/centos/zeusircd/src'

Se probo expulsar a un determinado usuario del canal y el op pierde sincro con el canal.
Se probó banear y expulsar a un determinado usuario del canal y sucede lo mismo.
CHaN elimina correctamente el ban al minuto (esto segun como este zeus.conf)

Revisar porque se pierde "sincro" cuando un op/ircop efectua kick, ban, kickban.

Testeado con misma ip del ircop/op, y con otra ip desde webchat.

El usuario que expulsa desaparece para el resto
El usuario que fue kickeado regresa y no ve al usuario que lo expulso
El usuario que expulso debe entrar y salir para ser visible en el canal y que lo lean/que pueda leer.

Se probo ZeusIRCD desde webchat KiwIRC y se determino que entre los 3 y 5 minutos de un usuario sin actividad el webchat pierde conexion con el server, determinar si esto es bug o si esta configurado para evitar "idleadores"