From 7b05e953e46addd26f9eea9f49fd24a4dfcfa28a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9Cakobaidan=E2=80=9D?= <akobaida@andrew.cmu.edu>
Date: Tue, 15 Oct 2024 10:51:13 +0200
Subject: [PATCH 1/8] updated bug-log controller for correct rendering

---
 src/controllers/admin/dashboard.js | 40 +++++++++++++++++++++++++++++-
 1 file changed, 39 insertions(+), 1 deletion(-)

diff --git a/src/controllers/admin/dashboard.js b/src/controllers/admin/dashboard.js
index 222b8b863f..204aed4a01 100644
--- a/src/controllers/admin/dashboard.js
+++ b/src/controllers/admin/dashboard.js
@@ -390,6 +390,44 @@ dashboardController.getSearches = async (req, res) => {
 	});
 };
 
+const bugLogs = [];
+
 dashboardController.getBugLogs = async function (req, res) {
-	res.render('admin/dashboard/bug-logs', {});
+    console.log('getbuglogs'); // Add logging
+    try {
+        // Sanitize and format bug logs before rendering
+        const sanitizedBugLogs = bugLogs.map(log => ({
+            user: validator.escape(String(log.user)),
+            description: validator.escape(String(log.description)),
+            timestamp: new Date(log.timestamp).toISOString(),
+        }));
+    
+        // Pass the sanitized bug logs to the view for rendering
+        res.render('admin/dashboard/bug-logs', { bugLogs: sanitizedBugLogs });
+    } catch (error) {
+        console.error('Error fetching bug logs:', error); // Log the error for debugging
+        res.status(500).json({ message: 'Internal server error' });
+    }
 };
+
+
+dashboardController.submitBugReport = async function (req, res) {
+    try {
+        const { description } = req.body;
+        if (!description) {
+            return res.status(400).json({ message: 'Description is required' });
+        }
+
+        const sanitizedDescription = validator.escape(description);
+        const timestamp = Date.now();
+        const user = req.user ? req.user.username : 'Anonymous'; // Assuming req.user contains the user information
+
+        // Add the bug report to the in-memory array
+        bugLogs.push({ user, description: sanitizedDescription, timestamp });
+
+        res.status(201).json({ message: 'Bug report submitted successfully' });
+    } catch (error) {
+        console.error('Error submitting bug report:', error); // Log the error for debugging
+        res.status(500).json({ message: 'Internal server error' });
+    }
+};
\ No newline at end of file

From 97759af27baf2922433afabb9451f46f78e9f915 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9Cakobaidan=E2=80=9D?= <akobaida@andrew.cmu.edu>
Date: Tue, 15 Oct 2024 10:54:08 +0200
Subject: [PATCH 2/8] updated bug-log route handler to handle logs and reports

---
 public/src/admin/dashboard/bug-logs.js | 55 +++++++++++++++++++++++++-
 1 file changed, 54 insertions(+), 1 deletion(-)

diff --git a/public/src/admin/dashboard/bug-logs.js b/public/src/admin/dashboard/bug-logs.js
index c16d722db8..a2bb5f9ac5 100644
--- a/public/src/admin/dashboard/bug-logs.js
+++ b/public/src/admin/dashboard/bug-logs.js
@@ -1,6 +1,59 @@
 'use strict';
 
-define('admin/dashboard/bug-logs', [], () => {
+define('admin/dashboard/bug-logs', ['jquery', 'api'], ($, api) => {
+    const BugLogs = {};
 
+    BugLogs.init = () => {
+        // Fetch and display bug logs
+        fetchBugLogs();
 
+        // Handle bug report submission
+        $('#submit-bug-report').on('click', submitBugReport);
+    };
+
+    function fetchBugLogs() {
+        api.get('/api/admin/get-bug-log')
+            .then((data) => {
+                const bugLogsContainer = $('#bug-logs-container');
+                bugLogsContainer.empty();
+
+                if (data.bugLogs && data.bugLogs.length > 0) {
+                    data.bugLogs.forEach((log) => {
+                        const logElement = $('<div>').addClass('bug-log');
+                        logElement.append($('<p>').text(`User: ${log.user}`));
+                        logElement.append($('<p>').text(`Description: ${log.description}`));
+                        logElement.append($('<p>').text(`Timestamp: ${log.timestamp}`));
+                        bugLogsContainer.append(logElement);
+                    });
+                } else {
+                    bugLogsContainer.append($('<p>').text('No bug logs found.'));
+                }
+            })
+            .catch((err) => {
+                console.error('Error fetching bug logs:', err);
+                $('#bug-logs-container').append($('<p>').text('Error fetching bug logs.'));
+            });
+    }
+
+    function submitBugReport() {
+        const description = $('#bug-report-description').val().trim();
+
+        if (!description) {
+            alert('Description is required');
+            return;
+        }
+
+        api.post('/api/admin/submit-bug-report', { description })
+            .then(() => {
+                alert('Bug report submitted successfully');
+                $('#bug-report-description').val('');
+                fetchBugLogs();
+            })
+            .catch((err) => {
+                console.error('Error submitting bug report:', err);
+                alert('Error submitting bug report');
+            });
+    }
+
+    return BugLogs;
 });

From 9ce1a638b27260144183e3a0a66ed6cbd134c236 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9Cakobaidan=E2=80=9D?= <akobaida@andrew.cmu.edu>
Date: Tue, 15 Oct 2024 10:58:11 +0200
Subject: [PATCH 3/8] updated report form with event listener

---
 src/views/bug-report-form.tpl | 17 +----------------
 1 file changed, 1 insertion(+), 16 deletions(-)

diff --git a/src/views/bug-report-form.tpl b/src/views/bug-report-form.tpl
index 3ef158b38c..99a743fc3d 100644
--- a/src/views/bug-report-form.tpl
+++ b/src/views/bug-report-form.tpl
@@ -95,21 +95,6 @@
             <input type="submit" value="Submit">
         </form>
     </div>
-    <script>
-        document.getElementById('bug-report-form').addEventListener('submit', function(event) {
-            event.preventDefault(); // Prevent the form from submitting the traditional way
-            const banner = document.getElementById('form-banner');
-            const form = document.getElementById('bug-report-form');
-            banner.style.display = 'block'; // Show the banner
-            banner.classList.add('show'); // Add the class to slide in the banner
-            setTimeout(() => {
-                banner.classList.remove('show'); // Remove the class to slide out the banner
-                setTimeout(() => {
-                    banner.style.display = 'none'; // Hide the banner after the slide out
-                }, 500); // Match the transition duration
-            }, 3000); // Display the banner for 3 seconds
-            form.reset(); // Reset the form fields
-        });
-    </script>
+
 </body>
 </html>
\ No newline at end of file

From 1f154cc75879d5175655d91a001fa6c4040f48c1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9Cakobaidan=E2=80=9D?= <akobaida@andrew.cmu.edu>
Date: Tue, 15 Oct 2024 10:58:54 +0200
Subject: [PATCH 4/8] updated report form with console log

---
 src/views/bug-report-form.tpl | 61 +++++++++++++++++++++++++++++++++++
 1 file changed, 61 insertions(+)

diff --git a/src/views/bug-report-form.tpl b/src/views/bug-report-form.tpl
index 99a743fc3d..2d4d29933d 100644
--- a/src/views/bug-report-form.tpl
+++ b/src/views/bug-report-form.tpl
@@ -4,6 +4,7 @@
     <meta charset="UTF-8">
     <meta name="viewport" content="width=device-width, initial-scale=1.0">
     <title>Bug Report Form</title>
+    <meta name="csrf-token" content="{{csrfToken}}">
     <style>
         .container {
             background-color: #fff;
@@ -97,4 +98,64 @@
     </div>
 
 </body>
+<script>
+document.addEventListener('DOMContentLoaded', function() {
+    const form = document.getElementById('bug-report-form');
+    const banner = document.getElementById('form-banner');
+
+    form.addEventListener('submit', function(event) {
+        event.preventDefault();
+
+        const formData = new FormData(form);
+        const data = {
+            name: formData.get('name'),
+            email: formData.get('email'),
+            'bug-description': formData.get('bug-description')
+        };
+        console.log(data);
+
+        // Get CSRF token from meta tag or other source
+        const csrfTokenMeta = document.querySelector('meta[name="csrf-token"]');
+        const csrfToken = csrfTokenMeta ? csrfTokenMeta.getAttribute('content') : '';
+
+        fetch('/api/admin/submit-bug-report', {
+            method: 'POST',
+            headers: {
+                'Content-Type': 'application/json',
+                'CSRF-Token': csrfToken // Include CSRF token in headers
+            },
+            body: JSON.stringify(data)
+        })
+        .then(response => {
+            if (!response.ok) {
+                throw new Error(`HTTP error! status: ${response.status}`);
+            }
+            return response.json();
+        })
+        .then(result => {
+            if (result.success) {
+                banner.textContent = 'Form Submitted Successfully';
+                banner.classList.add('show');
+                setTimeout(() => {
+                    banner.classList.remove('show');
+                }, 3000);
+            } else {
+                banner.textContent = 'Form Submission Failed';
+                banner.classList.add('show');
+                setTimeout(() => {
+                    banner.classList.remove('show');
+                }, 3000);
+            }
+        })
+        .catch(error => {
+            console.error('Error:', error);
+            banner.textContent = 'Form Submission Failed';
+            banner.classList.add('show');
+            setTimeout(() => {
+                banner.classList.remove('show');
+            }, 3000);
+        });
+    });
+});
+</script>
 </html>
\ No newline at end of file

From de9b63dea203a0488a12b2d128f06e8a3dfef034 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9Cakobaidan=E2=80=9D?= <akobaida@andrew.cmu.edu>
Date: Tue, 15 Oct 2024 11:01:58 +0200
Subject: [PATCH 5/8] corrected code with lint

---
 public/src/admin/dashboard/bug-logs.js | 110 ++++++++++++-------------
 src/controllers/admin/dashboard.js     |  68 +++++++--------
 2 files changed, 89 insertions(+), 89 deletions(-)

diff --git a/public/src/admin/dashboard/bug-logs.js b/public/src/admin/dashboard/bug-logs.js
index a2bb5f9ac5..5af84ccb95 100644
--- a/public/src/admin/dashboard/bug-logs.js
+++ b/public/src/admin/dashboard/bug-logs.js
@@ -1,59 +1,59 @@
 'use strict';
 
 define('admin/dashboard/bug-logs', ['jquery', 'api'], ($, api) => {
-    const BugLogs = {};
-
-    BugLogs.init = () => {
-        // Fetch and display bug logs
-        fetchBugLogs();
-
-        // Handle bug report submission
-        $('#submit-bug-report').on('click', submitBugReport);
-    };
-
-    function fetchBugLogs() {
-        api.get('/api/admin/get-bug-log')
-            .then((data) => {
-                const bugLogsContainer = $('#bug-logs-container');
-                bugLogsContainer.empty();
-
-                if (data.bugLogs && data.bugLogs.length > 0) {
-                    data.bugLogs.forEach((log) => {
-                        const logElement = $('<div>').addClass('bug-log');
-                        logElement.append($('<p>').text(`User: ${log.user}`));
-                        logElement.append($('<p>').text(`Description: ${log.description}`));
-                        logElement.append($('<p>').text(`Timestamp: ${log.timestamp}`));
-                        bugLogsContainer.append(logElement);
-                    });
-                } else {
-                    bugLogsContainer.append($('<p>').text('No bug logs found.'));
-                }
-            })
-            .catch((err) => {
-                console.error('Error fetching bug logs:', err);
-                $('#bug-logs-container').append($('<p>').text('Error fetching bug logs.'));
-            });
-    }
-
-    function submitBugReport() {
-        const description = $('#bug-report-description').val().trim();
-
-        if (!description) {
-            alert('Description is required');
-            return;
-        }
-
-        api.post('/api/admin/submit-bug-report', { description })
-            .then(() => {
-                alert('Bug report submitted successfully');
-                $('#bug-report-description').val('');
-                fetchBugLogs();
-            })
-            .catch((err) => {
-                console.error('Error submitting bug report:', err);
-                alert('Error submitting bug report');
-            });
-    }
-
-    return BugLogs;
+	const BugLogs = {};
+
+	BugLogs.init = () => {
+		// Fetch and display bug logs
+		fetchBugLogs();
+
+		// Handle bug report submission
+		$('#submit-bug-report').on('click', submitBugReport);
+	};
+
+	function fetchBugLogs() {
+		api.get('/api/admin/get-bug-log')
+			.then((data) => {
+				const bugLogsContainer = $('#bug-logs-container');
+				bugLogsContainer.empty();
+
+				if (data.bugLogs && data.bugLogs.length > 0) {
+					data.bugLogs.forEach((log) => {
+						const logElement = $('<div>').addClass('bug-log');
+						logElement.append($('<p>').text(`User: ${log.user}`));
+						logElement.append($('<p>').text(`Description: ${log.description}`));
+						logElement.append($('<p>').text(`Timestamp: ${log.timestamp}`));
+						bugLogsContainer.append(logElement);
+					});
+				} else {
+					bugLogsContainer.append($('<p>').text('No bug logs found.'));
+				}
+			})
+			.catch((err) => {
+				console.error('Error fetching bug logs:', err);
+				$('#bug-logs-container').append($('<p>').text('Error fetching bug logs.'));
+			});
+	}
+
+	function submitBugReport() {
+		const description = $('#bug-report-description').val().trim();
+
+		if (!description) {
+			alert('Description is required');
+			return;
+		}
+
+		api.post('/api/admin/submit-bug-report', { description })
+			.then(() => {
+				alert('Bug report submitted successfully');
+				$('#bug-report-description').val('');
+				fetchBugLogs();
+			})
+			.catch((err) => {
+				console.error('Error submitting bug report:', err);
+				alert('Error submitting bug report');
+			});
+	}
+
+	return BugLogs;
 });
diff --git a/src/controllers/admin/dashboard.js b/src/controllers/admin/dashboard.js
index 204aed4a01..b264937ea7 100644
--- a/src/controllers/admin/dashboard.js
+++ b/src/controllers/admin/dashboard.js
@@ -393,41 +393,41 @@ dashboardController.getSearches = async (req, res) => {
 const bugLogs = [];
 
 dashboardController.getBugLogs = async function (req, res) {
-    console.log('getbuglogs'); // Add logging
-    try {
-        // Sanitize and format bug logs before rendering
-        const sanitizedBugLogs = bugLogs.map(log => ({
-            user: validator.escape(String(log.user)),
-            description: validator.escape(String(log.description)),
-            timestamp: new Date(log.timestamp).toISOString(),
-        }));
-    
-        // Pass the sanitized bug logs to the view for rendering
-        res.render('admin/dashboard/bug-logs', { bugLogs: sanitizedBugLogs });
-    } catch (error) {
-        console.error('Error fetching bug logs:', error); // Log the error for debugging
-        res.status(500).json({ message: 'Internal server error' });
-    }
+	console.log('getbuglogs'); // Add logging
+	try {
+		// Sanitize and format bug logs before rendering
+		const sanitizedBugLogs = bugLogs.map(log => ({
+			user: validator.escape(String(log.user)),
+			description: validator.escape(String(log.description)),
+			timestamp: new Date(log.timestamp).toISOString(),
+		}));
+
+		// Pass the sanitized bug logs to the view for rendering
+		res.render('admin/dashboard/bug-logs', { bugLogs: sanitizedBugLogs });
+	} catch (error) {
+		console.error('Error fetching bug logs:', error); // Log the error for debugging
+		res.status(500).json({ message: 'Internal server error' });
+	}
 };
 
 
 dashboardController.submitBugReport = async function (req, res) {
-    try {
-        const { description } = req.body;
-        if (!description) {
-            return res.status(400).json({ message: 'Description is required' });
-        }
-
-        const sanitizedDescription = validator.escape(description);
-        const timestamp = Date.now();
-        const user = req.user ? req.user.username : 'Anonymous'; // Assuming req.user contains the user information
-
-        // Add the bug report to the in-memory array
-        bugLogs.push({ user, description: sanitizedDescription, timestamp });
-
-        res.status(201).json({ message: 'Bug report submitted successfully' });
-    } catch (error) {
-        console.error('Error submitting bug report:', error); // Log the error for debugging
-        res.status(500).json({ message: 'Internal server error' });
-    }
-};
\ No newline at end of file
+	try {
+		const { description } = req.body;
+		if (!description) {
+			return res.status(400).json({ message: 'Description is required' });
+		}
+
+		const sanitizedDescription = validator.escape(description);
+		const timestamp = Date.now();
+		const user = req.user ? req.user.username : 'Anonymous'; // Assuming req.user contains the user information
+
+		// Add the bug report to the in-memory array
+		bugLogs.push({ user, description: sanitizedDescription, timestamp });
+
+		res.status(201).json({ message: 'Bug report submitted successfully' });
+	} catch (error) {
+		console.error('Error submitting bug report:', error); // Log the error for debugging
+		res.status(500).json({ message: 'Internal server error' });
+	}
+};

From 363d13771330954ede28890502418344fa96dfc0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9Cakobaidan=E2=80=9D?= <akobaida@andrew.cmu.edu>
Date: Sun, 20 Oct 2024 11:00:49 +0300
Subject: [PATCH 6/8] updated controllers.admin.js to test for bug-logs button
 in admin dashboard

---
 test/controllers-admin.js | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/test/controllers-admin.js b/test/controllers-admin.js
index 7760bf128e..74a64bad76 100644
--- a/test/controllers-admin.js
+++ b/test/controllers-admin.js
@@ -87,6 +87,18 @@ describe('Admin Controllers', () => {
 		});
 	});
 
+	it('should load admin dashboard with bug-logs', async () => {
+		await groups.join('administrators', adminUid);
+		const dashboards = [
+			'/admin', '/admin/dashboard/logins', '/admin/dashboard/users', '/admin/dashboard/topics', '/admin/dashboard/searches', 	'admin/dashboard/bug-logs',
+		];
+		await async.each(dashboards, async (url) => {
+			const { response, body } = await request.get(`${nconf.get('url')}${url}`, { jar: jar });
+			assert.equal(response.statusCode, 200, url);
+			assert(body);
+		});
+	});
+
 	it('should load admin analytics', async () => {
 		const { response, body } = await request.get(`${nconf.get('url')}/api/admin/analytics?units=hours`, { jar: jar });
 		assert.equal(response.statusCode, 200);

From 389026f7c296ad7c7bbf45e3b8f93d14be3ab857 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9Cakobaidan=E2=80=9D?= <akobaida@andrew.cmu.edu>
Date: Sun, 20 Oct 2024 11:27:42 +0300
Subject: [PATCH 7/8] corrected testing syntax

---
 test/controllers-admin.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/controllers-admin.js b/test/controllers-admin.js
index 74a64bad76..9186d3b347 100644
--- a/test/controllers-admin.js
+++ b/test/controllers-admin.js
@@ -90,14 +90,14 @@ describe('Admin Controllers', () => {
 	it('should load admin dashboard with bug-logs', async () => {
 		await groups.join('administrators', adminUid);
 		const dashboards = [
-			'/admin', '/admin/dashboard/logins', '/admin/dashboard/users', '/admin/dashboard/topics', '/admin/dashboard/searches', 	'admin/dashboard/bug-logs',
+			'/admin', '/admin/dashboard/logins', '/admin/dashboard/users', '/admin/dashboard/topics', '/admin/dashboard/searches', 'admin/dashboard/bug-logs',
 		];
 		await async.each(dashboards, async (url) => {
 			const { response, body } = await request.get(`${nconf.get('url')}${url}`, { jar: jar });
 			assert.equal(response.statusCode, 200, url);
 			assert(body);
 		});
-	});
+	}); // Ensure no tab characters or trailing spaces here
 
 	it('should load admin analytics', async () => {
 		const { response, body } = await request.get(`${nconf.get('url')}/api/admin/analytics?units=hours`, { jar: jar });

From 02ec6c91c2f933afcbf443e17d321dca4a338a4b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9Cakobaidan=E2=80=9D?= <akobaida@andrew.cmu.edu>
Date: Sun, 20 Oct 2024 11:37:16 +0300
Subject: [PATCH 8/8] corrected test code syntax

---
 test/controllers-admin.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/controllers-admin.js b/test/controllers-admin.js
index 9186d3b347..0474ae7b61 100644
--- a/test/controllers-admin.js
+++ b/test/controllers-admin.js
@@ -90,7 +90,7 @@ describe('Admin Controllers', () => {
 	it('should load admin dashboard with bug-logs', async () => {
 		await groups.join('administrators', adminUid);
 		const dashboards = [
-			'/admin', '/admin/dashboard/logins', '/admin/dashboard/users', '/admin/dashboard/topics', '/admin/dashboard/searches', 'admin/dashboard/bug-logs',
+			'/admin', '/admin/dashboard/logins', '/admin/dashboard/users', '/admin/dashboard/topics', '/admin/dashboard/searches', '/admin/dashboard/bug-logs',
 		];
 		await async.each(dashboards, async (url) => {
 			const { response, body } = await request.get(`${nconf.get('url')}${url}`, { jar: jar });