Don't log exception trace before retrying HA Payment (#751)

without payer info

---------

Co-authored-by: Timothée Robert <[email protected]>

Author: armanddidierjean

app/core/payment/payment_tool.py

@@ -204,8 +204,11 @@ async def init_checkout(
                     # We know that HelloAsso may refuse some payer infos, like using the firstname "test"
                     # Even when prefilling the payer infos,the user will be able to edit them on the payment page,
                     # so we can safely retry without the payer infos
-                    hyperion_error_logger.exception(
-                        f"Payment: failed to init a checkout with HA for module {module} and name {checkout_name}. Retrying without payer infos",
+                    payer_user_name = ""
+                    if payer_user:
+                        payer_user_name = f"{payer_user.firstname} {payer_user.name}"
+                    hyperion_error_logger.warning(
+                        f"Payment: failed to init a checkout with HA for module {module} and name {checkout_name}. Retrying without payer {payer_user_name} infos",
                     )
 
                     init_checkout_body.payer = None

app/core/utils/log.py

@@ -81,7 +81,12 @@ class console_color:
     def get_config_dict(self, settings: Settings):
         # We can't use a dependency to access settings as this function is not an endpoint. The object must thus be passed as a parameter.
 
-        MINIMUM_LOG_LEVEL: str = "DEBUG" if settings.LOG_DEBUG_MESSAGES else "INFO"
+        # /!\ WARNING /!\
+        # MINIMUM_LOG_LEVEL should never be set higher than INFO
+        # as it would prevent important information to be logged, like MyECLPay operations
+        MINIMUM_LOG_LEVEL: str = (
+            "DEBUG" if settings.LOG_DEBUG_MESSAGES else "INFO"
+        )  # /!\ read warning before modifying this /!\
 
         return {
             "version": 1,
@@ -267,29 +272,29 @@ def get_config_dict(self, settings: Settings):
                         "matrix_errors",
                         "console",
                     ],
-                    "level": "DEBUG",
+                    "level": MINIMUM_LOG_LEVEL,
                     "propagate": False,
                 },
                 "hyperion.myeclpay": {
                     "handlers": [
                         "myeclpay_s3",
                     ],
-                    "level": "DEBUG",
+                    "level": MINIMUM_LOG_LEVEL,
                 },
                 "hyperion.s3.fallback": {
                     "handlers": [
                         "file_s3",
                         "matrix_errors",
                         "console",
                     ],
-                    "level": "DEBUG",
+                    "level": MINIMUM_LOG_LEVEL,
                     "propagate": False,
                 },
                 "hyperion.s3": {
                     "handlers": [
                         "s3",
                     ],
-                    "level": "DEBUG",
+                    "level": MINIMUM_LOG_LEVEL,
                 },
                 "hyperion.amap": {
                     "handlers": [
@@ -327,6 +332,51 @@ def get_config_dict(self, settings: Settings):
                     "level": MINIMUM_LOG_LEVEL,
                     "propagate": False,
                 },
+                "boto3": {
+                    "handlers": [
+                        "console",
+                        "file_errors",
+                        "matrix_errors",
+                    ],
+                    "level": MINIMUM_LOG_LEVEL,
+                    "propagate": False,
+                },
+                "botocore": {
+                    "handlers": [
+                        "console",
+                        "file_errors",
+                        "matrix_errors",
+                    ],
+                    "level": MINIMUM_LOG_LEVEL,
+                    "propagate": False,
+                },
+                "s3transfer": {
+                    "handlers": [
+                        "console",
+                        "file_errors",
+                        "matrix_errors",
+                    ],
+                    "level": MINIMUM_LOG_LEVEL,
+                    "propagate": False,
+                },
+                "urllib3": {
+                    "handlers": [
+                        "console",
+                        "file_errors",
+                        "matrix_errors",
+                    ],
+                    "level": MINIMUM_LOG_LEVEL,
+                    "propagate": False,
+                },
+                "python_multipart.multipart": {
+                    "handlers": [
+                        "console",
+                        "file_errors",
+                        "matrix_errors",
+                    ],
+                    "level": MINIMUM_LOG_LEVEL,
+                    "propagate": False,
+                },
             },
         }