-
Notifications
You must be signed in to change notification settings - Fork 75
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
"getQueryParamValue" is case sensitive #57
Comments
What sort of environment are you working in that has case-insensitive URL params? |
This is an issue for me as well... I have code like this, looking for 'iscloud' querystring: Now users of my website have to get the casing right or my code doesn't find the param. Right now, it's a bug and I have to tell these particular users to use lower case please. |
@derek-watson Would you accept pull request for this? I'm thinking adding a boolean argument will make sense. |
@alFReD-NSH do you have a fork with the necessary change... I would like to go there. |
@derek-watson , I made a pull request #63. I appreciate if you could have a look and tell me what you think. |
Could
getQueryParamValue
ignore case sensitive? So these will return the same result:The text was updated successfully, but these errors were encountered: