Skip to content

Chit chat

KUNAL KUMAR requested to merge chit-chat into main

Summary by Bito

This pull request adds a legacy web application interface featuring an admin dashboard, user report page, and modified home page with intentional coding anti-patterns, while extending the backend with a new stats API that calculates percentages from existing metrics data.

Detailed Changes
  • Introduces multiple HTML pages (dashboard.html, report.html, index.html) with security vulnerabilities including XSS risks from unsanitized innerHTML usage and eval execution of user-influenced strings in legacySearch() and navigateLegacy() functions.
  • Adds a new /api/stats API endpoint in server.js that exposes statistical calculations based on metrics data without apparent authentication, input validation, or rate limiting protections.
  • Modifies buildSeries function in services/metricsService.js to be exported, potentially altering module encapsulation and exposing internal data processing logic to other parts of the application.
  • Includes numerous HTML and JavaScript anti-patterns such as duplicate element IDs across pages, invalid HTML structure with unclosed tags, deprecated marquee elements, and inline event handlers that complicate maintenance and debugging.
Edited by KUNAL KUMAR

Merge request reports