fix: raise instead of swallowing on outbound-message path errors

Per reviewer request (chengyongru): raise exceptions on the outbound
message path so ChannelManager can trigger retry logic, matching the
pattern from commit 98c2f7cc (Weixin channel cleanup).

Changes:
- _resolve_server_upload_limit_bytes: warning → error (non-fatal config)
- _upload_and_send_attachment media upload: raise instead of swallow
- _upload_and_send_attachment room send: raise instead of swallow
- send_delta stream edit: error + raise after cleanup
- weixin _load_state: warning → error (non-fatal state load)
This commit is contained in:
Vilius Vystartas 2026-05-07 06:08:03 +01:00 committed by Xubin Ren
parent ba1e5036f5
commit 31c45fe798
2 changed files with 6 additions and 7 deletions

View File

@ -412,7 +412,7 @@ class MatrixChannel(BaseChannel):
try:
response = await self.client.content_repository_config()
except Exception:
self.logger.warning("Failed to fetch server upload limit", exc_info=True)
self.logger.error("Failed to fetch server upload limit", exc_info=True)
return None
upload_size = getattr(response, "upload_size", None)
if isinstance(upload_size, int) and upload_size > 0:
@ -458,8 +458,7 @@ class MatrixChannel(BaseChannel):
filesize=size_bytes,
)
except Exception:
self.logger.warning("Media upload failed", exc_info=True)
return fail
raise
upload_response = upload_result[0] if isinstance(upload_result, tuple) else upload_result
encryption_info = upload_result[1] if isinstance(upload_result, tuple) and isinstance(upload_result[1], dict) else None
@ -478,8 +477,7 @@ class MatrixChannel(BaseChannel):
try:
await self._send_room_content(room_id, content)
except Exception:
self.logger.warning("Failed to send room content to room_id=%s", room_id, exc_info=True)
return fail
raise
return None
async def send(self, msg: OutboundMessage) -> None:
@ -556,8 +554,9 @@ class MatrixChannel(BaseChannel):
# we are editing the same message all the time, so only the first time the event id needs to be set
buf.event_id = response.event_id
except Exception:
self.logger.warning("Stream send/edit failed for chat_id=%s", chat_id, exc_info=True)
self.logger.error("Stream send/edit failed for chat_id=%s", chat_id, exc_info=True)
await self._stop_typing_keepalive(chat_id, clear_typing=True)
raise
def _register_event_callbacks(self) -> None:

View File

@ -208,7 +208,7 @@ class WeixinChannel(BaseChannel):
self.config.base_url = base_url
return bool(self._token)
except Exception:
self.logger.warning("Failed to load Weixin account state", exc_info=True)
self.logger.error("Failed to load Weixin account state", exc_info=True)
return False
def _save_state(self) -> None: