SignInsRequestBuilderGetRequestConfiguration $queryParameters->filter is really doing case sensitive $filter
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 25/100
Rechercherichtung
Beginne mit SignInsRequestBuilderGetRequestConfiguration und dem Anfragebeispiel im Issue und vergleiche anschließend die generierte Abfrage mit der verlinkten Dokumentation zu den OData system-query-options. Überprüfe, ob der Unterschied bei der Groß-/Kleinschreibung im PHP SDK oder in Microsoft Graph entsteht; die Aufgabe ist abgeschlossen, wenn das Verhalten der Sign-in-Suche mit gemischter Groß-/Kleinschreibung korrigiert oder mit einem unterstützten Workaround eindeutig dokumentiert ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Describe the bug
When doing a query for signins for users it is doing a case sensitive search, $filter, instead of the case insensitive search, filter. I got the following from fiddler.
https://graph.microsoft.com/v1.0/auditLogs/signIns?$filter=userPrincipalName%20eq%20%27user%40place.com%27&$top=5
Because of this, results don't come back when they are expected.
per
https://learn.microsoft.com/en-us/odata/concepts/queryoptions-overview#system-query-options
case sensitive is $filter case insensitive is filter.
Here is the code I am using.
$requestConfiguration = new SignInsRequestBuilderGetRequestConfiguration();
$queryParameters = SignInsRequestBuilderGetRequestConfiguration::createQueryParameters();
$queryParameters->filter = "userPrincipalName eq '".$username."'";
$queryParameters->top = 5;
$requestConfiguration->queryParameters = $queryParameters;
If this is by design, how would someone do a case insensitive search? AKA
https://graph.microsoft.com/v1.0/auditLogs/signIns?filter=userPrincipalName%20eq%20%27user%40place.com%27&$top=5
Expected behavior
results to be returned
How to reproduce
mix case on a upn in the https://graph.microsoft.com/v1.0/auditLogs/signIns space
SDK Version
2.11.0
Latest version known to work for scenario above?
No response
Known Workarounds
make sure you match case.
Debug output
Click to expand log
```</details>
### Configuration
_No response_
### Other information
_No response_
- Vorherrschende Sprache
- PHP
- Sterne
- 669
- Forks
- 149
- Ø Merge
- 56 Min.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus microsoftgraph/msgraph-sdk-php
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
microsoftgraph/msgraph-sdk-php#1794 · 2 Kommentare ·
-
bug: metadata type:bug
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
microsoftgraph/msgraph-sdk-php#1739 · 6 Kommentare ·
-
status:waiting-for-triage type:bug
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
microsoftgraph/msgraph-sdk-php#1643 · 8 Kommentare · 4 Reaktionen ·
-
status:waiting-for-triage
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
microsoftgraph/msgraph-sdk-php#1800 · 2 Reaktionen ·
-
NowOffen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 1/100
microsoftgraph/msgraph-sdk-php#1799 ·
Alle Issues in microsoftgraph/msgraph-sdk-php
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Spomky-Labs/pwa-bundle#476 ·
-
domain/discovery
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
neos/flow-development-collection#3639 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
The PHP version check in prepare.php still allows PHP 7.2 and 7.3Evtl. vergeben @ekamran hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
WordPress/phpunit-test-runner#359 ·
Maintainer antworten meist innerhalb von 2 Tagen