mirror of
https://github.com/woodpecker-ci/woodpecker.git
synced 2024-11-22 09:51:01 +00:00
logline output buffers rather than updating on every ws message
In builds with lots of output, trying to reload the page partway through the build results in thousands of websocket messages (one per line) that each update the DOM. This can cause the browser to freeze. Instead, use requestAnimation frame to delay the DOM updates.
This commit is contained in:
parent
77ff7971d3
commit
6e92e73a98
1 changed files with 30 additions and 7 deletions
|
@ -133,16 +133,39 @@
|
|||
outputWS.onopen = function () { console.log('output websocket open'); };
|
||||
outputWS.onerror = function (e) { console.log('websocket error: ' + e); };
|
||||
outputWS.onclose = function (e) { window.location.reload(); };
|
||||
outputWS.onmessage = function (e) {
|
||||
outputBox.innerHTML += formatLine(e.data);
|
||||
if (window.autofollow) {
|
||||
window.scrollTo(0, document.body.scrollHeight);
|
||||
}
|
||||
|
||||
window.requestAnimationFrame = window.requestAnimationFrame ||
|
||||
window.webkitRequestAnimationFrame ||
|
||||
function(callback, element) {
|
||||
return window.setTimeout(function() {
|
||||
callback(+new Date());
|
||||
}, 1000 / 60);
|
||||
};
|
||||
|
||||
var lineBuffer = "";
|
||||
|
||||
outputWS.onmessage = function (e) {
|
||||
lineBuffer += formatLine(e.data);
|
||||
};
|
||||
{{ else }}
|
||||
|
||||
function updateScreen() {
|
||||
if(lineBuffer.length > 0) {
|
||||
outputBox.innerHTML += lineBuffer;
|
||||
lineBuffer = '';
|
||||
|
||||
if (window.autofollow) {
|
||||
window.scrollTo(0, document.body.scrollHeight);
|
||||
}
|
||||
}
|
||||
requestAnimationFrame(updateScreen);
|
||||
}
|
||||
|
||||
requestAnimationFrame(updateScreen);
|
||||
|
||||
{{ else }}
|
||||
$.get("/{{ .Repo.Slug }}/commit/{{ .Commit.Hash }}/build/{{ .Build.Slug }}/out.txt", function( data ) {
|
||||
$( "#stdout" ).html(formatLine(data));
|
||||
});
|
||||
{{ end }}
|
||||
</script>
|
||||
{{ end }}
|
||||
{{ end }}
|
Loading…
Reference in a new issue