Skip to content

return 400 result instead of 500 on invalid input#2094

Merged
Erwinvandervalk merged 1 commit intoreleases/bff/3.1.0from
ev/bff/400_result
Jul 14, 2025
Merged

return 400 result instead of 500 on invalid input#2094
Erwinvandervalk merged 1 commit intoreleases/bff/3.1.0from
ev/bff/400_result

Conversation

@Erwinvandervalk
Copy link
Contributor

What issue does this PR address?
This PR is stacked on previous pr's.

Makes sure the BFF returns a 400 error instead of a 500 error when receiving invalid input.

Important: Any code or remarks in your Pull Request are under the following terms:

If You provide us with any comments, bug reports, feedback, enhancements, or modifications proposed or suggested by You for the Software, such Feedback is provided on a non-confidential basis (notwithstanding any notice to the contrary You may include in any accompanying communication), and Licensor shall have the right to use such Feedback at its discretion, including, but not limited to the incorporation of such suggested changes into the Software. You hereby grant Licensor a perpetual, irrevocable, transferable, sublicensable, nonexclusive license under all rights necessary to incorporate and use your Feedback for any purpose, including to make and sell any products and services.

(see our license, section 7)

@Erwinvandervalk Erwinvandervalk self-assigned this Jul 7, 2025
@Erwinvandervalk Erwinvandervalk added this to the bff-3.1 milestone Jul 7, 2025
@Erwinvandervalk Erwinvandervalk requested a review from damianh July 7, 2025 06:09
@Erwinvandervalk Erwinvandervalk force-pushed the ev/bff/400_result branch 2 times, most recently from 68cc3fe to 430f880 Compare July 8, 2025 06:10
@Erwinvandervalk Erwinvandervalk changed the title Ev/bff/400 result return 400 result instead of 500 on invalid input Jul 11, 2025
@Erwinvandervalk Erwinvandervalk merged commit 93f4212 into releases/bff/3.1.0 Jul 14, 2025
6 checks passed
@Erwinvandervalk Erwinvandervalk deleted the ev/bff/400_result branch July 14, 2025 14:25
pgermishuys pushed a commit that referenced this pull request Sep 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants