Skip to content

Merge dev updates and refresh ScriptSentry - #28

Merged
techspence merged 12 commits into
mainfrom
2026-update
Sep 17, 2026
Merged

techspence merged 12 commits into
mainfrom
2026-update

Conversation

@techspence

Copy link
Copy Markdown
Owner

Summary

  • merge the existing dev work with the current main branch
  • add LDAP credentials and explicit domain controller targeting
  • keep the refactored findings and consolidated CSV output
  • fix SYSVOL discovery, mapped path extraction, and false negatives found during DC01 testing
  • update usage documentation and example output

Validation

  • PowerShell parsing passed
  • default discovery tested on DC01
  • -Server DC01.dunder.corp -Domain dunder.corp tested on DC01
  • staged unsafe ACLs, plaintext credentials, UNC paths, GPO scripts, and nonexistent shares were detected

-Credential applies to LDAP queries. UNC and SMB access continues to use the PowerShell process identity.

Closes #20
Addresses #23

techspence and others added 12 commits November 10, 2024 13:01
Minor fix for PSSA: $null should be on the left side of equality comparisons.
"If a command parameter takes its value from the pipeline, the command must use a process block to bind the input objects from the pipeline to that parameter."
Fix: add missing process block
Fix $null check position
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.

Is the script supposed to handle authentication?

2 participants