Skip to content

Conversation

@jeremy5189
Copy link
Contributor

No description provided.

@jeremy5189 jeremy5189 requested a review from Copilot June 5, 2025 18:48

This comment was marked as outdated.

@jeremy5189 jeremy5189 requested a review from Copilot June 5, 2025 19:51

This comment was marked as outdated.

@jeremy5189 jeremy5189 requested a review from Copilot June 5, 2025 19:55
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 introduces bot fingerprinting functionality to enhance bot detection and integrity checks. Key changes include:

  • Addition of new Nginx maps for bot fingerprint values.
  • Extension of the JavaScript payload in the SHA inverse challenge with additional fingerprint data.
  • Updates to integrity check functions, tests, and HTTP response headers to incorporate the new fingerprint information.

Reviewed Changes

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

Show a summary per file
File Description
supporting-containers/nginx/nginx.conf Added bot fingerprint mappings and updated logging JSON output.
internal/sha-inverse-challenge.html Converted the IIFE to an async arrow function and extended payload.
internal/integrity_check_test.go Updated tests to verify non-empty fingerprint values.
internal/integrity_check.go Refactored bot score calculation to include the fingerprint wrapper.
internal/http_server.go Updated access control functions to include fingerprint headers.

@jeremy5189 jeremy5189 merged commit 7266428 into main Jun 5, 2025
6 checks passed
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