Skip to content

fix(utils/body,validator): normalize Content-Type media type for case-insensitive matching - #5067

Merged
yusukebe merged 1 commit into
mainfrom
fix/normalize-content-type
Jul 1, 2026
Merged

yusukebe merged 1 commit into
mainfrom
fix/normalize-content-type

Conversation

@yusukebe

@yusukebe yusukebe commented Jul 1, 2026 •

Copy link
Copy Markdown
Member

Fixes #5060

The author should do the following, if applicable

  • Add tests
  • Run tests
  • bun run format:fix && bun run lint:fix to format the code
  • Add TSDoc/JSDoc to document the code

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

HTTP Performance Benchmark

Framework Runtime Average Ping Query Body
hono (origin/main) bun 44,046.27 53,746.17 42,528.27 35,864.38
hono (current) bun 45,017.08 55,983.88 43,265.02 35,802.34
Change +2.20% +4.16% +1.73% -0.17%

@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.47059% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.08%. Comparing base (9728702) to head (5226d41).

Files with missing lines Patch % Lines
src/utils/body.ts 75.00% 3 Missing ⚠️
src/utils/buffer.ts 50.00% 1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (76.47%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5067      +/-   ##
==========================================
- Coverage   79.09%   79.08%   -0.01%     
==========================================
  Files         154      154              
  Lines       10727    10735       +8     
  Branches     2238     2239       +1     
==========================================
+ Hits         8484     8490       +6     
- Misses       2243     2245       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yusukebe
yusukebe merged commit b20d422 into main Jul 1, 2026
20 checks passed
@yusukebe
yusukebe deleted the fix/normalize-content-type branch July 1, 2026 09:42
kamui added a commit to kamui/skills that referenced this pull request Sep 24, 2026
Vetting hunts and independent adjudicators, each a headless Opus 5.5 high session that saw
no reviewer output, chose and ruled on withastro/astro#16079 (security, web backend),
honojs/hono#5067 (released-compatibility break), jonhadfield/soba#195 (clean refactor) and
mui/base-ui#5460 (React component logic). Each target has a factual packet from
build_packet.py --factual, a verified truncated mirror, a hashed dependency cache, smoke
checks at head and base, and its register sealed by seal.py with the plaintext hash in
target.json. The hunt reports and rulings are sealed in the research bundle; the ledger
records the eight sessions ($26.29).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

parseBody and validator skip mixed-case Content-Type media types

1 participant