Skip to content

Mejoras y pequeños cambios en el log de la aplicación#38

Merged
euribates merged 1 commit intopythoncanarias:masterfrom
misanram:Mejoras-en-el-log
Apr 5, 2026
Merged

Mejoras y pequeños cambios en el log de la aplicación#38
euribates merged 1 commit intopythoncanarias:masterfrom
misanram:Mejoras-en-el-log

Conversation

@misanram
Copy link
Copy Markdown
Contributor

@misanram misanram commented Apr 3, 2026

La función trace está definida así:
def trace(self, msg):
self.logger.info('bot asked to execute /status commamd')
if self.verbose:
print(msg)

Evidentemente siempre loguea lo mismo: 'bot asked to execute /status commamd'

El módulo la llama 12 veces:

self.trace('bot asked to execute /status commamd')
self.trace(text)
self.trace('Received command /start')
self.trace('Received command /help')
self.trace('Received command /zen')
self.trace('Received new user event')
self.trace(f'Waiting {config.WELCOME_DELAY} seconds until user completes captcha...')
self.trace(f'Skipping welcome message, user {new_member.name} is no longer in the chat')
self.trace(f'send welcome message for {new_member.name}')
self.trace(f'bot sends reply {reply_spec.reply}')
self.trace('Starting bot...')
self.trace('Bot is ready')

Se cambiar la linea self.logger.info('bot asked to execute /status commamd') por self.logger.info(msg)
Se uniformizan los mensajes del log (uso de mayúsculas, etc).

Se ponen a nivel WARNING todos los logger de las librerías que use la aplicación.
Se pone a nivel config.LOG_LEVEL el log de la aplicación

Se genera un fichero log (bot.log) de la aplicación usando RotatingFileHandler

En modo verbose he sustituido el print por un manejador en consola: console_handler = StreamHandler()
Si el modo verbose no se activa, se usa el manejador console_handler = NullHandler()

Copy link
Copy Markdown
Contributor

@euribates euribates left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfecto 👍🏽

@euribates euribates merged commit e335a75 into pythoncanarias:master Apr 5, 2026
1 check passed
@misanram misanram deleted the Mejoras-en-el-log branch April 5, 2026 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants