Skip to content

fix: append charset=utf-8 to text/html#122

Merged
lukeed merged 5 commits intolukeed:masterfrom
aleclarson:fix/106
Oct 15, 2021
Merged

fix: append charset=utf-8 to text/html#122
lukeed merged 5 commits intolukeed:masterfrom
aleclarson:fix/106

Conversation

@aleclarson
Copy link
Contributor

@aleclarson aleclarson commented Oct 15, 2021

Closes #106

I considered fixing this upstream in "mime" or "mime-db" packages, but then I came across these issues, which indicated that it's not a viable path.

broofa/mime#174
jshttp/mime-db#94

Size before: 5204 bytes
Size after: 5108 bytes (-96)

Closes lukeed#106

I considered fixing this upstream in "mime" or "mime-db" packages,
but then I came across these issues, which indicated that it was
not a viable path.

broofa/mime#174
jshttp/mime-db#94
@aleclarson
Copy link
Contributor Author

aleclarson commented Oct 15, 2021

Hmm, I guess wc -c isn't accurate.

The build script reports 3.07 kB before and 3.03 kB after (for mjs module).

@lukeed
Copy link
Owner

lukeed commented Oct 15, 2021

Thanks! I'll review eod tomorrow or the weekend and get a release out. I'll tweak code style here a bit but that's on me, no worries :)

Can you add a test though, just to lock it in for the future? Ty

Copy link
Owner

@lukeed lukeed left a comment

Choose a reason for hiding this comment

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

Thanks!

@lukeed lukeed merged commit d1b8ba6 into lukeed:master Oct 15, 2021
@btea btea mentioned this pull request Oct 1, 2023
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.

Content-Type charset=UTF-8 missing for text/html

2 participants