Set fail_fast to true by default
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 35/100
Research direction
Start by locating the fail_fast option and the validation tests that define its current behavior. Review how the default is configured and how multiple invalid parameters are reported. Done means the intended default behavior is agreed, implemented consistently, and covered by tests without breaking existing API expectations.
Written by the indexing model from the issue text.
Description
I would like to set this option to true for my entire API. Before I create a PR with such an ability, I would like to discuss it. In my opinion, when a parameter is considered invalid, there is no sense in running another validators, because:
- request will not be processed normally anyway
- we spent time and cpu on useless validation
So, I think we should change behavior of validation to stop after first invalid parameter.
- Dominant language
- Ruby
- Stars
- 10k
- Forks
- 1.2k
- Avg merge
- 14h 38m
- Merged PRs (30d)
- 92
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ruby-grape/grape
-
feature request you can help
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
ruby-grape/grape#2487 · 2 comments · 2 reactions ·
-
chore
Difficulty 2/5 1-3 hours Newbie friendliness 30/100
ruby-grape/grape#2562 · 4 comments ·
-
feature request
Difficulty 5/5 Over a week Newbie friendliness 25/100
ruby-grape/grape#2522 · 3 comments ·
-
bug?
Difficulty 4/5 3-5 days Newbie friendliness 48/100
ruby-grape/grape#2473 · 6 comments · 1 reaction ·
-
chore discuss!
Difficulty 5/5 Over a week Newbie friendliness 25/100
ruby-grape/grape#2446 · 7 comments · 1 reaction ·
All issues in ruby-grape/grape
Similar issues
-
バグ
Difficulty 1/5 Under an hour Newbie friendliness 92/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
voxpupuli/puppet-epel#186 · 1 comment ·
-
external_created_at is no longer used for the message timestamp since the new message UI (v4.4.0) OpenBug Frontend
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
TheOdinProject/curriculum#31402 · 1 comment ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100