Justin Grote

Results 875 comments of Justin Grote

Like I said you can work around it by instantiating acd_cli in parallel, but I've seen nothing that says this is safe to do yet and saw where developer recommended...

I was doing some profiling on this and a couple of the major ones are taking the dependency on the Configuration Module and the Add-Type for EmptyStringAsNullAttribute. The add-type adds...

Using Fiddler, I did the exact same query with Graph Explorer (which works) and the module (which returns all results, note the body size is smaller). The URL are identical...

Here are the header differences, going to start stepping through them with Invoke-Restmethod until I find which one is causing the issue ![image](https://user-images.githubusercontent.com/15258962/105555490-5a180f80-5cbe-11eb-919c-0f04bd12b1e3.png)

Found it! The `ConsistencyLevel: Eventual` header is missing which is required for the advanced queries. https://developer.microsoft.com/en-us/identity/blogs/microsoft-graph-advanced-queries-for-directory-objects-are-now-generally-available/

Also referenced here in the OpenApi Spec: https://github.com/microsoftgraph/msgraph-sdk-powershell/blob/7399ffc1e490a7cfca01392f2f74aad8b488f047/openApiDocs/v1.0/Applications.yml#L16-L25

@peombwa yeah I've expressed that however since this is now GA functionality in the graph, it should be done automatically for these advanced search parameters, especially since it doesn't warn...

Thanks, I should also note that you basically cannot use these parameters at all anyways unless you specify it, so it is effectively not optional, and is described as such...

Thanks for the response @Licantrop0 > production release for this is few quarters away. So consider for your module users that they are using the switches like `-Search`, `-ExpandProperty`, `-OrderBy`,...

@darrelmiller I then *strongly* suggest that you add a check if -Search, -ExpandProperty, -OrderBy, -Count are in use and eventualconsistency is not specified, that you throw an error or at...