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

Adds check for HTTPS and Channel Binding aka EPA for ESC8 #254

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sploutchy
Copy link
Contributor

@sploutchy sploutchy commented Feb 12, 2025

Improves on the pull request 203 to check for presence of HTTPS web enrollment endpoint and channel binding requirement.

Requires a modern version of impacket, e.g.:

pipx install git+https://github.com/sploutchy/Certipy@esc8_https_channel_binding --suffix custom
pipx inject certipy-adcustom git+https://github.com/fortra/impacket --force

Outputs if HTTP is enabled and/or HTTPS is enabled and channel binding not required:

└─$ certipy find -target-ip 10.0.1.100 -target child.testlab.local -dc-ip 10.0.1.100 -ns 10.0.1.100 -username [email protected] -password 'oops' -stdout
Certipy v4.8.2 - by Oliver Lyak (ly4k)
[CUT]
[+] Resolved 'CA1.child.testlab.local' from cache: 10.0.1.102
[+] Connecting to http://10.0.1.102/certsrv
[+] Resolved 'CA1.child.testlab.local' from cache: 10.0.1.102
[+] Connecting to https://10.0.1.102/certsrv
[+] Web enrollment seems enabled over https
[+] Resolved 'CA1.child.testlab.local' from cache: 10.0.1.102
[+] Testing Channel Binding. Performing connection to CA1.child.testlab.local without CB
[+] 301
[*] Channel Binding not enforced for CA1.child.testlab.local
[CUT]
      ESC8                              : Web Enrollment is enabled over HTTPS, Channel Binding is disabled and Request Disposition is set to Issue
[CUT]

This patch should be compatible with https://github.com/zimedev/certipy-merged

@sploutchy
Copy link
Contributor Author

Note: relaying to HTTPS is possible with impacket's ntlmrelayx.py as per this comment.

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.

1 participant