Fix left handside indent from exec output

[NeffIsBack]

Description

When retrieving output from .execute() the output is first cleaned .rstrip, which correctly removes trailing spaces/indents:

NetExec/nxc/protocols/smb.py

Line 811 in 6fcfb8d

output = "\n".join([ll.rstrip() for ll in output.splitlines() if ll.strip()])

However, before logging it to the console the output is .strip() again, which now also removed leading spaces/indents. This destroys formatting from the powershell output. As we already stripped it with .rstrip we can just remove the duplicate strip (we also don't need that StringIOstuff).

Also changed the linting pipeline to trigger on code reviews, so that not linted PRs are pushed to main.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Against a Windows Server in Goad

Screenshots (if appropriate):

Before&After example of the Command Get-WindowsFeature, which uses indents to separate categories:

[NeffIsBack]

Also tested the modules which use = connection.execute(command, True) and therefore could be affected by the formatting changes. All looking good

[NeffIsBack]

Also changed the linting pipeline to trigger on code reviews, so that not linted PRs are pushed to main.

With that we can enforce that only linted PRs are merged into main, so that it doesn't slip through like in #450