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

Add powershell help and log file #2

Merged
merged 3 commits into from
Sep 26, 2024

Conversation

anotherbridge
Copy link
Contributor

A PowerShell help is added for the script so it is possible to retrieve information on the usage via the Get-Help cmdlet. Further, there were a couple of usages of the Out-Null cmdlet which hide information which can useful for troubleshooting as well as auditing purposes.
This information is now logged to a file.

@simeononsecurity
Copy link
Owner

In the future, please be mindful of submitting changes that could cause merge conflicts, especially for simple updates. However, I am approving this PR based on the following improvements:

  • PowerShell help documentation has been added, allowing users to access usage information through the Get-Help cmdlet.
  • Several instances of the Out-Null cmdlet were removed, as they were hiding potentially useful information for troubleshooting and auditing. This information is now being logged to a file, enhancing visibility and traceability.

Thank you for your contributions.

@simeononsecurity simeononsecurity merged commit 3a056dc into simeononsecurity:master Sep 26, 2024
1 check failed
@simeononsecurity simeononsecurity self-assigned this Sep 26, 2024
@simeononsecurity simeononsecurity self-requested a review September 26, 2024 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants