Ensure minified HTML is truncated before converting to String

Closes #1304
This commit is contained in:
Musee Ullah 2021-01-12 20:02:51 -06:00
parent 9444b920d2
commit 0a5181d621
No known key found for this signature in database
GPG key ID: FAF190AD6EB56A44

View file

@ -6,8 +6,8 @@ pub fn html(html: String) -> Result<String> {
let mut input_bytes = html.as_bytes().to_vec(); let mut input_bytes = html.as_bytes().to_vec();
match with_friendly_error(&mut input_bytes, cfg) { match with_friendly_error(&mut input_bytes, cfg) {
Ok(len) => match std::str::from_utf8(&input_bytes) { Ok(len) => match std::str::from_utf8(&input_bytes[..len]) {
Ok(result) => Ok(result[..len].to_string()), Ok(result) => Ok(result.to_string()),
Err(err) => bail!("Failed to convert bytes to string : {}", err), Err(err) => bail!("Failed to convert bytes to string : {}", err),
}, },
Err(minify_error) => { Err(minify_error) => {
@ -47,4 +47,16 @@ mod tests {
let res = html(input.to_owned()).unwrap(); let res = html(input.to_owned()).unwrap();
assert_eq!(res, expected); assert_eq!(res, expected);
} }
// https://github.com/getzola/zola/issues/1304
#[test]
fn can_minify_multibyte_characters() {
let input = r#"
"#;
let expected = r#"俺が好きなのはキツネの…ケツねw ー丁寧なインタネット生活の人より"#;
let res = html(input.to_owned()).unwrap();
assert_eq!(res, expected);
}
} }