search

OfficeDev / O365-InvestigationTooling

MIT License
574 stars 360 forks source link

Update InactiveUsersLast90Days.ps1 #60

Closed EricVanAken closed 5 years ago

EricVanAken commented 5 years ago

Changed the code to support MFA sign-in

msftclas commented 5 years ago

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

:x: EricVanAken sign now
You have signed the CLA already but the status is still pending? Let us recheck it.

PsychoData commented 5 years ago

This will break it's functionality for non-MFA people who haven't installed the Exchange Online Hybrid .Application Also, we should make sure that we're using the latest version of the MFA Capable DLL.

When you run the Hybrid Shell through the start-menu, it auto-updates and it's fairly common to have 3,4, or more versions of the Exchange Online DLL in there, we should use the latest version.

Preferably with something to hook into the .Application or .appref-ms to check to see if there is an updated version available to use