Skip to content

Conversation

@Zacgoose
Copy link

Added functions to compute page source hash and check for significant changes. Implemented scheduling for threat-triggered re-scans with progressive delays.

Copilot AI review requested due to automatic review settings November 24, 2025 09:33
Copilot finished reviewing on behalf of Zacgoose November 24, 2025 09:37
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the security extension's page monitoring capabilities by adding source hash-based change detection and implementing a threat-triggered re-scan mechanism with progressive delays to catch late-loading malicious content.

Key changes:

  • Implemented djb2-based hashing with intelligent sampling to efficiently detect page source changes
  • Added automated re-scan scheduling when threats are detected, with progressive delays (800ms, 2000ms) and a maximum of 2 follow-up scans
  • Integrated page source change detection into the rate limiting logic to avoid unnecessary re-scans when content hasn't actually changed

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Zacgoose Zacgoose marked this pull request as draft December 4, 2025 00:21
@Zacgoose
Copy link
Author

Zacgoose commented Dec 5, 2025

do not worry, I will squash and rebase if required...

@Zacgoose Zacgoose requested a review from Copilot December 5, 2025 10:58
Copilot finished reviewing on behalf of Zacgoose December 5, 2025 10:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 32 out of 33 changed files in this pull request and generated 7 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 37 out of 38 changed files in this pull request and generated 5 comments.

Comments suppressed due to low confidence (1)

scripts/content.js:1713

  • Unused variable result.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Introduces full Firefox 109+ support, including new manifest, browser polyfill, and enterprise deployment via policies.json. Updates documentation to cover Firefox installation, deployment, and configuration. Adds new settings (e.g., validPageBadgeTimeout), expands detection rule documentation for code-driven logic, and updates branding and general settings guides for cross-browser compatibility.
@Zacgoose Zacgoose marked this pull request as ready for review December 6, 2025 04:09
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.

2 participants