apunkt/SurfSense
1
0
Fork 0
mirror of https://github.com/MODSetter/SurfSense.git synced 2026-07-22 23:31:12 +02:00
Code Issues Projects Releases Packages Wiki Activity

fix: resolve setTimeout memory leak on unmounted component

Browse source
This commit is contained in:
SohamBhattacharjee2003 2026-04-04 01:28:45 +05:30
parent 414c4c86e9
commit 0cd997f673
2 changed files with 19 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

10
surfsense_web/components/report-panel/report-panel.tsx
View file

@ -123,6 +123,13 @@ export function ReportPanelContent({
const [copied, setCopied] = useState(false); const [copied, setCopied] = useState(false);
const [exporting, setExporting] = useState<string | null>(null); const [exporting, setExporting] = useState<string | null>(null);
const [saving, setSaving] = useState(false); const [saving, setSaving] = useState(false);
const copyTimerRef = useRef<ReturnType<typeof setTimeout> | undefined>(undefined);
useEffect(() => {
return () => {
if (copyTimerRef.current) clearTimeout(copyTimerRef.current);
};
}, []);
// Editor state — tracks the latest markdown from the Plate editor // Editor state — tracks the latest markdown from the Plate editor
const [editedMarkdown, setEditedMarkdown] = useState<string | null>(null); const [editedMarkdown, setEditedMarkdown] = useState<string | null>(null);
@ -197,7 +204,8 @@ export function ReportPanelContent({
try { try {
await navigator.clipboard.writeText(currentMarkdown); await navigator.clipboard.writeText(currentMarkdown);
setCopied(true); setCopied(true);
setTimeout(() => setCopied(false), 2000); if (copyTimerRef.current) clearTimeout(copyTimerRef.current);
copyTimerRef.current = setTimeout(() => setCopied(false), 2000);
} catch (err) { } catch (err) {
console.error("Failed to copy:", err); console.error("Failed to copy:", err);
} }

12
surfsense_web/hooks/use-api-key.ts
View file

@ -1,4 +1,4 @@
import { useCallback, useEffect, useState } from "react"; import { useCallback, useEffect, useRef, useState } from "react";
import { toast } from "sonner"; import { toast } from "sonner";
import { getBearerToken } from "@/lib/auth-utils"; import { getBearerToken } from "@/lib/auth-utils";
import { copyToClipboard as copyToClipboardUtil } from "@/lib/utils"; import { copyToClipboard as copyToClipboardUtil } from "@/lib/utils";
@ -14,6 +14,13 @@ export function useApiKey(): UseApiKeyReturn {
const [apiKey, setApiKey] = useState<string | null>(null); const [apiKey, setApiKey] = useState<string | null>(null);
const [copied, setCopied] = useState(false); const [copied, setCopied] = useState(false);
const [isLoading, setIsLoading] = useState(true); const [isLoading, setIsLoading] = useState(true);
const copyTimerRef = useRef<ReturnType<typeof setTimeout> | undefined>(undefined);
useEffect(() => {
return () => {
if (copyTimerRef.current) clearTimeout(copyTimerRef.current);
};
}, []);
useEffect(() => { useEffect(() => {
// Load API key from localStorage // Load API key from localStorage
@ -41,7 +48,8 @@ export function useApiKey(): UseApiKeyReturn {
if (success) { if (success) {
setCopied(true); setCopied(true);
toast.success("API key copied to clipboard"); toast.success("API key copied to clipboard");
setTimeout(() => { if (copyTimerRef.current) clearTimeout(copyTimerRef.current);
copyTimerRef.current = setTimeout(() => {
setCopied(false); setCopied(false);
}, 2000); }, 2000);
} else { } else {