Improve update check (#5945)
* Send etherpad version in update check request * Cache etherpad informations for one hour * Change format of User-Agent * Improve exception handlingpull/5946/head
parent
f2faa3fa84
commit
f9a3a6aaae
|
@ -2,28 +2,39 @@
|
||||||
const semver = require('semver');
|
const semver = require('semver');
|
||||||
const settings = require('./Settings');
|
const settings = require('./Settings');
|
||||||
const axios = require('axios');
|
const axios = require('axios');
|
||||||
let infos;
|
const headers = {
|
||||||
|
'User-Agent': 'Etherpad/' + settings.getEpVersion(),
|
||||||
|
}
|
||||||
|
|
||||||
const loadEtherpadInformations = () =>
|
const updateInterval = 60 * 60 * 1000; // 1 hour
|
||||||
axios.get('https://static.etherpad.org/info.json')
|
let infos;
|
||||||
|
let lastLoadingTime = null;
|
||||||
|
|
||||||
|
const loadEtherpadInformations = () => {
|
||||||
|
if (lastLoadingTime !== null && Date.now() - lastLoadingTime < updateInterval) {
|
||||||
|
return Promise.resolve(infos);
|
||||||
|
}
|
||||||
|
|
||||||
|
return axios.get('https://static.etherpad.org/info.json', {headers: headers})
|
||||||
.then(async resp => {
|
.then(async resp => {
|
||||||
try {
|
|
||||||
infos = await resp.data;
|
infos = await resp.data;
|
||||||
if (infos === undefined || infos === null) {
|
if (infos === undefined || infos === null) {
|
||||||
await Promise.reject("Could not retrieve current version")
|
await Promise.reject("Could not retrieve current version")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
lastLoadingTime = Date.now();
|
||||||
return await Promise.resolve(infos);
|
return await Promise.resolve(infos);
|
||||||
}
|
|
||||||
catch (err) {
|
|
||||||
return await Promise.reject(err);
|
|
||||||
}
|
|
||||||
})
|
})
|
||||||
|
.catch(async err => {
|
||||||
|
return await Promise.reject(err);
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
exports.getLatestVersion = () => {
|
exports.getLatestVersion = () => {
|
||||||
exports.needsUpdate();
|
exports.needsUpdate().catch();
|
||||||
return infos.latestVersion;
|
return infos?.latestVersion;
|
||||||
};
|
};
|
||||||
|
|
||||||
exports.needsUpdate = async (cb) => {
|
exports.needsUpdate = async (cb) => {
|
||||||
|
|
Loading…
Reference in New Issue