Skip to content

chore: boundary error message#7419

Merged
jasonsaayman merged 2 commits intoaxios:v1.xfrom
cuiweixie:typo
Apr 24, 2026
Merged

chore: boundary error message#7419
jasonsaayman merged 2 commits intoaxios:v1.xfrom
cuiweixie:typo

Conversation

@cuiweixie
Copy link
Copy Markdown
Contributor

@cuiweixie cuiweixie commented Feb 20, 2026


Summary by cubic

Fixes the boundary length error message in formDataToStream to say “1–70” instead of “10–70,” matching the actual validation. No behavior changes, only clearer feedback.

Description

  • Change thrown error text to “1–70” to match the existing check (<1 or >70).
  • No logic or API changes.

Docs, Testing, Versioning

  • Docs: If any docs mention the boundary range, update /docs/ to “1–70”.
  • Testing: No new tests. Update assertions only if they check this exact message.
  • Semantic version impact: Patch.

Written for commit d4e8435. Summary will update on new commits.

@cuiweixie cuiweixie changed the title chore: boundary error chore: boundary error message Feb 20, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@jasonsaayman jasonsaayman added priority::medium A medium priority commit::chore The PR is related to a chore labels Apr 24, 2026
Copy link
Copy Markdown
Member

@jasonsaayman jasonsaayman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jasonsaayman jasonsaayman merged commit 2a08578 into axios:v1.x Apr 24, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit::chore The PR is related to a chore priority::medium A medium priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants