Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fail on missing path params #50

Closed
jkrems opened this issue Dec 31, 2015 · 1 comment
Closed

Fail on missing path params #50

jkrems opened this issue Dec 31, 2015 · 1 comment

Comments

@jkrems
Copy link
Collaborator

jkrems commented Dec 31, 2015

When /{x}/ placeholders are found but no pathParams.x is provided, the call should fail. The resulting URL won't be valid and it only leads to confusion.
@jkrems jkrems mentioned this issue Apr 15, 2016
Closed
31 tasks
jkrems pushed a commit that referenced this issue Apr 18, 2016
fix: Verify path params
8c84f46 
Fixes #50
@jkrems
Copy link
Collaborator Author

jkrems commented Nov 18, 2016

This was implemented in gofer 3.

@jkrems jkrems closed this as completed Nov 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jkrems