Skip to content

Commit 266c4af

Browse files
Merge pull request #299 from QuietMisdreavus/long-long-readmes
skip loading readme files if it won't fit in the DB
2 parents 456bed4 + d9578fe commit 266c4af

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/db/add_package.rs

+8-1
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,14 @@ fn get_readme(pkg: &Package) -> Result<Option<String>> {
238238
let mut reader = try!(fs::File::open(readme_path).map(|f| BufReader::new(f)));
239239
let mut readme = String::new();
240240
try!(reader.read_to_string(&mut readme));
241-
Ok(Some(readme))
241+
242+
if readme.is_empty() {
243+
Ok(None)
244+
} else if readme.len() > 51200 {
245+
Ok(Some(format!("(Readme ignored due to being too long. ({} > 51200))", readme.len())))
246+
} else {
247+
Ok(Some(readme))
248+
}
242249
}
243250

244251

0 commit comments

Comments
 (0)