Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

status code and text not required #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mozfreddyb
Copy link

According to some quick testing, most browsers won't need these, so that would save you a few bytes.
Question is how many browsers you aim to support though and what's more important to you (bytes / browsers), so this wasn't really clear to me up front

According to some quick testing, most browsers won't need these, so that would save you a few bytes.
Question is how many browsers you aim to support though and what's more important to you (bytes / browsers), so this wasn't really clear to me up front
@diracdeltas
Copy link
Owner

thanks for the bytes saved! what browsers did you test out of curiosity?

@mozfreddyb
Copy link
Author

Ugh, I tested locally but without deflate. simple html content works though. 😕

Alternatively, one could save a few bytes by removing all sorts of HTTP headers and going for HTTP/0.9 support (which surprisingly, chrome and firefox still have), but that also wouldn't allow deflate afaict.

Unfortunately, I couldnt get the test server in utils/ to give me a response that's similar enough to the .c server. Maybe I should have just tested with the c server directly 😊

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.

2 participants