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

enum_proxy: Cleanup and support non-Meterpreter sessions #17332

Merged
merged 1 commit into from
Dec 5, 2022

Conversation

bcoles
Copy link
Contributor

@bcoles bcoles commented Dec 4, 2022

  • Resolves Rubocop violations.
  • Adds documentation.
  • Adds Notes module meta information.
  • Adds support for non-Meterpreter sessions.

With a few small changes, this module now works for local sessions on non-Meterpreter sessions (shell, powershell).

Support for non-Meterpreter session with the Registry library has been improved recently. Unfortunately the Registry libraries do not support reading from a remote registry on non-Meterpreter sessions. For non-Meterpreter sessions, this module will fail with BadConfig if rhost is set. Given that this is a post/gather module I expect that gathering from the local system is the primary expected use case. I'm not too concerned about remote registry functionality.

For what it's worth, the remote registry functionality didn't work for me (both before and after this change). Possibly user error, but also outside my use case. Unless I've overlooked something silly, the changes in this module should preserve the existing logic for remote registry functionality.

@adfoster-r7 adfoster-r7 merged commit a4aff65 into rapid7:master Dec 5, 2022
@adfoster-r7
Copy link
Contributor

Release Notes

Update windows/gather/enum_proxy to support non-Meterpreter sessions (shell, powershell)

@adfoster-r7 adfoster-r7 added the rn-enhancement release notes enhancement label Dec 5, 2022
@bcoles bcoles deleted the enum_proxy branch December 5, 2022 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants