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

IntuneDeviceEnrollmentStatusPageWindows10: Return all authentication methods #4697

Merged
merged 1 commit into from
May 22, 2024

Conversation

ricmestre
Copy link
Contributor

@ricmestre ricmestre commented May 22, 2024

Pull Request (PR) description

IntuneDeviceEnrollmentStatusPageWindows10 doesn't return all authentication methods on Get nor TenantId, currently only AccessTokens is returned. This may fail deducing the OrganizationName via TenantId, which is null, just like it's currently happening to me. The fix is to just return all authentication methods on Get.

The error message taken from the event log is the following:

{ You cannot call a method on a null-valued expression. } \ at Get-M365DSCExportContentForResource, C:\Program Files\WindowsPowerShell\Modules\Microsoft365DSC\1.24.515.2\Modules\M365DSCUtil.psm1: line 3554
 \ at Export-TargetResource, C:\Program Files\WindowsPowerShell\Modules\Microsoft365DSC\1.24.515.2\DSCResources\MSFT_IntuneDeviceEnrollmentStatusPageWindows10\MSFT_IntuneDeviceEnrollmentStatusPageWindows10.psm1: line 778
 \ at Start-M365DSCConfigurationExtract, C:\Program Files\WindowsPowerShell\Modules\Microsoft365DSC\1.24.515.2\Modules\M365DSCReverse.psm1: line 677
 \ at Export-M365DSCConfiguration, C:\Program Files\WindowsPowerShell\Modules\Microsoft365DSC\1.24.515.2\Modules\M365DSCUtil.psm1: line 1375
 \ at <ScriptBlock>, D:\a\1\s\Source\Scripts\TestADOConnectivity.ps1: line 124
 \ at <ScriptBlock>, D:\a\_temp\169d254d-e56a-41a6-9741-a145801e34b3.ps1: line 37
 \ at <ScriptBlock>, <No file>: line 1

This Pull Request (PR) fixes the following issues

@NikCharlebois NikCharlebois merged commit 33cb392 into microsoft:Dev May 22, 2024
2 checks passed
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.

None yet

2 participants