Fix

Author: neob91-close

inbox/events/recurring.py

@@ -89,9 +89,7 @@ def parse_rrule(event):  # type: ignore[no-untyped-def]  # noqa: ANN201
 
             return rule
         except Exception as e:
-            log.exception(
-                "Error parsing RRULE entry", event_id=event.id, error=e
-            )
+            log.exception("Error parsing RRULE entry", event_id=event.id)
     return None
 
 

inbox/models/event.py

@@ -510,19 +510,15 @@ def __init__(self, **kwargs) -> None:  # type: ignore[no-untyped-def]
         try:
             self.unwrap_rrule()
         except Exception as e:
-            log.exception(
-                "Error parsing RRULE entry", event_id=self.id, error=e
-            )
+            log.exception("Error parsing RRULE entry", event_id=self.id)
 
     # FIXME @karim: use an overrided property instead of a reconstructor.
     @reconstructor
     def reconstruct(self) -> None:
         try:
             self.unwrap_rrule()
         except Exception as e:
-            log.exception(
-                "Error parsing stored RRULE entry", event_id=self.id, error=e
-            )
+            log.exception("Error parsing stored RRULE entry", event_id=self.id)
 
     def inflate(self, start=None, end=None):  # type: ignore[no-untyped-def]  # noqa: ANN201
         # Convert a RecurringEvent into a series of InflatedEvents

inbox/sendmail/smtp/postel.py

@@ -161,7 +161,7 @@ def _connect(self, host, port):  # type: ignore[no-untyped-def]
             self.connection.connect(host, port)
         except OSError as e:
             # 'Connection refused', SSL errors for non-TLS connections, etc.
-            log.exception("SMTP connection error", server_error=e.strerror)
+            log.exception("SMTP connection error")
             msg = _transform_ssl_error(e.strerror)
             raise SendMailException(msg, 503)  # noqa: B904
 
@@ -362,7 +362,7 @@ def _send(self, recipients, msg):  # type: ignore[no-untyped-def]
                         )
             except smtplib.SMTPException as err:
                 last_error = err
-                self.log.exception("Error sending", error=err)
+                self.log.exception("Error sending")
 
         assert last_error is not None
         self.log.error(