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

Update Check-SharedMailboxes.PS1 #123

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gioxx
Copy link

@gioxx gioxx commented Sep 30, 2024

I propose a change to filter out false positives related to shared mailboxes that do not actually log in, but third-party login attempts that are unsuccessful and therefore not subject to the mandatory license at all.

I propose a change to filter out false positives related to shared mailboxes that do not actually log in, but third-party login attempts that are unsuccessful and therefore not subject to the mandatory license at all.
@12Knocksinna 12Knocksinna self-requested a review September 30, 2024 21:16
Copy link
Owner

@12Knocksinna 12Knocksinna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you see many false positives?

@gioxx
Copy link
Author

gioxx commented Oct 1, 2024

Ciao @12Knocksinna.
On my corporate tenant, about 15 shared mailboxes out of 534 (just made PowerShell count). I am aware that this is a very low number, but I still have to take it into consideration. And in fact, by manually checking the access logs after receiving the results from your script, I noticed it. By modifying the script, I got the shared mailboxes that needed a dedicated license (and solved a couple of situations).

So again, thank you for your work 😃

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