Alex

To "1.": - `subsubfolder3` might be skipped because it is empty? However imo when we download a folder i would expect to fully have its structure downloaded - `subsubfolder2` only...

> Yeah it doesn't download empty folders because... what's the point? I guess I can add that in if you think it's a good idea... What's the use of having...

> Okay, I can update it. Do you think doing something like displaying [EMPTY] when downloading it would be helpful? My concern is people downloading a big file structure that's...

> There is still something weird going on:  > > 1. It doesn't download all folders in the folder Basically i don't want users to experience the same confusion...

@Marshall-Hallenbeck there is still the bug with the forward slashes and therefore adding the remote path to the local structure: Possibly missing path sanitation before creating the local folder structure?...

Maybe the 3rd option would even be the best as there won't be an AD with a disabled DC machine account, but there will definitely (or hopefully) ones with the...

Nice one, thanks for the PR!

I have changed stuff a bit: - name `sccmrecon`->`sccm-recon6` because that more accurately represents the technique: https://github.com/subat0mik/Misconfiguration-Manager/blob/main/attack-techniques/RECON/RECON-6/recon-6_description.md - Muted the output if we can't query the SMS, because this will...

> Hi! thx for the improvement, should the name be sccm_recon6? Honestly i am not a huuuge fan of underscores instead of the single dash, but if you prefer that...

@Mauriceter if the changes look good to you we are ready for merging