GNU/Weeb Mailing List <[email protected]>
 help / color / mirror / Atom feed
From: Muhammad Rizki <[email protected]>
To: Ammar Faizi <[email protected]>
Cc: Muhammad Rizki <[email protected]>,
	Alviro Iskandar Setiawan <[email protected]>,
	GNU/Weeb Mailing List <[email protected]>
Subject: [PATCH v2 00/17] Everything about logger changes and some fixes
Date: Thu, 19 Jan 2023 00:53:39 +0700	[thread overview]
Message-ID: <[email protected]> (raw)

Hi sir,
This series is about refactor logger method, fix charset issue, and some
improvements. In this series we use a simple logging method that so
easily to manage logger config using .conf file.

- What's new?
  v1:
    1) Add return type annotation for the send_te.xt_mail() and
       send_patch_email() for the Discord bot

- Fixes:
  v1:
    1) Unknown email payload charset `unknown-8bit` that is not in the
       Python's built-in `decode()`. So, we manually using `8bit`
       charset and force it using `surrogateescape`.

    2) Fix type annotations for both Discord and Telegram decorators that
       handle flood request.

    3) Implement DaemonException() and report_err() in the Telegram's
       scrape.py and Discord's get_lore_mail.py

  v2:
    1) Add `typing-extensions` into the `requirements.txt` to fix no
       no module error.

- Improvements:
  v1:
    1) Simplify code to get the DB_PORT.
    2) Remove some unnecessary comments.

  v2:
    1) Remove custom logger since it's not use anymore.

There are 17 patches in this series:
- Patch 1 is to simplify to get the DB_PORT from .env for the TG bot.
- Patch 2 is to simplify to get the DB_PORT from .env for the DC bot.
- Patch 3 is to add a telegram.logger.conf.
- Patch 4 is to add a discord.logger.conf.
- Patch 5 is to initialize the logger config for the Telegram bot.
- Patch 6 is to initialize the logger config for the Discord bot.
- Patch 7 is to fix the type annotations for the Telegram's decorator.
- Patch 8 is to remove some unnecessary comments.
- Patch 9 is to fix the type annotations for the Discord's decorator.
- Patch 10 is to add return type annotations for the Discord bot.
- Patch 11 is to implement DaemonException and report_err for TG scrape.
- Patch 12 is to fix `unknown-8bit` charset when decoding email payload.
- Patch 13 is to refactor logging method in all files for Telegram bot.
- Patch 14 is to implement DaemonException and report_err for DC bot.
- Patch 15 is to refactor logging method in all files for Discord bot.
- Patch 16 is to move requirements.txt to the daemon and update deps.
- Patch 17 is to remove custom logger since it's not used anymore.

Please give it a test, thanks!

Signed-off-by: Muhammad Rizki <[email protected]>
---

Muhammad Rizki (17):
  telegram: Simplify code to get DB_PORT from env
  discord: Simplify code to get DB_PORT from env
  telegram: logger: Add a telegram.logger.conf
  discord: logger: Add a discord.logger.conf
  telegram: logger: Initialize the configuration for the Telegram
    logger.
  discord: logger: Initialize the configuration for the Discord logger.
  telegram: fix: Fix the type annoations for the decorator
  discord: cleanup: Remove some unnecessary comments
  discord: fix: Fix the type annotations for the decorator
  discord: typing: Add return type annotations
  telegram: Implement DaemonException() and report_err() in scrape.py
  utils: fix: Fix charset issue for get_decoded_payload()
  telegram: logger: Refactor all logging method
  discord: Implement DaemonException and report_err in get_lore_mail.py
  discord: logger: Refactor all logging method
  refactor: Move requirements.txt
  cleanup: Remove the custom logger module

 daemon/atom/utils.py                          |  3 ++
 daemon/dc.py                                  | 20 +++----
 daemon/dscord/discord.logger.conf             | 36 +++++++++++++
 daemon/dscord/gnuweeb/client.py               | 25 +++++----
 daemon/dscord/gnuweeb/filters.py              | 32 ++++++------
 .../dscord/gnuweeb/plugins/events/on_ready.py |  5 +-
 .../plugins/slash_commands/get_lore_mail.py   |  9 ++--
 daemon/dscord/mailer/listener.py              | 17 +++---
 daemon/logger/__init__.py                     |  1 -
 daemon/logger/log.py                          | 52 -------------------
 daemon/{telegram => }/requirements.txt        |  2 +
 daemon/telegram/mailer/listener.py            | 15 +++---
 daemon/telegram/packages/client.py            | 21 +++++---
 daemon/telegram/packages/decorator.py         | 31 ++++++-----
 .../packages/plugins/commands/scrape.py       |  9 ++--
 daemon/telegram/telegram.logger.conf          | 36 +++++++++++++
 daemon/tg.py                                  | 18 +++----
 17 files changed, 184 insertions(+), 148 deletions(-)
 create mode 100644 daemon/dscord/discord.logger.conf
 delete mode 100644 daemon/logger/__init__.py
 delete mode 100644 daemon/logger/log.py
 rename daemon/{telegram => }/requirements.txt (81%)
 create mode 100644 daemon/telegram/telegram.logger.conf


base-commit: 1ad0c452d2f87c086a76dd9ceee7a109c8874f57
--
Muhammad Rizki

             reply	other threads:[~2023-01-18 17:54 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-18 17:53 Muhammad Rizki [this message]
2023-01-18 17:53 ` [PATCH v2 01/17] telegram: Simplify code to get DB_PORT from env Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 02/17] discord: " Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 03/17] telegram: logger: Add a telegram.logger.conf Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 04/17] discord: logger: Add a discord.logger.conf Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 05/17] telegram: logger: Initialize the configuration for the Telegram logger Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 06/17] discord: logger: Initialize the configuration for the Discord logger Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 07/17] telegram: fix: Fix the type annoations for the decorator Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 08/17] discord: cleanup: Remove some unnecessary comments Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 09/17] discord: fix: Fix the type annotations for the decorator Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 10/17] discord: typing: Add return type annotations Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 11/17] telegram: Implement DaemonException() and report_err() in scrape.py Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 12/17] utils: fix: Fix charset issue for get_decoded_payload() Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 13/17] telegram: logger: Refactor all logging method Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 14/17] discord: Implement DaemonException and report_err in get_lore_mail.py Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 15/17] discord: logger: Refactor all logging method Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 16/17] refactor: Move requirements.txt Muhammad Rizki
2023-01-18 17:53 ` [PATCH v2 17/17] cleanup: Remove the custom logger module Muhammad Rizki
2023-01-19  6:54 ` [PATCH v2 00/17] Everything about logger changes and some fixes Ammar Faizi
2023-01-19 12:49   ` Muhammad Rizki
2023-01-19  6:54 ` Ammar Faizi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox