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

Passing query string values #28

Closed
DamianMehers opened this Issue Jan 30, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@DamianMehers
Contributor

DamianMehers commented Jan 30, 2016

I've found I can pass query string values by escaping the question mark:

    [UriFormat("/GetTemperatures\\?start={start}&end={end}")]
    public GetResponse GetTemperatures(string start, string end) {
      Debug.WriteLine("Received request start " + start + " end " + end);
      return new GetResponse(GetResponse.ResponseStatus.OK);
    }

Would you like a pull request to escape it in code?

@tomkuijsten

This comment has been minimized.

Show comment
Hide comment
@tomkuijsten

tomkuijsten Jan 31, 2016

Owner

That's an interesting approach ☺. Pull request would be nice.

Owner

tomkuijsten commented Jan 31, 2016

That's an interesting approach ☺. Pull request would be nice.

@DamianMehers

This comment has been minimized.

Show comment
Hide comment
@DamianMehers
Contributor

DamianMehers commented Jan 31, 2016

@tomkuijsten

This comment has been minimized.

Show comment
Hide comment
@tomkuijsten

tomkuijsten Feb 8, 2016

Owner

Thanks. I've just been sick a couple of days, I'll merge this soon.

Owner

tomkuijsten commented Feb 8, 2016

Thanks. I've just been sick a couple of days, I'll merge this soon.

@tomkuijsten tomkuijsten added the feature label Feb 8, 2016

@tomkuijsten tomkuijsten self-assigned this Feb 8, 2016

@tomkuijsten

This comment has been minimized.

Show comment
Hide comment
@tomkuijsten

tomkuijsten Feb 11, 2016

Owner

Merged you're request. Might need some more rework to fully support all query parameter options.

Owner

tomkuijsten commented Feb 11, 2016

Merged you're request. Might need some more rework to fully support all query parameter options.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment