Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ADCS/PKI - Merge ly4k/BloodHound with main branch #684

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

1mm0rt41PC
Copy link

Just a simple PR with the code developed by @ly4k.
I've cleaned up the code to ignore white spaces and code design changes.
The PR contains:

  • Support for new node type: CertificateTemplate, CA
  • Support for new relationship: Enroll, AutoEnroll, ManageCa, ManageCertificates, EnabledBy
  • Extend the query field to a textarea
  • Support for the collector Certipy
  • Add pre-build queries for ADCS
  • Add support to custom icon via node attribute bl-icon

@github-actions
Copy link

github-actions bot commented Jul 7, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@1mm0rt41PC
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@1mm0rt41PC
Copy link
Author

Recheck

@probablypablito
Copy link

This would be great! +1

@e99ran
Copy link

e99ran commented Aug 13, 2023

It would be awesome to see this in BloodHound!

@AkechiShiro
Copy link

I'd agree that this would be nice, but I believe a new version of BloodHound has been recently released, this one is the old one, the new one is here, I'm not sure if those changes can make it to the new one without an adaptation : https://github.com/SpecterOps/BloodHound

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.

4 participants