feat(validator): check for json subtypes in validator

[ztiromoritz]

A minimal fix for #2633 using a regex.
I liked the simplicity of the current startsWith-approach but it excluded subtypes.

I did:

• Add tests
• Run tests
• bun denoify to generate files for Deno

[yusukebe]

• You should use the test instead of the match if you don't capture strings.
• ^application\/[a-z-]*\+ should be application\/([a-z-]+\+)*.
• The last (\s*;.*)? is necessary?

[ztiromoritz]

Thanks!

• I use test now
• Landed on this regexp: /^application\/([a-z-]+\+)?json/
  • the ^ in front is needed to not fail the Xapplication/json test
  • I added ? to the ([a-z-]+\+) group instead of *, because there can only be one subtype
• Yes, (\s*;.*)? is not needed.

[yusukebe]

Hi @ztiromoritz

Thanks! Looks good. To avoid the CI error, could you run bun run format:fix && bun run lint:fix to format the code?

[yusukebe]

LGTM!

[yusukebe]

Hi @ztiromoritz

Thank you for your contribution. I'll merge this into the main now and will release it in a patch release since this is a minor feat.

[ztiromoritz]

Fine. My pleasure.
I close #2633 then. There where minor additional ideas, but overall I see no problem now.

[ztiromoritz]

(Hope this is according to the projects workflow. 😁 )