From cb258d72c31ddceb2022a015ae31b5970a8a2add Mon Sep 17 00:00:00 2001 From: Tola Leng Date: Mon, 14 Jul 2025 15:45:05 +0700 Subject: [PATCH] Disable debug console logs for production --- .../incident/form/IncidentBasicFields.tsx | 6 +-- .../servers/ServerHistoryCharts.tsx | 6 +-- .../servers/ServerSystemInfoCard.tsx | 2 +- .../components/servers/charts/dataUtils.ts | 10 ++--- .../charts/hooks/useServerHistoryData.ts | 4 +- .../services/hooks/useDefaultUptimeData.ts | 26 ++++++------ .../alerts-templates/hooks/useTemplateForm.ts | 16 ++++---- .../user-management/hooks/useUsersList.ts | 8 ++-- application/src/pages/ServerDetail.tsx | 4 +- application/src/services/serverService.ts | 40 +++++++++---------- .../ssl/notification/sslCheckNotifier.ts | 26 ++++++------ .../src/services/types/serverService.ts | 28 ++++++------- 12 files changed, 88 insertions(+), 88 deletions(-) diff --git a/application/src/components/schedule-incident/incident/form/IncidentBasicFields.tsx b/application/src/components/schedule-incident/incident/form/IncidentBasicFields.tsx index 33b35d5..7cb648b 100644 --- a/application/src/components/schedule-incident/incident/form/IncidentBasicFields.tsx +++ b/application/src/components/schedule-incident/incident/form/IncidentBasicFields.tsx @@ -40,10 +40,10 @@ export const IncidentBasicFields: React.FC = () => { queryFn: async () => { try { const usersList = await userService.getUsers(); - console.log("Fetched users for incident assignment:", usersList); + // console.log("Fetched users for incident assignment:", usersList); return Array.isArray(usersList) ? usersList : []; } catch (error) { - console.error("Failed to fetch users:", error); + // console.error("Failed to fetch users:", error); return []; } }, @@ -53,7 +53,7 @@ export const IncidentBasicFields: React.FC = () => { // Add user to assigned_to const addUser = (userId: string) => { // For now, we're using a single user assignment - console.log("Setting user ID in form:", userId); + // console.log("Setting user ID in form:", userId); form.setValue('assigned_to', userId, { shouldValidate: true, shouldDirty: true }); setSelectedUserIds([userId]); }; diff --git a/application/src/components/servers/ServerHistoryCharts.tsx b/application/src/components/servers/ServerHistoryCharts.tsx index 22dcfc9..274f966 100644 --- a/application/src/components/servers/ServerHistoryCharts.tsx +++ b/application/src/components/servers/ServerHistoryCharts.tsx @@ -24,7 +24,7 @@ export const ServerHistoryCharts = ({ serverId }: ServerHistoryChartsProps) => { isFetching } = useServerHistoryData(serverId); - console.log('ServerHistoryCharts: Rendering with serverId:', serverId, 'timeRange:', timeRange); + //console.log('ServerHistoryCharts: Rendering with serverId:', serverId, 'timeRange:', timeRange); // Memoize latest data calculation to prevent unnecessary recalculations const latestData = useMemo(() => { @@ -71,7 +71,7 @@ export const ServerHistoryCharts = ({ serverId }: ServerHistoryChartsProps) => { } if (error) { - console.error('ServerHistoryCharts: Error loading data:', error); + // console.error('ServerHistoryCharts: Error loading data:', error); return (
@@ -122,7 +122,7 @@ export const ServerHistoryCharts = ({ serverId }: ServerHistoryChartsProps) => { ); } - console.log('ServerHistoryCharts: Rendering charts with', chartData.length, 'data points for time range:', timeRange); + // console.log('ServerHistoryCharts: Rendering charts with', chartData.length, 'data points for time range:', timeRange); return (
diff --git a/application/src/components/servers/ServerSystemInfoCard.tsx b/application/src/components/servers/ServerSystemInfoCard.tsx index 893b60d..002a849 100644 --- a/application/src/components/servers/ServerSystemInfoCard.tsx +++ b/application/src/components/servers/ServerSystemInfoCard.tsx @@ -15,7 +15,7 @@ export function ServerSystemInfoCard({ server }: ServerSystemInfoCardProps) { ? JSON.parse(server.system_info) : server.system_info; } catch (error) { - console.log('Error parsing system_info:', error); + // console.log('Error parsing system_info:', error); } } diff --git a/application/src/components/servers/charts/dataUtils.ts b/application/src/components/servers/charts/dataUtils.ts index 17c935a..d58e2a0 100644 --- a/application/src/components/servers/charts/dataUtils.ts +++ b/application/src/components/servers/charts/dataUtils.ts @@ -61,14 +61,14 @@ export const filterMetricsByTimeRange = (metrics: any[], timeRange: TimeRange): const bufferMinutes = timeRange === '60m' ? 5 : timeRange === '1d' ? 30 : 60; const cutoffTime = new Date(now.getTime() - (selectedRange.hours * 60 * 60 * 1000) - (bufferMinutes * 60 * 1000)); - console.log('filterMetricsByTimeRange: timeRange:', timeRange, 'cutoffTime:', cutoffTime.toISOString()); +// console.log('filterMetricsByTimeRange: timeRange:', timeRange, 'cutoffTime:', cutoffTime.toISOString()); const filtered = metrics.filter(metric => { const metricTime = new Date(metric.created || metric.timestamp); return metricTime >= cutoffTime && metricTime <= now; }); - console.log('filterMetricsByTimeRange: Filtered', metrics.length, 'to', filtered.length, 'metrics'); +// console.log('filterMetricsByTimeRange: Filtered', metrics.length, 'to', filtered.length, 'metrics'); // Sort by timestamp for proper chart display return filtered.sort((a, b) => @@ -129,12 +129,12 @@ const formatTimestamp = (timestamp: string, timeRange: TimeRange): string => { // Optimized chart data formatting with better performance export const formatChartData = (metrics: any[], timeRange: TimeRange) => { - console.log('formatChartData: Input metrics count:', metrics?.length || 0, 'timeRange:', timeRange); +// console.log('formatChartData: Input metrics count:', metrics?.length || 0, 'timeRange:', timeRange); if (!metrics?.length) return []; const filteredMetrics = filterMetricsByTimeRange(metrics, timeRange); - console.log('formatChartData: After time filtering:', filteredMetrics?.length || 0, 'metrics'); +// console.log('formatChartData: After time filtering:', filteredMetrics?.length || 0, 'metrics'); if (!filteredMetrics.length) return []; @@ -154,7 +154,7 @@ export const formatChartData = (metrics: any[], timeRange: TimeRange) => { ? filteredMetrics.filter((_, index) => index % Math.ceil(filteredMetrics.length / maxDataPoints) === 0) : filteredMetrics; - console.log('formatChartData: After sampling:', sampledMetrics.length, 'metrics for display'); +// console.log('formatChartData: After sampling:', sampledMetrics.length, 'metrics for display'); // Batch process the data transformation for better performance return sampledMetrics.map((metric) => { diff --git a/application/src/components/servers/charts/hooks/useServerHistoryData.ts b/application/src/components/servers/charts/hooks/useServerHistoryData.ts index d28eae7..d08a437 100644 --- a/application/src/components/servers/charts/hooks/useServerHistoryData.ts +++ b/application/src/components/servers/charts/hooks/useServerHistoryData.ts @@ -17,9 +17,9 @@ export const useServerHistoryData = (serverId: string) => { } = useQuery({ queryKey: ['server-metrics-history', serverId, timeRange], queryFn: async () => { - console.log('useServerHistoryData: Fetching metrics for serverId:', serverId, 'timeRange:', timeRange); + // console.log('useServerHistoryData: Fetching metrics for serverId:', serverId, 'timeRange:', timeRange); const result = await serverService.getServerMetrics(serverId, timeRange); - console.log('useServerHistoryData: Raw metrics result for timeRange', timeRange, ':', result?.length || 0, 'records'); + // console.log('useServerHistoryData: Raw metrics result for timeRange', timeRange, ':', result?.length || 0, 'records'); return result || []; }, enabled: !!serverId, diff --git a/application/src/components/services/hooks/useDefaultUptimeData.ts b/application/src/components/services/hooks/useDefaultUptimeData.ts index 498bb76..9cdd546 100644 --- a/application/src/components/services/hooks/useDefaultUptimeData.ts +++ b/application/src/components/services/hooks/useDefaultUptimeData.ts @@ -19,10 +19,10 @@ export const useDefaultUptimeData = ({ serviceId, serviceType, status, interval queryKey: ['strictDefaultUptimeHistory', serviceId, serviceType], queryFn: async () => { if (!serviceId) { - console.log('No serviceId provided, skipping fetch'); + // console.log('No serviceId provided, skipping fetch'); return []; } - console.log(`Fetching STRICT Default (Agent ID 1) uptime data for service ${serviceId} of type ${serviceType}`); + // console.log(`Fetching STRICT Default (Agent ID 1) uptime data for service ${serviceId} of type ${serviceType}`); // Get raw uptime history data const rawData = await uptimeService.getUptimeHistory(serviceId, 50, undefined, undefined, serviceType); @@ -38,15 +38,15 @@ export const useDefaultUptimeData = ({ serviceId, serviceType, status, interval ); if (isStrictDefault) { - console.log(`✓ ACCEPTED Default record: ${record.id} - ${record.timestamp} - Pure default monitoring`); + // console.log(`✓ ACCEPTED Default record: ${record.id} - ${record.timestamp} - Pure default monitoring`); } else { - console.log(`✗ REJECTED non-default record: ${record.id} - region: ${record.region_name || 'none'}, agent: ${record.agent_id || 'none'}`); + // console.log(`✗ REJECTED non-default record: ${record.id} - region: ${record.region_name || 'none'}, agent: ${record.agent_id || 'none'}`); } return isStrictDefault; }); - console.log(`ULTRA-STRICT FILTER: Reduced ${rawData.length} records to ${strictDefaultData.length} pure default monitoring records`); + //console.log(`ULTRA-STRICT FILTER: Reduced ${rawData.length} records to ${strictDefaultData.length} pure default monitoring records`); return strictDefaultData; }, enabled: !!serviceId, @@ -61,7 +61,7 @@ export const useDefaultUptimeData = ({ serviceId, serviceType, status, interval const processUptimeData = (data: UptimeData[], intervalSeconds: number): UptimeData[] => { if (!data || data.length === 0) return []; - console.log(`Processing ${data.length} strict default monitoring records for service ${serviceId}`); + // console.log(`Processing ${data.length} strict default monitoring records for service ${serviceId}`); // Sort data by timestamp (newest first) const sortedData = [...data].sort((a, b) => @@ -76,9 +76,9 @@ export const useDefaultUptimeData = ({ serviceId, serviceType, status, interval if (!timestampMap.has(exactTimestamp)) { timestampMap.set(exactTimestamp, record); - console.log(`✓ Added unique default record: ${record.id} - ${exactTimestamp}`); + // console.log(`✓ Added unique default record: ${record.id} - ${exactTimestamp}`); } else { - console.log(`✗ REJECTED absolute duplicate timestamp: ${record.id} - ${exactTimestamp} (exact timestamp match)`); + // console.log(`✗ REJECTED absolute duplicate timestamp: ${record.id} - ${exactTimestamp} (exact timestamp match)`); } }); @@ -86,7 +86,7 @@ export const useDefaultUptimeData = ({ serviceId, serviceType, status, interval const uniqueRecords = Array.from(timestampMap.values()); const recentData = uniqueRecords.slice(0, 20); - console.log(`FINAL RESULT: ${recentData.length} absolutely unique default monitoring records`); + //console.log(`FINAL RESULT: ${recentData.length} absolutely unique default monitoring records`); return recentData; }; @@ -94,12 +94,12 @@ export const useDefaultUptimeData = ({ serviceId, serviceType, status, interval // Update history items when data changes useEffect(() => { if (uptimeData && uptimeData.length > 0) { - console.log(`Received ${uptimeData.length} strict default monitoring records for service ${serviceId}`); + // console.log(`Received ${uptimeData.length} strict default monitoring records for service ${serviceId}`); const processedData = processUptimeData(uptimeData, interval); setHistoryItems(processedData); } else if (!serviceId || (uptimeData && uptimeData.length === 0)) { // Generate placeholder data when no real data is available - console.log(`No strict default monitoring data available for service ${serviceId}, generating placeholder`); + // console.log(`No strict default monitoring data available for service ${serviceId}, generating placeholder`); const statusValue = (status === "up" || status === "down" || status === "warning" || status === "paused") ? status @@ -157,7 +157,7 @@ export const useDefaultUptimeData = ({ serviceId, serviceType, status, interval seenTimestamps.add(item.timestamp); absolutelyUniqueItems.push(item); } else { - console.log(`FINAL VALIDATION: Removing absolute duplicate timestamp ${item.timestamp} from display`); + // console.log(`FINAL VALIDATION: Removing absolute duplicate timestamp ${item.timestamp} from display`); } }); @@ -166,7 +166,7 @@ export const useDefaultUptimeData = ({ serviceId, serviceType, status, interval new Date(b.timestamp).getTime() - new Date(a.timestamp).getTime() ); - console.log(`DISPLAY READY: ${sortedValidated.length} absolutely validated default monitoring items`); + // console.log(`DISPLAY READY: ${sortedValidated.length} absolutely validated default monitoring items`); return sortedValidated.slice(0, 20); }; diff --git a/application/src/components/settings/alerts-templates/hooks/useTemplateForm.ts b/application/src/components/settings/alerts-templates/hooks/useTemplateForm.ts index 26684f8..02e953b 100644 --- a/application/src/components/settings/alerts-templates/hooks/useTemplateForm.ts +++ b/application/src/components/settings/alerts-templates/hooks/useTemplateForm.ts @@ -52,7 +52,7 @@ export const useTemplateForm = ({ templateId, open, onOpenChange, onSuccess }: U const queryClient = useQueryClient(); const isEditMode = !!templateId; - console.log("Template form initialized with templateId:", templateId, "isEditMode:", isEditMode); + // console.log("Template form initialized with templateId:", templateId, "isEditMode:", isEditMode); const form = useForm({ resolver: zodResolver(templateFormSchema), @@ -74,7 +74,7 @@ export const useTemplateForm = ({ templateId, open, onOpenChange, onSuccess }: U // Set form values when template data is loaded useEffect(() => { if (templateData && open) { - console.log("Setting form values with template data:", templateData); + // console.log("Setting form values with template data:", templateData); form.reset({ name: templateData.name || "", @@ -120,7 +120,7 @@ export const useTemplateForm = ({ templateId, open, onOpenChange, onSuccess }: U onSuccess(); }, onError: (error) => { - console.error("Error creating template:", error); + // console.error("Error creating template:", error); toast({ title: "Error", description: "Failed to create template. Please check your inputs and try again.", @@ -142,7 +142,7 @@ export const useTemplateForm = ({ templateId, open, onOpenChange, onSuccess }: U onSuccess(); }, onError: (error) => { - console.error("Error updating template:", error); + // console.error("Error updating template:", error); toast({ title: "Error", description: "Failed to update template. Please check your inputs and try again.", @@ -155,7 +155,7 @@ export const useTemplateForm = ({ templateId, open, onOpenChange, onSuccess }: U // Handle form submission const onSubmit = (formData: TemplateFormData) => { - console.log("Submitting form data:", formData); + // console.log("Submitting form data:", formData); // Ensure all required fields are present const completeData: CreateUpdateTemplateData = { @@ -173,10 +173,10 @@ export const useTemplateForm = ({ templateId, open, onOpenChange, onSuccess }: U }; if (isEditMode && templateId) { - console.log("Updating template with ID:", templateId); + // console.log("Updating template with ID:", templateId); updateMutation.mutate({ id: templateId, data: completeData }); } else { - console.log("Creating new template"); + // console.log("Creating new template"); createMutation.mutate(completeData); } }; @@ -184,7 +184,7 @@ export const useTemplateForm = ({ templateId, open, onOpenChange, onSuccess }: U // Reset form when dialog closes useEffect(() => { if (!open) { - console.log("Dialog closed, resetting form"); + // console.log("Dialog closed, resetting form"); form.reset(defaultFormValues); } }, [open, form]); diff --git a/application/src/components/settings/user-management/hooks/useUsersList.ts b/application/src/components/settings/user-management/hooks/useUsersList.ts index cacb611..288d903 100644 --- a/application/src/components/settings/user-management/hooks/useUsersList.ts +++ b/application/src/components/settings/user-management/hooks/useUsersList.ts @@ -13,16 +13,16 @@ export const useUsersList = () => { setLoading(true); setError(null); try { - console.log("Fetching users list"); + // console.log("Fetching users list"); const data = await userService.getUsers(); - console.log("Received users data:", data); + // console.log("Received users data:", data); if (Array.isArray(data) && data.length >= 0) { setUsers(data); // Clear any previous errors setError(null); } else { - console.error("Invalid users data format:", data); + // console.error("Invalid users data format:", data); setUsers([]); setError("Invalid data format received from server"); toast({ @@ -32,7 +32,7 @@ export const useUsersList = () => { }); } } catch (error) { - console.error("Error fetching users:", error); + // console.error("Error fetching users:", error); setError(error instanceof Error ? error.message : "Unknown error"); toast({ title: "Error fetching users", diff --git a/application/src/pages/ServerDetail.tsx b/application/src/pages/ServerDetail.tsx index 6522f3a..3dee273 100644 --- a/application/src/pages/ServerDetail.tsx +++ b/application/src/pages/ServerDetail.tsx @@ -24,7 +24,7 @@ const ServerDetail = () => { const { sidebarCollapsed, toggleSidebar } = useSidebar(); const [currentUser, setCurrentUser] = useState(authService.getCurrentUser()); - console.log('ServerDetail component loaded with serverId:', serverId); + //console.log('ServerDetail component loaded with serverId:', serverId); const { data: server, @@ -46,7 +46,7 @@ const ServerDetail = () => { }; if (serverError) { - console.error('Server detail error:', serverError); + // console.error('Server detail error:', serverError); return (
diff --git a/application/src/services/serverService.ts b/application/src/services/serverService.ts index e259df5..c808783 100644 --- a/application/src/services/serverService.ts +++ b/application/src/services/serverService.ts @@ -7,7 +7,7 @@ export const serverService = { const records = await pb.collection('servers').getFullList(); return records; } catch (error) { - console.error('Error fetching servers:', error); + // console.error('Error fetching servers:', error); throw error; } }, @@ -17,22 +17,22 @@ export const serverService = { const record = await pb.collection('servers').getOne(serverId); return record; } catch (error) { - console.error('Error fetching server:', error); + // console.error('Error fetching server:', error); throw error; } }, async getServerMetrics(serverId: string, timeRange?: string): Promise { try { - console.log('serverService.getServerMetrics: Fetching metrics for serverId:', serverId, 'timeRange:', timeRange); + // console.log('serverService.getServerMetrics: Fetching metrics for serverId:', serverId, 'timeRange:', timeRange); // First, get the server to find the correct server_id for metrics let server; try { server = await this.getServer(serverId); - console.log('serverService.getServerMetrics: Found server:', server); + // console.log('serverService.getServerMetrics: Found server:', server); } catch (error) { - console.log('serverService.getServerMetrics: Could not fetch server details:', error); + // console.log('serverService.getServerMetrics: Could not fetch server details:', error); } // Try multiple filter strategies to find data @@ -43,17 +43,17 @@ export const serverService = { if (server && server.server_id) { metricsServerId = server.server_id; filter = `server_id = "${metricsServerId}"`; - console.log('serverService.getServerMetrics: Strategy 1 - Using server.server_id for metrics:', metricsServerId); + // console.log('serverService.getServerMetrics: Strategy 1 - Using server.server_id for metrics:', metricsServerId); } else { // Strategy 2: Use the serverId directly filter = `server_id = "${serverId}"`; - console.log('serverService.getServerMetrics: Strategy 2 - Using serverId directly for metrics:', serverId); + // console.log('serverService.getServerMetrics: Strategy 2 - Using serverId directly for metrics:', serverId); } // Add agent_id filter if available in server data if (server && server.agent_id) { filter += ` && agent_id = "${server.agent_id}"`; - console.log('serverService.getServerMetrics: Added agent_id filter:', server.agent_id); + // console.log('serverService.getServerMetrics: Added agent_id filter:', server.agent_id); } // Add time range filter @@ -83,10 +83,10 @@ export const serverService = { const cutoffISO = cutoffTime.toISOString(); filter += ` && created >= "${cutoffISO}"`; - console.log('serverService.getServerMetrics: Using time filter from:', cutoffISO, 'to now'); + // console.log('serverService.getServerMetrics: Using time filter from:', cutoffISO, 'to now'); } - console.log('serverService.getServerMetrics: Final filter:', filter); + // console.log('serverService.getServerMetrics: Final filter:', filter); // Fetch filtered records with proper sorting let records = await pb.collection('server_metrics').getFullList({ @@ -95,11 +95,11 @@ export const serverService = { requestKey: null }); - console.log('serverService.getServerMetrics: Found', records.length, 'records with primary filter'); + // console.log('serverService.getServerMetrics: Found', records.length, 'records with primary filter'); // If no records found with primary strategy, try fallback strategies if (records.length === 0) { - console.log('serverService.getServerMetrics: No records found, trying fallback strategies...'); + // console.log('serverService.getServerMetrics: No records found, trying fallback strategies...'); // Fallback 1: Try without agent_id filter let fallbackFilter = `server_id = "${metricsServerId}"`; @@ -131,19 +131,19 @@ export const serverService = { fallbackFilter += ` && created >= "${cutoffISO}"`; } - console.log('serverService.getServerMetrics: Trying fallback filter without agent_id:', fallbackFilter); + // console.log('serverService.getServerMetrics: Trying fallback filter without agent_id:', fallbackFilter); records = await pb.collection('server_metrics').getFullList({ filter: fallbackFilter, sort: '-created', requestKey: null }); - console.log('serverService.getServerMetrics: Fallback found', records.length, 'records'); + // console.log('serverService.getServerMetrics: Fallback found', records.length, 'records'); // Fallback 2: Try with different server_id strategies if (records.length === 0) { const alternativeIds = [serverId, server?.server_id, server?.id].filter(Boolean); - console.log('serverService.getServerMetrics: Trying alternative server IDs:', alternativeIds); + // console.log('serverService.getServerMetrics: Trying alternative server IDs:', alternativeIds); for (const altId of alternativeIds) { if (altId && altId !== metricsServerId) { @@ -176,7 +176,7 @@ export const serverService = { altFilter += ` && created >= "${cutoffISO}"`; } - console.log('serverService.getServerMetrics: Trying alternative ID filter:', altFilter); + // console.log('serverService.getServerMetrics: Trying alternative ID filter:', altFilter); const altRecords = await pb.collection('server_metrics').getFullList({ filter: altFilter, sort: '-created', @@ -184,7 +184,7 @@ export const serverService = { }); if (altRecords.length > 0) { - console.log('serverService.getServerMetrics: Alternative ID found', altRecords.length, 'records'); + // console.log('serverService.getServerMetrics: Alternative ID found', altRecords.length, 'records'); records = altRecords; break; } @@ -193,14 +193,14 @@ export const serverService = { } } - console.log('serverService.getServerMetrics: Final result:', records.length, 'records found'); + // console.log('serverService.getServerMetrics: Final result:', records.length, 'records found'); if (records.length > 0) { - console.log('serverService.getServerMetrics: Sample record:', records[0]); + // console.log('serverService.getServerMetrics: Sample record:', records[0]); } return records; } catch (error) { - console.error('Error fetching server metrics:', error); + // console.error('Error fetching server metrics:', error); throw error; } }, diff --git a/application/src/services/ssl/notification/sslCheckNotifier.ts b/application/src/services/ssl/notification/sslCheckNotifier.ts index 356dadf..5657118 100644 --- a/application/src/services/ssl/notification/sslCheckNotifier.ts +++ b/application/src/services/ssl/notification/sslCheckNotifier.ts @@ -10,7 +10,7 @@ import { toast } from "sonner"; * This should be called once per day */ export async function checkAllCertificatesAndNotify(): Promise { - console.log("Starting daily SSL certificates check..."); + // console.log("Starting daily SSL certificates check..."); try { // Fetch all SSL certificates from database @@ -18,16 +18,16 @@ export async function checkAllCertificatesAndNotify(): Promise { // Properly cast the items as SSLCertificate const certificates = response.items as unknown as SSLCertificate[]; - console.log(`Found ${certificates.length} certificates to check`); + // console.log(`Found ${certificates.length} certificates to check`); // Check each certificate for (const cert of certificates) { await checkCertificateAndNotify(cert); } - console.log("Daily SSL certificates check completed"); + // console.log("Daily SSL certificates check completed"); } catch (error) { - console.error("Error during SSL certificates daily check:", error); + // console.error("Error during SSL certificates daily check:", error); } } @@ -37,7 +37,7 @@ export async function checkAllCertificatesAndNotify(): Promise { * Note: SSL checking is now handled by the Go service, this function focuses on notifications */ export async function checkCertificateAndNotify(certificate: SSLCertificate): Promise { - console.log(`Checking certificate for ${certificate.domain}...`); +// console.log(`Checking certificate for ${certificate.domain}...`); try { // Use the current certificate data (updated by Go service) @@ -47,7 +47,7 @@ export async function checkCertificateAndNotify(certificate: SSLCertificate): Pr const warningThreshold = Number(certificate.warning_threshold) || 30; const expiryThreshold = Number(certificate.expiry_threshold) || 7; - console.log(`Certificate ${certificate.domain} thresholds: warning=${warningThreshold}, expiry=${expiryThreshold}, days left=${daysLeft}`); + // console.log(`Certificate ${certificate.domain} thresholds: warning=${warningThreshold}, expiry=${expiryThreshold}, days left=${daysLeft}`); // Update status based on thresholds const status = determineSSLStatus(daysLeft, warningThreshold, expiryThreshold); @@ -67,7 +67,7 @@ export async function checkCertificateAndNotify(certificate: SSLCertificate): Pr isCritical = false; } - console.log(`${certificate.domain}: ${daysLeft} days left, status: ${status}, should notify: ${shouldNotify}, critical: ${isCritical}`); + // console.log(`${certificate.domain}: ${daysLeft} days left, status: ${status}, should notify: ${shouldNotify}, critical: ${isCritical}`); // Update certificate status in database await pb.collection('ssl_certificates').update(certificate.id, { @@ -76,7 +76,7 @@ export async function checkCertificateAndNotify(certificate: SSLCertificate): Pr // Send notification if needed if (shouldNotify && certificate.notification_channel) { - console.log(`Sending notification for ${certificate.domain}`); + // console.log(`Sending notification for ${certificate.domain}`); // Different message based on expiry threshold const message = isCritical @@ -92,28 +92,28 @@ export async function checkCertificateAndNotify(certificate: SSLCertificate): Pr last_notified: new Date().toISOString() }); - console.log(`Notification sent for ${certificate.domain}`); + // console.log(`Notification sent for ${certificate.domain}`); // Show toast for manual checks toast.success(`Notification sent for ${certificate.domain}`); return true; } else { - console.error(`Failed to send notification for ${certificate.domain}`); + // console.error(`Failed to send notification for ${certificate.domain}`); // Show error toast for manual checks toast.error(`Failed to send notification for ${certificate.domain}`); return false; } } else if (shouldNotify && !certificate.notification_channel) { - console.log(`No notification channel set for ${certificate.domain}, skipping notification`); + // console.log(`No notification channel set for ${certificate.domain}, skipping notification`); toast.info(`No notification channel set for ${certificate.domain}, skipping notification`); } else { - console.log(`No notification needed for ${certificate.domain} (${daysLeft} days left)`); + // console.log(`No notification needed for ${certificate.domain} (${daysLeft} days left)`); // For manual checks, inform the user that thresholds weren't met toast.info(`Certificate for ${certificate.domain} is valid (${daysLeft} days left)`); } return true; } catch (error) { - console.error(`Error checking certificate for ${certificate.domain}:`, error); + // console.error(`Error checking certificate for ${certificate.domain}:`, error); toast.error(`Error checking certificate: ${error instanceof Error ? error.message : "Unknown error"}`); return false; } diff --git a/application/src/services/types/serverService.ts b/application/src/services/types/serverService.ts index e95d9d7..179096f 100644 --- a/application/src/services/types/serverService.ts +++ b/application/src/services/types/serverService.ts @@ -8,7 +8,7 @@ export const serverService = { const records = await pb.collection('servers').getFullList(); return records; } catch (error) { - console.error('Error fetching servers:', error); + // console.error('Error fetching servers:', error); throw error; } }, @@ -18,29 +18,29 @@ export const serverService = { const record = await pb.collection('servers').getOne(serverId); return record; } catch (error) { - console.error('Error fetching server:', error); + // console.error('Error fetching server:', error); throw error; } }, async getServerMetrics(serverId: string): Promise { try { - console.log('serverService.getServerMetrics: Fetching metrics for serverId:', serverId); + // console.log('serverService.getServerMetrics: Fetching metrics for serverId:', serverId); // First, get the server to find the correct server_id for metrics let server; try { server = await this.getServer(serverId); - console.log('serverService.getServerMetrics: Found server:', server); + // console.log('serverService.getServerMetrics: Found server:', server); } catch (error) { - console.log('serverService.getServerMetrics: Could not fetch server details:', error); + // console.log('serverService.getServerMetrics: Could not fetch server details:', error); } // Try to get metrics using the server's server_id field if available let metricsServerId = serverId; if (server && server.server_id) { metricsServerId = server.server_id; - console.log('serverService.getServerMetrics: Using server.server_id for metrics:', metricsServerId); + // console.log('serverService.getServerMetrics: Using server.server_id for metrics:', metricsServerId); } // Try filtering by server_id first @@ -50,11 +50,11 @@ export const serverService = { requestKey: null }); - console.log('serverService.getServerMetrics: Filtered records by server_id:', filteredRecords.length); + // console.log('serverService.getServerMetrics: Filtered records by server_id:', filteredRecords.length); // If no records found with server_id, try alternative approaches if (filteredRecords.length === 0) { - console.log('serverService.getServerMetrics: No records found with server_id filter, trying alternatives...'); + // console.log('serverService.getServerMetrics: No records found with server_id filter, trying alternatives...'); // Get all records to see what's available const allRecords = await pb.collection('server_metrics').getFullList({ @@ -62,24 +62,24 @@ export const serverService = { requestKey: null }); - console.log('serverService.getServerMetrics: Total server_metrics records:', allRecords.length); + // console.log('serverService.getServerMetrics: Total server_metrics records:', allRecords.length); if (allRecords.length > 0) { - console.log('serverService.getServerMetrics: Sample record fields:', Object.keys(allRecords[0])); - console.log('serverService.getServerMetrics: Sample server_id values:', allRecords.slice(0, 5).map(r => r.server_id)); + // console.log('serverService.getServerMetrics: Sample record fields:', Object.keys(allRecords[0])); + // console.log('serverService.getServerMetrics: Sample server_id values:', allRecords.slice(0, 5).map(r => r.server_id)); } // For now, return some sample data from available records if server matches pattern // This is temporary until the correct server_id mapping is established if (allRecords.length > 0) { - console.log('serverService.getServerMetrics: Using available records as fallback'); + // console.log('serverService.getServerMetrics: Using available records as fallback'); filteredRecords = allRecords.slice(0, 50); // Get recent 50 records } } - console.log('serverService.getServerMetrics: Returning', filteredRecords.length, 'records'); + // console.log('serverService.getServerMetrics: Returning', filteredRecords.length, 'records'); return filteredRecords; } catch (error) { - console.error('Error fetching server metrics:', error); + // console.error('Error fetching server metrics:', error); throw error; } },