Skip to content

Commit 2e07849

Browse files
committed
refactor(calendar): optimize logging for production readiness
- Change routine operation logs from info to debug level - Simplify success messages for better readability - Remove redundant calendar/path information from log messages - Align logging style with repository standards Following patterns established by repository maintainer in WebDAV client cleanup.
1 parent 7291c93 commit 2e07849

File tree

1 file changed

+9
-17
lines changed

1 file changed

+9
-17
lines changed

nextcloud_mcp_server/client/calendar.py

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ async def list_calendars(self) -> List[Dict[str, Any]]:
120120
}
121121
)
122122

123-
logger.info(f"Found {len(calendars)} calendars for user {self.username}")
123+
logger.debug(f"Found {len(calendars)} calendars")
124124
return calendars
125125

126126
except HTTPStatusError as e:
@@ -210,7 +210,7 @@ async def get_calendar_events(
210210
if len(events) >= limit:
211211
break
212212

213-
logger.info(f"Found {len(events)} events in calendar {calendar_name}")
213+
logger.debug(f"Found {len(events)} events")
214214
return events
215215

216216
except HTTPStatusError as e:
@@ -242,9 +242,7 @@ async def create_event(
242242
)
243243
response.raise_for_status()
244244

245-
logger.info(
246-
f"Successfully created event {event_uid} in calendar {calendar_name}"
247-
)
245+
logger.debug(f"Created event {event_uid}")
248246
return {
249247
"uid": event_uid,
250248
"href": event_path,
@@ -300,9 +298,7 @@ async def update_event(
300298
)
301299
response.raise_for_status()
302300

303-
logger.info(
304-
f"Successfully updated event {event_uid} in calendar {calendar_name}"
305-
)
301+
logger.debug(f"Updated event {event_uid}")
306302
return {
307303
"uid": event_uid,
308304
"href": event_path,
@@ -326,14 +322,12 @@ async def delete_event(self, calendar_name: str, event_uid: str) -> Dict[str, An
326322
response = await self._client.delete(event_path)
327323
response.raise_for_status()
328324

329-
logger.info(
330-
f"Successfully deleted event {event_uid} from calendar {calendar_name}"
331-
)
325+
logger.debug(f"Deleted event {event_uid}")
332326
return {"status_code": response.status_code}
333327

334328
except HTTPStatusError as e:
335329
if e.response.status_code == 404:
336-
logger.info(f"Event {event_uid} not found in calendar {calendar_name}")
330+
logger.debug(f"Event {event_uid} not found")
337331
return {"status_code": 404}
338332
logger.error(f"HTTP error deleting event: {e}")
339333
raise e
@@ -363,9 +357,7 @@ async def get_event(
363357
event_data["href"] = event_path
364358
event_data["etag"] = etag
365359

366-
logger.info(
367-
f"Successfully retrieved event {event_uid} from calendar {calendar_name}"
368-
)
360+
logger.debug(f"Retrieved event {event_uid}")
369361
return event_data, etag
370362

371363
except HTTPStatusError as e:
@@ -948,7 +940,7 @@ async def create_calendar(
948940
)
949941
response.raise_for_status()
950942

951-
logger.info(f"Successfully created calendar: {calendar_name}")
943+
logger.debug(f"Created calendar: {calendar_name}")
952944
return {
953945
"name": calendar_name,
954946
"display_name": display_name or calendar_name,
@@ -969,7 +961,7 @@ async def delete_calendar(self, calendar_name: str) -> Dict[str, Any]:
969961
response = await self._client.delete(calendar_path)
970962
response.raise_for_status()
971963

972-
logger.info(f"Successfully deleted calendar: {calendar_name}")
964+
logger.debug(f"Deleted calendar: {calendar_name}")
973965
return {"status_code": response.status_code}
974966

975967
except Exception as e:

0 commit comments

Comments
 (0)