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

ESC8 Identification is Incomplete #125

Open
Adonist opened this issue Mar 5, 2024 · 7 comments
Open

ESC8 Identification is Incomplete #125

Adonist opened this issue Mar 5, 2024 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@Adonist
Copy link

Adonist commented Mar 5, 2024

Hi,

It seems that ESC8 identification is not accurate.
In my case I can confirm web enrollment is not installed and Windows authentication for CEP and CES is set to: Negoriate:Kerberos and Extended Protection is Required.
Still, running Locksmith comes up with "HTTP enrollment is enabled."

image
image
image

@Adonist
Copy link
Author

Adonist commented Mar 5, 2024

Forgot to mention that the SSL Settings is also set to Require SSL
image

@TrimarcJake
Copy link
Owner

I've confirmed this! And I see what's causing it. Dang regex.

@TrimarcJake TrimarcJake self-assigned this Mar 5, 2024
@TrimarcJake TrimarcJake added the bug Something isn't working label Mar 5, 2024
TrimarcJake added a commit that referenced this issue Mar 5, 2024
@TrimarcJake
Copy link
Owner

Hi, @Adonist

I just pushed changes to the fix-esc8-125 branch. Please test it and let me know if it works for you! (If you're unsure how to test with a specific branch, feel free to reach out for assistance.)

@Adonist
Copy link
Author

Adonist commented Mar 5, 2024

I'll check it tomorrow and let you know. Thank you for the quick action on this!
Also, I see that the Fix for the issue for ESC8 is showing up as [TODO].
I would be happy to assist with remediation steps to fix things based on the findings.

Thanks

@TrimarcJake
Copy link
Owner

Thank you for the quick action on this!

No, thank you for testing and reporting! It helps to get reports from real users. Lab tests only go so far.

I would be happy to assist with remediation steps to fix things based on the findings.

I would love that. Feel free to submit a PR, or we can chat about it in a different space. Totally up to you.

@Adonist
Copy link
Author

Adonist commented Mar 8, 2024

Apologies for the delay in testing it and getting back to you.
I've tested now and it picks up HTTPS which is good. However, this is not necessary an issue. Enforcing HTTPS is one of the mitigations for ESC8.
image

I think for ESC8 it should consider:

  • Is web enrollment installed ?
  • Is EPA required ?
  • Is SSL required ?
  • Is windows authentication set to Kerberos ?
  • Is NTLM enabled ?

If using HTTPS, Kerberos AUthentication and EPA for CEP and CES, it shouldn't be vulnerable to ESC8 essentially.

The regex is picking up correctly though.

@TrimarcJake
Copy link
Owner

First: thank you for confirming the change now properly identifies the HTTPS endpoints as HTTPS!

As for the rest of your comment: we generally approach things from a defender's standpoint. The mitigations you listed reduce or eliminate the risk of NTLM relay if fully implemented, but in our opinion, the best solution is eliminating the endpoint altogether.

That's why this is a VERY basic check. If you find an HTTPS enrollment endpoint, it's low risk, but it's still a risk.

But we are aware of the limitations of this approach and have started discussing methods for improving this test. Assessing the actual risk of the HTTPS finding is simply impossible with the data we gather at the moment, so step one will be gathering additional data.

If you'd like to help build this check, we'd gladly accept a PR! You can even post PoC code in this Issue, and I'll keep it open until you're satisfied. :D

TrimarcJake added a commit that referenced this issue Mar 9, 2024
@TrimarcJake TrimarcJake changed the title ESC8 Identification not Accurate ESC8 Identification is Incomplete Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants