admin/mcp-nextcloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(calendar): optimize logging for production readiness

Browse Source 
- 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.
This commit is contained in:
Neovasky
2025-07-27 00:46:57 -04:00
parent 7291c930c4
commit 2e078498b1
1 changed files with 9 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nextcloud_mcp_server/client/calendar.py
+9 -17
View File
@@ -120,7 +120,7 @@ class CalendarClient(BaseNextcloudClient):
} }
) )
logger.info(f"Found {len(calendars)} calendars for user {self.username}") logger.debug(f"Found {len(calendars)} calendars")
return calendars return calendars
except HTTPStatusError as e: except HTTPStatusError as e:
@@ -210,7 +210,7 @@ class CalendarClient(BaseNextcloudClient):
if len(events) >= limit: if len(events) >= limit:
break break
logger.info(f"Found {len(events)} events in calendar {calendar_name}") logger.debug(f"Found {len(events)} events")
return events return events
except HTTPStatusError as e: except HTTPStatusError as e:
@@ -242,9 +242,7 @@ class CalendarClient(BaseNextcloudClient):
) )
response.raise_for_status() response.raise_for_status()
logger.info( logger.debug(f"Created event {event_uid}")
f"Successfully created event {event_uid} in calendar {calendar_name}"
)
return { return {
"uid": event_uid, "uid": event_uid,
"href": event_path, "href": event_path,
@@ -300,9 +298,7 @@ class CalendarClient(BaseNextcloudClient):
) )
response.raise_for_status() response.raise_for_status()
logger.info( logger.debug(f"Updated event {event_uid}")
f"Successfully updated event {event_uid} in calendar {calendar_name}"
)
return { return {
"uid": event_uid, "uid": event_uid,
"href": event_path, "href": event_path,
@@ -326,14 +322,12 @@ class CalendarClient(BaseNextcloudClient):
response = await self._client.delete(event_path) response = await self._client.delete(event_path)
response.raise_for_status() response.raise_for_status()
logger.info( logger.debug(f"Deleted event {event_uid}")
f"Successfully deleted event {event_uid} from calendar {calendar_name}"
)
return {"status_code": response.status_code} return {"status_code": response.status_code}
except HTTPStatusError as e: except HTTPStatusError as e:
if e.response.status_code == 404: if e.response.status_code == 404:
logger.info(f"Event {event_uid} not found in calendar {calendar_name}") logger.debug(f"Event {event_uid} not found")
return {"status_code": 404} return {"status_code": 404}
logger.error(f"HTTP error deleting event: {e}") logger.error(f"HTTP error deleting event: {e}")
raise e raise e
@@ -363,9 +357,7 @@ class CalendarClient(BaseNextcloudClient):
event_data["href"] = event_path event_data["href"] = event_path
event_data["etag"] = etag event_data["etag"] = etag
logger.info( logger.debug(f"Retrieved event {event_uid}")
f"Successfully retrieved event {event_uid} from calendar {calendar_name}"
)
return event_data, etag return event_data, etag
except HTTPStatusError as e: except HTTPStatusError as e:
@@ -948,7 +940,7 @@ class CalendarClient(BaseNextcloudClient):
) )
response.raise_for_status() response.raise_for_status()
logger.info(f"Successfully created calendar: {calendar_name}") logger.debug(f"Created calendar: {calendar_name}")
return { return {
"name": calendar_name, "name": calendar_name,
"display_name": display_name or calendar_name, "display_name": display_name or calendar_name,
@@ -969,7 +961,7 @@ class CalendarClient(BaseNextcloudClient):
response = await self._client.delete(calendar_path) response = await self._client.delete(calendar_path)
response.raise_for_status() response.raise_for_status()
logger.info(f"Successfully deleted calendar: {calendar_name}") logger.debug(f"Deleted calendar: {calendar_name}")
return {"status_code": response.status_code} return {"status_code": response.status_code}
except Exception as e: except Exception as e: