-
Notifications
You must be signed in to change notification settings - Fork 47
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
Get-CosmosDbDocument -QueryParameters hashtable array is case sensitive #446
Comments
Ok, so this cmdlet just passes through the hashtable directly as seen here:
Here is a proof of concept for what would need done to translate parameters to always be lower case:
I'd expect the performance hit to be quite minimal if implemented. Alternatively, it could be nice to check if the hashtable array contains any name or value keys that aren't fully lower case and error if so. Thoughts on what direction you'd like to go? |
Hi @Szeraax - ah yes, I see what you mean. It doesn't specifically call this out in the docs (https://docs.microsoft.com/en-us/rest/api/cosmos-db/query-documents), but makes sense. The fix makes sense, but just would need slight style adjustments. What I'd also suggest is moving the whole code block to a helper function that we can put unit tests on (just makes testing easier). Happy to do this or take a PR if you want? |
I have to do lower case name and value keys in each hashtable.
If I do upper case NAME, I get the error:
If I do upper case VALUE, I get no results.
Will start digging into code shortly.
The text was updated successfully, but these errors were encountered: