Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Scoubi
Copy link
Contributor

@Scoubi Scoubi commented Oct 27, 2025

Add: runZeroHound & TaskHound

Summary by CodeRabbit

  • Documentation
    • Added OpenGraph entries: ManagerOfHound (Active Directory), runZeroHound (AWS), TaskHound (Windows), and IAMhounddog (AWS).
    • Renamed and broadened the Rapid7 entry from "Rapid7 InsightVM" to "Rapid7" with an updated heading.
    • Organized the new entries into closed/collapsible groups in the OpenGraph library for easier browsing.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 27, 2025

Walkthrough

Adds and updates OpenGraph library catalog entries in documentation: new entries for ManagerOfHound (Active Directory), IAMhounddog (AWS), runZeroHound (AWS), TaskHound (Windows), and renames the Rapid7 entry from "Rapid7 InsightVM" to "Rapid7". Changes are confined to docs/opengraph/library.mdx.

Changes

Cohort / File(s) Summary
OpenGraph library documentation
docs/opengraph/library.mdx
Added entries: ManagerOfHound (Active Directory), IAMhounddog (AWS), runZeroHound (AWS), TaskHound (Windows); renamed Rapid7 InsightVMRapid7; added descriptions, authors/maintainers, and repository links; adjusted accordion grouping. No code or public API changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify entry metadata (descriptions, authors, repo URLs).
  • Confirm Rapid7 rename consistency and formatting of accordion blocks.

Possibly related PRs

Suggested labels

documentation

Suggested reviewers

  • StephenHinck
  • jeff-matthews

Poem

🐰 I hopped through lines of docs so bright,
New hounds and headings stitched just right.
A Rapid7 rename, four friends more,
I left a trail of carrots at the door. 🥕

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Title Check ⚠️ Warning The PR title "Add 3 Hounds" is related to the main objective of adding new hound entries to the documentation, but it contains a factual inaccuracy. The raw summary shows that the changeset adds four new hound entries: ManagerOfHound, IAMhounddog, runZeroHound, and TaskHound. The title's specific count of "3" does not match the actual number of hounds being added to the changeset, making it misleading for someone reviewing the pull request history. Update the title to accurately reflect the changeset. Either change it to "Add 4 Hounds" to match all new hound entries being added, or if only a subset were intended, make the title more specific such as "Add runZeroHound and TaskHound" to align with the PR objectives description and avoid misleading reviewers about the scope of changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 2-hounds

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5266da7 and e6bacb6.

📒 Files selected for processing (1)
  • docs/opengraph/library.mdx (2 hunks)
🔇 Additional comments (1)
docs/opengraph/library.mdx (1)

294-310: TaskHound entry looks good.

The new TaskHound entry follows the established formatting and structure of other community contributions. The description is clear, author attribution is present, and the repository link is included.

Copy link
Contributor

@jeff-matthews jeff-matthews left a comment

Choose a reason for hiding this comment

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

Other than CR's comment about the URL for runZero, looks good.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
docs/opengraph/library.mdx (1)

236-249: Minor: Clarify Rapid7 entry scope.

The accordion title was changed to "Rapid7" (from "Rapid7 InsightVM"), but the tool heading still says "OpenGraph DLT (Rapid7 InsightVM)". If this entry is now intended to cover broader Rapid7 products, consider updating the tool name heading for consistency. If it remains InsightVM-specific, the title could reflect that.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6157fb6 and 96a4473.

📒 Files selected for processing (1)
  • docs/opengraph/library.mdx (4 hunks)
🔇 Additional comments (3)
docs/opengraph/library.mdx (3)

59-77: Well-formatted new Active Directory entry.

The ManagerOfHound entry follows all conventions: proper SO_Icon usage, clear description of the functionality, accurate author/org attribution, and valid repo link. No issues detected.


251-267: Previous URL issue resolved; entry looks good.

The runZeroHound entry correctly links to https://www.runzero.com/ (addressing the prior review's concern about incorrect organization URL). The entry follows all formatting conventions with proper community icon usage, author attribution, and valid repo link. ✅


313-328: Well-formatted new Windows Scheduled Tasks entry.

The TaskHound entry follows community entry conventions: proper icon usage, clear description of functionality, valid author link, and valid repo. The absence of organization affiliation is consistent with other community entries in this catalog. No issues detected.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
docs/opengraph/library.mdx (1)

236-249: Minor: Consider updating heading to match shortened accordion title.

The accordion title was shortened to "Rapid7" (line 236), but the internal heading still reads "OpenGraph DLT (Rapid7 InsightVM)" (line 238). Either approach is valid—keeping the full name provides clarity about which Rapid7 tool this is, while matching the title improves consistency. No action required unless you prefer uniform title/heading naming.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 96a4473 and 2f22b92.

📒 Files selected for processing (1)
  • docs/opengraph/library.mdx (4 hunks)
🔇 Additional comments (3)
docs/opengraph/library.mdx (3)

251-266: ✅ Past review issue resolved.

The runZeroHound author affiliation URL has been correctly updated to https://www.runzero.com/. The fix from the previous review has been applied properly.


60-77: ✅ ManagerOfHound entry looks good.

Properly formatted with clear description, author attribution to SpecterOps, and valid repository link. Includes helpful reference to the demonstration post. Consistent with other SpecterOps entries in the catalog.


313-328: ✅ TaskHound entry is well-structured.

Clear description of the tool's purpose, proper community icon attribution, and valid repository link. The author lacks an organization affiliation, which is appropriate for community contributions. Formatting is consistent with other community entries in the catalog.

Copy link
Contributor

@jeff-matthews jeff-matthews left a comment

Choose a reason for hiding this comment

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

Just a few minor editorial suggestions.


**Description**

ManagerOfHound is an OpenGraph extension for BloodHound that collect manager-subordinate relationships from Active Directory and exports them as custom "ManagerOf" edges for BloodHound ingestion.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ManagerOfHound is an OpenGraph extension for BloodHound that collect manager-subordinate relationships from Active Directory and exports them as custom "ManagerOf" edges for BloodHound ingestion.
ManagerOfHound is an OpenGraph extension for BloodHound that collects manager-subordinate relationships from Active Directory and exports them as custom "ManagerOf" edges for BloodHound ingestion.


ManagerOfHound is an OpenGraph extension for BloodHound that collect manager-subordinate relationships from Active Directory and exports them as custom "ManagerOf" edges for BloodHound ingestion.

Some organizations implement self-service portals where managers can control the user accounts of their subordinates (e.g. password resets). This can create implicit privilege escalation paths not captured by the default BloodHound edges. ManagerOfHound makes these hidden relationships visible through OpenGraph, enabling security teams to identify and assess novel attack paths in their environment.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Some organizations implement self-service portals where managers can control the user accounts of their subordinates (e.g. password resets). This can create implicit privilege escalation paths not captured by the default BloodHound edges. ManagerOfHound makes these hidden relationships visible through OpenGraph, enabling security teams to identify and assess novel attack paths in their environment.
Some organizations implement self-service portals where managers can control the user accounts of their subordinates (e.g., password resets). This can create implicit privilege escalation paths not captured by the default BloodHound edges. ManagerOfHound makes these hidden relationships visible through OpenGraph, enabling security teams to identify and assess novel attack paths in their environment.


Windows Privileged Scheduled Task Discovery Tool for fun and profit.

TaskHound hunts for Windows scheduled tasks that run with privileged accounts and stored credentials. It enumerates tasks over SMB, parses XMLs, and identifies high-value attack opportunities through BloodHound export support.
Copy link
Contributor

Choose a reason for hiding this comment

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

Does "XMLs" refer to "XML files"? If so, I would use the latter as a general doc style convention.

@Scoubi Scoubi changed the title Add 2 Hounds Add 3 Hounds Oct 28, 2025
@Scoubi Scoubi merged commit 3b0ce79 into main Oct 28, 2025
3 checks passed
@Scoubi Scoubi deleted the 2-hounds branch October 28, 2025 21:33
@github-actions github-actions bot locked and limited conversation to collaborators Oct 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants