Merge pull request #3753 from thelounge/xpaw/yarn-stdout

Wrap stdout parsing from yarn into try/catch
This commit is contained in:
Pavel Djundik 2020-02-14 14:52:01 +02:00 committed by GitHub
commit 7fdb70d451
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -132,10 +132,16 @@ class Utils {
.trim() .trim()
.split("\n") .split("\n")
.forEach((line) => { .forEach((line) => {
line = JSON.parse(line); try {
line = JSON.parse(line);
if (line.type === "success") { if (line.type === "success") {
success = true; success = true;
}
} catch (e) {
// Stdout buffer has limitations and yarn may print
// big package trees, for example in the upgrade command
// See https://github.com/thelounge/thelounge/issues/3679
} }
}); });
}); });