canix1 / ADACLScanner

Repo for ADACLScan.ps1 - Your number one script for ACL's in Active Directory
Microsoft Public License
958 stars 162 forks source link

Extra hidden characters in code #44

Open mczepieltpc opened 3 years ago

mczepieltpc commented 3 years ago

I just pulled down the new 6.8 version and was comparing to v6.4. Running the script produced quite a few errors too. I noticed the following: line 1266, 1267, 1271, 15217, 15218, 15222, 15600, 15601, 15602, 15605 - show hidden character line 3042, 3063, 11846, 11860, 15054 - quotes are showing as funny character line 14254 - hidden character Sorry, I can't attach a file to visually show for you

canix1 commented 3 years ago

What version of powershell are you using?

mczepieltpc commented 3 years ago

5.1 on Server 2016. Once I removed the funky characters, it all ran great. Just wanted to let you know in case.

Thanks for the great tool.

Mike

On Wed, Nov 10, 2021, 5:33 PM Robin Granberg @.***> wrote:

What version of powershell are you using?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/canix1/ADACLScanner/issues/44#issuecomment-965807287, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGCH7O2RGL5GHHT3QEJLOH3ULLXMJANCNFSM5HXW2VTA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

canix1 commented 3 years ago

You're welcome ! Sorry, I cannot reproduce the errors or find the funny characters in 6.8. Tried it on Win Server 2016 and PS 5.1. No error. Could you share a screenshot of the errors or explain what you had to remove?

mczepieltpc commented 3 years ago

Robin - The attached is what I was seeing when running BeyondCompare looking at the differences between the two versions.

Mike

On Thu, Nov 11, 2021 at 6:36 AM Robin Granberg @.***> wrote:

You're welcome ! Sorry, I cannot reproduce the errors or find the funny characters in 6.8. Tried it on Win Server 2016 and PS 5.1. No error. Could you share a screenshot of the errors or explain what you had to remove?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/canix1/ADACLScanner/issues/44#issuecomment-966231300, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGCH7O4O4DF65ERBHSXWJ33ULOTD3ANCNFSM5HXW2VTA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.