Shouldn't this just be `file.extension_str() == Some("html")`? Otherwise `foo.not-html` will be treated as if it were `.html`?