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

AADGroup members being stripped off for manual group #2481

Closed
anpiskai opened this issue Nov 2, 2022 · 1 comment · Fixed by #2648 or #2667
Closed

AADGroup members being stripped off for manual group #2481

anpiskai opened this issue Nov 2, 2022 · 1 comment · Fixed by #2648 or #2667
Labels
Bug Something isn't working Entra ID

Comments

@anpiskai
Copy link

anpiskai commented Nov 2, 2022

While the changes that fixed issue #2190 (#2190) seem to have resolved the attempted removal of members for dynamic groups, the problem still appears to exist for manual groups where the Members attribute is absent from the resource. Instead, the resource Members attribute is treated as present and blank resulting in DSC trying to remove all members.

These are the attributes specified on the affected resource:
image

and the output from the log (avoiding specifying internal data)
image

@andikrueger andikrueger added Bug Something isn't working Entra ID labels Nov 3, 2022
@andikrueger
Copy link
Collaborator

Thanks for reporting. The code does not check if the parameter is set or not. It treats the missing parameters as a drift and removes the members therefore.

NikCharlebois added a commit to NikCharlebois/Microsoft365DSC that referenced this issue Dec 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Entra ID
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants