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

Move CreateSession from advanced into basic options #18795

Conversation

cgranleese-r7
Copy link
Contributor

@cgranleese-r7 cgranleese-r7 commented Feb 6, 2024

This PR moves the CreateSession option from advanced into basic options, in order to increase discoverability.
image

I have left have the default as true within the mixin, but added an option within each session types login module to override this options to be false:

  • auxiliary/scanner/smb/smb_login
  • auxiliary/scanner/mssql/mssql_login
  • auxiliary/scanner/postgres/postgres_login
  • auxiliary/scanner/mysql/mysql_login

As part of the ticket I was asked to do data mining to get a list of other modules that made use of this options. So it can be reviewed at module hacking or similar, just to get more input on potentially unexpected issues with this change:

Verification

  • Start msfconsole
  • Use each new session type login module - use scanner/postgres/postgres_login
  • Verify the CreateSession option is now present when running the options command.
  • Verify the CreateSession option is no longer present when running the advanced command.
  • Verify the login modules still function as expected
  • Verify the module in the list of other modules that make use of the CreateSession options still have access to it and it defaults to true

@cgranleese-r7 cgranleese-r7 added the blocked Blocked by one or more additional tasks label Feb 6, 2024
@cgranleese-r7 cgranleese-r7 force-pushed the move-create-session-from-advanced-to-basic-options branch from 490e49e to 2c9fc5c Compare February 9, 2024 15:39
@cgranleese-r7 cgranleese-r7 removed the blocked Blocked by one or more additional tasks label Feb 9, 2024
@cgranleese-r7 cgranleese-r7 marked this pull request as ready for review February 9, 2024 15:40
@cgranleese-r7 cgranleese-r7 force-pushed the move-create-session-from-advanced-to-basic-options branch from 2c9fc5c to 699afaf Compare February 12, 2024 11:37
@cgranleese-r7 cgranleese-r7 changed the title <DRAFT> Move CreateSession from advanced into basic options Move CreateSession from advanced into basic options Feb 12, 2024
@adfoster-r7 adfoster-r7 merged commit b762d2b into rapid7:master Feb 13, 2024
35 checks passed
@cgranleese-r7 cgranleese-r7 deleted the move-create-session-from-advanced-to-basic-options branch February 13, 2024 10:00
@adfoster-r7
Copy link
Contributor

adfoster-r7 commented Feb 13, 2024

Release Notes

Moves the CreateSession option from advanced into basic options for modules, in order to increase discoverability

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

4 participants