Skip to content

Conversation

@chrismaddalena
Copy link
Collaborator

@chrismaddalena chrismaddalena commented Jan 22, 2026

CHANGELOG

[6.2.2] - 22 January 2026

Added

  • Added a sample BloodHound report to the DOCS/sample_reports directory
  • Added severity color values to BloodHound Enterprise findings data
    • These values mirror the color values provided for Ghostwriter findings (e.g., color, color_rgb, and color_hex)
    • You can use these values in report templates with Jinja2 templating lie cellbg
    • Examples are in the sample BloodHOund report document

Fixed

  • Added handling for an error that could occur when fetching BloodHound domains with an Azure tenant reported as an available domain

Copilot AI review requested due to automatic review settings January 22, 2026 01:34
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 BloodHound integration with UI improvements and bug fixes. It addresses error handling for Azure tenants without AD data, improves button styling consistency across screen sizes, and enriches BloodHound findings with severity color metadata to match native Ghostwriter findings.

Changes:

  • Added error handling for Azure tenant domain lookups that may return 404 errors
  • Fixed button text wrapping and height inconsistencies on narrow screens through new CSS utility classes
  • Enhanced BloodHound Enterprise findings with severity color properties (color, color_rgb, color_hex) matching Ghostwriter's native severity model

Reviewed changes

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

File Description
ghostwriter/static/css/styles.css Adds CSS utility classes for button text wrapping and equal-height button groups
ghostwriter/shepherd/external/bloodhound/client.py Adds severity color fields to findings, handles Azure tenant errors, optimizes severity lookups
ghostwriter/rolodex/templates/rolodex/project_detail.html Updates BloodHound buttons to use new CSS classes for consistent sizing and text wrapping

@codecov
Copy link

codecov bot commented Jan 22, 2026

Codecov Report

❌ Patch coverage is 3.44828% with 28 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.36%. Comparing base (e22fb59) to head (f13e516).
⚠️ Report is 19 commits behind head on master.

Files with missing lines Patch % Lines
ghostwriter/shepherd/external/bloodhound/client.py 3.44% 28 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #804      +/-   ##
==========================================
- Coverage   91.43%   91.36%   -0.08%     
==========================================
  Files         368      368              
  Lines       20941    20933       -8     
==========================================
- Hits        19148    19126      -22     
- Misses       1793     1807      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

chrismaddalena and others added 2 commits January 21, 2026 17:46
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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 3 out of 4 changed files in this pull request and generated 1 comment.

@chrismaddalena chrismaddalena merged commit 9e172ff into master Jan 22, 2026
6 of 9 checks passed
@chrismaddalena chrismaddalena deleted the feature/bh-enhancements branch January 22, 2026 23:15
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