Skip to content

Commit

Permalink
Reduce log spam when starting WPT server
Browse files Browse the repository at this point in the history
  • Loading branch information
domenic committed Mar 7, 2021
1 parent 708ca1c commit 9e94e80
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions test/web-platform-tests/start-wpt-server.js
Expand Up @@ -65,16 +65,21 @@ module.exports = ({ toUpstream = false } = {}) => {
);
};

function pollForServer(url) {
function pollForServer(url, lastLogTime = Date.now()) {
return requestHead(url, { strictSSL: false })
.then(() => {
console.log(`WPT server at ${url} is up!`);
return url;
})
.catch(err => {
console.log(`WPT server at ${url} is not up yet (${err.message}); trying again`);
// Only log every 5 seconds to be less spammy.
if (Date.now() - lastLogTime >= 5000) {
console.log(`WPT server at ${url} is not up yet (${err.message}); trying again`);
lastLogTime = Date.now();
}

return new Promise(resolve => {
setTimeout(() => resolve(pollForServer(url)), 500);
setTimeout(() => resolve(pollForServer(url, lastLogTime)), 500);
});
});
}

0 comments on commit 9e94e80

Please sign in to comment.