-
Notifications
You must be signed in to change notification settings - Fork 198
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
fix(http-server): append livereload to end of document; #79
Conversation
Nice ✅, probably don't really need |
Yeah, I don't think HTMLRewriter is needed at all anymore, but I felt bad removing it outright :D |
Hang on, just thought of an issue. If the |
I noticed the same thing while playing with the rewriter in cf playground. Have a note to open an internal issue tomorrow |
Oh, didn't realise it was an issue with the real implementation too. 😄 |
Oops – total brain fart. The CF playground implementation is okay – at least while returning |
Ok, I'll fix that now in Miniflare's implementation 👍 |
Scripts are still valid & will still execute, even if outside of the
<html>
tags. The browsers' HTML parsers are incredibly loose/forgiving hahaHere's a valid HTML document, which still logs to the console:
Closes #70