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

Urgent: Make response headers optional #2600

Closed
CarolKigoonya opened this issue Feb 19, 2024 · 4 comments · Fixed by #2604
Closed

Urgent: Make response headers optional #2600

CarolKigoonya opened this issue Feb 19, 2024 · 4 comments · Fixed by #2604
Assignees
Labels

Comments

@CarolKigoonya
Copy link

Provide a parameter e.g. -rhv v when you want to access response headers instead of returning 2 objects (which is a breaking change to customers).

@pbo-top
Copy link

pbo-top commented Feb 19, 2024

Sample of breaking-change we have hit since we picked up the 2.14.0 version in automated scripts during this weekend (was working with 2.13.1).

$entApp = Get-MgServicePrincipal -Filter "AppId eq '$($app.application_id)'"

When a given application_id was not found in 2.13.1 the function returned $null, which we were checking for, now the function returns a System.Collections.Hashtable with something like this, aka far from null.

x-ms-ags-diagnostic {"ServerInfo":{"DataCenter":"West Europe","Slice":"E","Ring":"5","ScaleUnit":"002","RoleInstance":"AM2PEPF0000BE1A"}}
OData-Version 4.0
Transfer-Encoding chunked
request-id f89fc6e7-9bea-4c62-a66a-82df45259498
Vary Accept-Encoding
Strict-Transport-Security max-age=31536000
client-request-id fc3fb26e-51ba-4c8e-a83c-30070d92bfc7
Date Mon, 19 Feb 2024 10:27:19 GMT
x-ms-resource-unit 1
Cache-Control no-cache

@skellyatsnhu
Copy link

skellyatsnhu commented Feb 19, 2024

Similarly, this cmdlet returns a pile of blank data (which it should never do) whereas before it would correctly filter things out:

Get-ADUser -Filter 'enabled -eq $true' |ForEach-Object{get-mguser -userid $_.userprincipalname -Property UserPrincipalName,otherMails,CreatedDateTime|Select-Object userprincipalname,OtherMails,CreatedDateTime}|Where-Object{!$_.OtherMails}

Do you have an ETA on a fix?

@BohrenAn
Copy link

Same Issue as documented here:
#2584

@CarolKigoonya
Copy link
Author

We have a new version out to address the issue. https://www.powershellgallery.com/packages/Microsoft.Graph/2.14.1. Apologies to everyone for inconveniences caused on the previous version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants