Skip to content

Added footer#174

Open
MaryamL12 wants to merge 1 commit intomasterfrom
header
Open

Added footer#174
MaryamL12 wants to merge 1 commit intomasterfrom
header

Conversation

@MaryamL12
Copy link
Contributor

@MaryamL12 MaryamL12 commented Mar 3, 2026

✨ PR Description

Purpose: Replace static footer component with dynamic script-based footer that loads external footer content asynchronously from Azure CDN.

Main changes:

  • Removed hardcoded footer HTML with navigation links, social media icons, and React Bootstrap dependencies
  • Implemented dynamic footer script injection using React useEffect hook with duplicate script prevention
  • Deleted footer.css stylesheet and its import from layout.css to reduce bundle size

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

Copy link

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✨ PR Review

The PR refactors the footer to load dynamically from an external script, which reduces code maintenance burden. However, there's a critical security concern with loading external scripts without integrity verification.

2 issues detected:

🔒 Security - Loading external scripts without integrity checks allows potential code injection if the source is compromised

Details: External script is loaded without Subresource Integrity (SRI) verification. This creates a security vulnerability where compromised or hijacked external resources could inject malicious code into the application.
File: src/components/footer.js

🐞 Bug - Script load failures are undetected and unlogged, making issues difficult to diagnose

Details: No error handling for script load failures. If the external script fails to load due to network issues or unavailable resource, the failure will be silent making debugging difficult.
File: src/components/footer.js

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how

@gitstream-cm
Copy link

gitstream-cm bot commented Mar 3, 2026

Please mark whether you used Copilot to assist coding in this PR

  • Copilot Assisted

@MaryamL12 MaryamL12 marked this pull request as ready for review March 3, 2026 14:30
@MaryamL12 MaryamL12 requested a review from rjriel March 3, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant