From 403948f7b2e89ed880d88224876d32a60b149496 Mon Sep 17 00:00:00 2001
From: austinkelsay <austinkelsay@protonmail.com>
Date: Mon, 12 May 2025 11:17:57 -0500
Subject: [PATCH] fix for memory leak

---
 src/hooks/encryption/useCourseDecryption.js |  1 +
 src/pages/course/[slug]/index.js            | 21 +++++++--------------
 2 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/src/hooks/encryption/useCourseDecryption.js b/src/hooks/encryption/useCourseDecryption.js
index 2f57781..b1b4b53 100644
--- a/src/hooks/encryption/useCourseDecryption.js
+++ b/src/hooks/encryption/useCourseDecryption.js
@@ -93,6 +93,7 @@ const useCourseDecryption = (session, paidCourse, course, lessons, setLessons, r
         clearTimeout(timeoutId);
         decryptTimeoutRef.current = null;
       } catch (error) {
+        clearTimeout(timeoutId);
         // If timeout or network error, schedule a retry
         retryTimeoutRef.current = setTimeout(() => {
           processingRef.current = false;
diff --git a/src/pages/course/[slug]/index.js b/src/pages/course/[slug]/index.js
index f6985b3..cd81817 100644
--- a/src/pages/course/[slug]/index.js
+++ b/src/pages/course/[slug]/index.js
@@ -1,4 +1,4 @@
-import React, { useEffect, useState, useCallback } from 'react';
+import React, { useEffect, useState, useCallback, useMemo } from 'react';
 import { useRouter } from 'next/router';
 import { findKind0Fields } from '@/utils/nostr';
 import { useNDKContext } from '@/context/NDKContext';
@@ -35,7 +35,6 @@ const Course = () => {
   const [nsec, setNsec] = useState(null);
   const [npub, setNpub] = useState(null);
   const [nAddress, setNAddress] = useState(null);
-  const [isDecrypting, setIsDecrypting] = useState(false);
   const windowWidth = useWindowWidth();
   const isMobileView = windowWidth <= 968;
   const navbarHeight = 60; // Match the height from Navbar component
@@ -140,18 +139,12 @@ const Course = () => {
     activeIndex
   );
 
-  useEffect(() => {
-    if (paidCourse && uniqueLessons.length > 0) {
-      const currentLesson = uniqueLessons[activeIndex];
-      if (currentLesson && !decryptedLessonIds[currentLesson.id]) {
-        setIsDecrypting(true);
-      } else {
-        setIsDecrypting(false);
-      }
-    } else {
-      setIsDecrypting(false);
-    }
-  }, [activeIndex, uniqueLessons, decryptedLessonIds, paidCourse]);
+  // Replace useState + useEffect with useMemo for derived state
+  const isDecrypting = useMemo(() => {
+    if (!paidCourse || uniqueLessons.length === 0) return false;
+    const current = uniqueLessons[activeIndex];
+    return current && !decryptedLessonIds[current.id];
+  }, [paidCourse, uniqueLessons, activeIndex, decryptedLessonIds]);
 
   useEffect(() => {
     if (uniqueLessons.length > 0) {