Skip to content
This repository was archived by the owner on Dec 24, 2022. It is now read-only.

Fixed SetQueryParam to set parameters correctly. #444

Merged
merged 1 commit into from
Apr 7, 2015

Conversation

ristaloff
Copy link

The SetQueryParam method would search for the first occurrence of the param key string. And it would get a positive match even if the param key string was found inside a preceding parameters value.

@ristaloff
Copy link
Author

Just realized that the same problem exists in the SetHashParam method which is not fixed in this commit.

mythz added a commit that referenced this pull request Apr 7, 2015
Fixed SetQueryParam to set parameters correctly.
@mythz mythz merged commit 5a23bd9 into ServiceStack:master Apr 7, 2015
@mythz
Copy link
Member

mythz commented Apr 7, 2015

Good catch, thx!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants