fix: Remove charset parameter from MIME type of application/json

[SaekiTominaga]

fix #3739

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

[yusukebe]

Hi @SaekiTominaga

Thank you for the PR! I think you should change this line too:

hono/src/context.ts

Line 780 in 8886320

this.#preparedHeaders['content-type'] = 'application/json; charset=UTF-8'

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.72%. Comparing base (47bb23c) to head (c30bf0d).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3743      +/-   ##
==========================================
- Coverage   91.73%   91.72%   -0.01%     
==========================================
  Files         159      159              
  Lines       10177    10177              
  Branches     2883     2888       +5     
==========================================
- Hits         9336     9335       -1     
- Misses        840      841       +1     
  Partials        1        1              

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

[EdamAme-x]

These files may also need to be changed. (Note: there are multiple lines in the same file.)
Thanks.

hono/src/middleware/basic-auth/index.ts

Line 123 in 47bb23c

'content-type': 'application/json; charset=UTF-8',

hono/src/context.ts

Line 780 in 47bb23c

this.#preparedHeaders['content-type'] = 'application/json; charset=UTF-8'

hono/src/middleware/bearer-auth/index.ts

Line 106 in 47bb23c

'content-type': 'application/json; charset=UTF-8',

hono/src/context.test.ts

Line 54 in 47bb23c

expect(res.headers.get('Content-Type')).toMatch('application/json; charset=UTF-8')

hono/src/adapter/aws-lambda/handler.test.ts

Line 16 in 47bb23c

expect(isContentTypeBinary('application/json; charset=UTF-8')).toBe(false)

hono/runtime-tests/lambda/index.test.ts

Line 540 in 47bb23c

'content-type': 'application/json; charset=UTF-8',

hono/src/middleware/bearer-auth/index.test.ts

Line 403 in 47bb23c

expect(res.headers.get('Content-Type')).toMatch('application/json; charset=UTF-8')

hono/src/validator/validator.test.ts

Line 132 in 47bb23c

'Content-Type': 'text/plain;charset=utf-8',

hono/src/adapter/lambda-edge/handler.test.ts

Line 16 in 47bb23c

expect(isContentTypeBinary('application/json; charset=UTF-8')).toBe(false)

hono/src/middleware/basic-auth/index.test.ts

Line 298 in 47bb23c

expect(res.headers.get('Content-Type')).toMatch('application/json; charset=UTF-8')

[SaekiTominaga]

Thanks for pointing that out!

@yusukebe I have addressed it.
@EdamAme-x I have removed charset parameter or deleted unnecessary test cases.

[yusukebe]

LGTM!

[yusukebe]

Thanks @SaekiTominaga ! Looks good. I'll merge it now.