[SMB] Allow force to disable SMBv1

[XiaoliChan]

Description

Allow force to disable smbv1 in smb connection

Type of change

Please delete options that are not relevant.

• [√] Bug fix (non-breaking change which fixes an issue)
• [√] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

nxc smb 127.0.0.1 -u a -p a --no-smbv1 --shares

Screenshots (if appropriate):

Checklist:

• [√] I have ran Ruff against my changes (via poetry: poetry run python -m ruff check . --preview, use --fix to automatically fix what it can)
• [√] I have added or updated the tests/e2e_commands.txt file if necessary
• [√] New and existing e2e tests pass locally with my changes
• [√] My code follows the style guidelines of this project (should be covered by Ruff above)
• [×] If reliant on third party dependencies, such as Impacket, dploot, lsassy, etc, I have linked the relevant PRs in those projects
• [√] I have performed a self-review of my own code
• [√] I have commented my code, particularly in hard-to-understand areas
• [×] I have made corresponding changes to the documentation (PR here: https://github.com/Pennyw0rth/NetExec-Wiki)

[mpgn]

Smbv3 you mean no ?

[NeffIsBack]

For what would you need it though?

[mpgn]

For what would you need it though?

could solve this kind of issue

#217

[NeffIsBack]

For what would you need it though?

could solve this kind of issue

#217

Fair enough, sounds good

[XiaoliChan]

Smbv3 you mean no ?

Yes, so what should we call to the new args, or --no-smbv1?

[XiaoliChan]

Test again

[XiaoliChan]

Now it won't break the smbv1 info in host info output

[mpgn]

Added my comments, if a user decide to use the --no-smbv1 then is normal to return None on the smbv1 info since the user don't want to check smbv1 (can be related to other thing than the shares :)
The big advantage: you don't have to change the logic from @NeffIsBack on the create_conn_obj method, just add the line no_smbv1 = self.args.no_smbv1 if self.args.no_smbv1 else no_smbv1 :)

[XiaoliChan]

Added my comments, if a user decide to use the --no-smbv1 then is normal to return None on the smbv1 info since the user don't want to check smbv1 (can be related to other thing than the shares :)

Got you

The big advantage: you don't have to change the logic from @NeffIsBack on the create_conn_obj method, just add the line no_smbv1 = self.args.no_smbv1 if self.args.no_smbv1 else no_smbv1 :)

OK, just reverted that changed