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

Missing params to null #71

Closed
alex-schneider opened this issue Dec 2, 2020 · 0 comments · Fixed by #78
Closed

Missing params to null #71

alex-schneider opened this issue Dec 2, 2020 · 0 comments · Fixed by #78
Assignees
@malud
Labels
bug Something isn't working
Milestone
0.3

Comments

@alex-schneider
Copy link
Contributor

alex-schneider commented Dec 2, 2020 •
edited
Loading

Currently, if I define my origin like origin = "http://${req.path_param.origin}" and the {origin} is not defined in the path of my endpoint, couper panics.

We should convert missing params to null. The same goes for JsonBody, Query, Post, etc.
@alex-schneider alex-schneider changed the title Missing params to nil Missing params to null Dec 2, 2020
@malud malud added the bug Something isn't working label Dec 4, 2020
@malud malud added this to the 0.3 milestone Dec 4, 2020
@malud malud self-assigned this Dec 8, 2020
malud pushed a commit that referenced this issue Dec 8, 2020
Fix handling cty null or unknown values during roundtrip eval #71
887baed
@malud malud mentioned this issue Dec 8, 2020
Merged
malud pushed a commit that referenced this issue Dec 8, 2020
Fix handling cty null or unknown values during roundtrip eval #71
c1d9927
alex-schneider pushed a commit that referenced this issue Dec 8, 2020
Merge pull request #78 from avenga/null-eval
84d8534 
Fix handling cty null or unknown values during roundtrip eval #71
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@malud malud

Labels
bug Something isn't working
Projects
None yet
Milestone
0.3
Development

Successfully merging a pull request may close this issue.

Fix handling cty null or unknown values during roundtrip eval #71 coupergateway/couper
2 participants
@malud @alex-schneider