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

Optionally test empty group in cisco_ssl_vpn #9999

Merged
merged 2 commits into from May 18, 2018

Conversation

jrobles-r7
Copy link
Contributor

@jrobles-r7 jrobles-r7 commented May 9, 2018

Remove empty group from cisco_ssl_vpn module that's added by default.
Put an advanced option for EmptyGroup in case it is needed

Verification Steps

  • ./msfconsole -q
  • use auxiliary/scanner/http/cisco_ssl_vpn
  • set rhosts <rhosts>
  • set username <username>
  • set password <password>
  • run
  • Verify enumerated groups are used during authentication attempts
  • set emptygroup true
  • run
  • Verify empty group "" is used for authentication attempt

@acammack-r7
Copy link
Contributor

Maybe this should have an option for looking in the empty group?

@jrobles-r7
Copy link
Contributor Author

Updated with EmptyGroup option.

@jmartin-tech
Copy link
Contributor

Jenkins test this please.

@wvu wvu changed the title remove empty group Optionally test empty group in cisco_ssl_vpn May 16, 2018
@busterb busterb self-assigned this May 18, 2018
@busterb busterb merged commit cc0fdee into rapid7:master May 18, 2018
@busterb
Copy link
Member

busterb commented May 30, 2018

Release Notes

This adds the possibility for specifying an empty authentication group with the Cisco SSL VPN authentication scanner module, cisco_ssl_vpn, making it the default value.

@tdoan-r7 tdoan-r7 added the rn-fix release notes fix label Jun 1, 2018
@jrobles-r7 jrobles-r7 deleted the cisco-empty-group branch June 7, 2018 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug module rn-fix release notes fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants