Conversation
andig
marked this pull request as ready for review
September 5, 2026 16:53
andig
added a commit
that referenced
this pull request
Sep 5, 2026
Production rejects a steady eight to twelve requests an hour with "All time series must have the same length" and the log could not say which series was short, so the client bug behind it cannot be traced. The response and the log now carry every series length. Values stay out of the log as before, a length is shape, not content. dt was not among the validated series. The model indexes every series by it, so a short dt was an IndexError and a 500 instead of a 400 naming the series. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
andig
added a commit
that referenced
this pull request
Sep 10, 2026
evcc sends User-Agent evcc/<version> on every request. A rejected request is usually a client bug, and the version names the release carrying it: the empty solar forecast behind the length mismatches was fixed in evcc 0.309.0, which the log could not tell. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
andig
added a commit
that referenced
this pull request
Sep 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Found during Azure KPI checks for #151.
Malformed JSON currently raises an
AttributeErrorin the custom BadRequest handler and returns HTTP 500. Return a JSON 400 response instead, and make ordinary 400 rejections diagnosable in Log Analytics.bad_requestevent with the endpoint, cause category, first failing schema validator, and affected field names.🤖 Generated with OpenCode