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

add necessary metadata for bruteforce #19352

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

zgoldman-r7
Copy link
Contributor

@zgoldman-r7 zgoldman-r7 commented Jul 30, 2024

This PR adjusts the metadata for the ldap login scanner, adding defaults and adjusting the service and protocol values.

To test:

  • spin up docker or windows instance with LDAP running (usable docker env details in test/ldap)
  • use ldap_login module against target and ensure functionality still works

@@ -11,19 +11,32 @@ class LDAP
include Metasploit::Framework::LDAP::Client
include Msf::Exploit::Remote::LDAP

# TODO: Verify if we need this additional metadata: https://github.com/rapid7/metasploit-framework/blob/master/lib/metasploit/framework/login_scanner/http.rb#L15-L23
Copy link
Contributor

Choose a reason for hiding this comment

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

We'll need to make sure that the ldaps functionality is implemented correctly here, similar to the above link

@zgoldman-r7 zgoldman-r7 force-pushed the adjust-ldap-scanner branch from 12d2e31 to 29bfc1c Compare August 6, 2024 15:15
@zgoldman-r7 zgoldman-r7 marked this pull request as ready for review August 6, 2024 15:22
@cgranleese-r7 cgranleese-r7 self-assigned this Sep 3, 2024
@cgranleese-r7
Copy link
Contributor

Tested and works as expected:
image

@cgranleese-r7 cgranleese-r7 added the rn-enhancement release notes enhancement label Sep 6, 2024
@cgranleese-r7 cgranleese-r7 merged commit 8e94a0d into rapid7:master Sep 6, 2024
39 checks passed
@cgranleese-r7
Copy link
Contributor

Release Notes

Adjusts the metadata for the ldap login scanner, adding defaults and adjusting the service and protocol values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rn-enhancement release notes enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants