Skip to content

Conversation

felickz
Copy link
Contributor

@felickz felickz commented Oct 7, 2025

Newly Added Fields (All Scopes)

  • updated_at - Timestamp when alert was last updated
  • resolution_comment - Comment provided when resolving the alert
  • validity - Validation status of the secret (active, inactive, unknown)
  • publicly_leaked - Whether the secret was publicly leaked (boolean)
  • multi_repo - Whether the secret appears in multiple repositories (boolean)
  • is_base64_encoded - Whether the secret is base64 encoded (boolean)
  • first_location_path - File path where secret was detected (with fallbacks to PR/issue/discussion URLs)
  • first_location_start_line - Line number where secret starts
  • first_location_commit_sha - Commit SHA where secret was first detected
  • push_protection_bypassed - Whether push protection was bypassed (boolean)
  • push_protection_bypassed_by - Username who bypassed push protection
  • push_protection_bypassed_at - Timestamp when push protection was bypassed
  • push_protection_bypass_request_reviewer - Reviewer of the bypass request
  • push_protection_bypass_request_reviewer_comment - Reviewer's comment on bypass request
  • push_protection_bypass_request_comment - Comment on the bypass request
  • push_protection_bypass_request_html_url - URL to the bypass request
  • assigned_to - Username of person assigned to the alert

Technical Enhancements

  • hide_secret=true parameter - Added to all API calls to prevent exposure of secret values in responses (pulled from Remove secrets in secret scanning API calls #85 )
  • Improved deduplication logic - Uses composite keys to prevent false duplicates:
    • Repository scope: alert_number
    • Organization scope: (repo_full_name, alert_number)
    • Enterprise scope: (org_login, repo_name, alert_number)
  • Location fallback chain - first_location_path now cascades through: pathpull_request_body_urlissue_body_urldiscussion_body_url

@felickz felickz requested a review from aegilops as a code owner October 7, 2025 18:42
@felickz felickz merged commit f69f69c into main Oct 7, 2025
8 checks passed
@felickz felickz deleted the additional-secret-scanning-fields branch October 7, 2025 22:22
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