[bug] DP validation exception requesting more than 7 day window

[braddf]

Describe the bug

If you request a window of more than 7 days, the API returns a 500 Internal Server Error due to a lack of validation on the input for this, and passing straight through to the DP.

Expected behavior

Should validate user input, and return a 422 response if doesn't match allowed input rules

[CodeVishal-17]

Hi, I’d like to work on this issue.

From what I understand, the API currently allows requests with a date window greater than 7 days, which leads to a 500 error from the downstream DP due to missing validation. I plan to add input validation at the API level to enforce the 7-day limit and return a 422 response for invalid requests.

I’ll also include test cases to cover valid, invalid, and edge scenarios.

Please let me know if this approach aligns with your expectations before I proceed. 🙂

[ocf-bot]

@CodeVishal-17 Thank you for reaching out! Unfortunately, this issue is not suitable for open source contribution and so hasn't been marked with "good first issue" or "contributions welcome". Please consult our contribution guidelines for more information on how you can contribute, and hope to see you around!

[peterdudfield]

We now do offer this for the national forecast route.
Would need to check other routes if this still happens

[CrazyTodd-one]

@peterdudfield I can help check the other routes. Which backend are the non-national routes using — still DP with the 7-day limit, or have they been migrated too? Happy to add validation for whichever routes still need it.

[ocf-bot]

@CrazyTodd-one Thank you for reaching out! Unfortunately, this issue is not suitable for open source contribution and so hasn't been marked with "good first issue" or "contributions welcome". Please consult our contribution guidelines for more information on how you can contribute, and hope to see you around!