Skip to content

Allow proto flags in combination with modules#1202

Merged
NeffIsBack merged 1 commit intomainfrom
neff-allow-commands-and-modules
Apr 14, 2026
Merged

Allow proto flags in combination with modules#1202
NeffIsBack merged 1 commit intomainfrom
neff-allow-commands-and-modules

Conversation

@NeffIsBack
Copy link
Copy Markdown
Member

Description

I don't really see a reason (anymore) to forbid the combination of flags (e.g. --dpapi) and modules (e.g. -M lsassy). As other PRs (#1067) needs this as well I just removed that restriction.

Type of change

Insert an "x" inside the brackets for relevant items (do not delete options)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Deprecation of feature or functionality
  • This change requires a documentation update
  • This requires a third party update (such as Impacket, Dploot, lsassy, etc)
  • This PR was created with the assistance of AI (list what type of assistance, tool(s)/model(s) in the description)

Setup guide for the review

nxc smb ... --dpapi -M lsassy

Screenshots (if appropriate):

image

Checklist:

@NeffIsBack NeffIsBack added the enhancement New feature or request label Apr 14, 2026
@NeffIsBack NeffIsBack merged commit 1884cd3 into main Apr 14, 2026
11 checks passed
@NeffIsBack NeffIsBack deleted the neff-allow-commands-and-modules branch April 14, 2026 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants