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

EXOAuthenticationPolicy: Recreate policy instead of changing it #4820

Merged
merged 4 commits into from
Oct 2, 2024

Conversation

ricmestre
Copy link
Contributor

Pull Request (PR) description

Unfortunately trying to change any existing EXOAuthenticationPolicy fails with the error message below, this is a problem in cmdlet ````Set-AuthenticationPolicy``` and the way to fix this is to just remove the current policy and recreate it with the new settings.

Object reference not set to an instance of an object.

This Pull Request (PR) fixes the following issues

@NikCharlebois
Copy link
Collaborator

Looping in PG folks responsible for the cmdlet.

//cc @Swatyario for visibility

@ricmestre
Copy link
Contributor Author

By the way, other people already reported this issue, see https://techcommunity.microsoft.com/t5/windows-powershell/exchange-online-error-write-errormessage-object-reference-not/m-p/4127852

@NikCharlebois
Copy link
Collaborator

PG is aware of the issue and has it on their backlog. I recommend parking this until it's fixed since deleting and recreating will generate unnecessary entries in the audits.

@ricmestre
Copy link
Contributor Author

Sure, just note that there are other EXO resources where this is also done, not sure if the Set cmdlets have issues or if don't exist though. I'll exclude this resource on our tool for now on deployments.

@FabienTschanz
Copy link
Contributor

@NikCharlebois Any news on the situation? Is the PG still working on it or does it work now?

@NikCharlebois
Copy link
Collaborator

Haven't heard any updates from the team. Will go ahead and merge your proposed PR and will continue to track PG's response.

@ricmestre
Copy link
Contributor Author

ricmestre commented Oct 2, 2024

OK it's not the best solution but at least works, let me check the conflict and I'll update the branch

EDIT: You were faster than me thank you for checking the conflict :)

@NikCharlebois NikCharlebois merged commit 60af821 into microsoft:Dev Oct 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EXOAuthenticationPolicy: Cannot change resources
3 participants