Get version from build VERSION variable (#19857)

* Fetch version from VERSION set during build
* When polling for new versions, compare to VERSION set during build
* Strip leading v from version, matching package.sh
pull/19926/head
James Salter 2021-11-23 18:42:24 +11:00 committed by GitHub
parent 9746517ef7
commit 194aeac19e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 12 deletions

View File

@ -33,8 +33,6 @@ import { logger } from "matrix-js-sdk/src/logger";
const POKE_RATE_MS = 10 * 60 * 1000; // 10 min
export default class WebPlatform extends VectorBasePlatform {
private runningVersion: string = null;
constructor() {
super();
// Register service worker if available on this platform
@ -102,7 +100,7 @@ export default class WebPlatform extends VectorBasePlatform {
return notification;
}
private getVersion(): Promise<string> {
private getMostRecentVersion(): Promise<string> {
// We add a cachebuster to the request to make sure that we know about
// the most recent version on the origin server. That might not
// actually be the version we'd get on a reload (particularly in the
@ -131,10 +129,15 @@ export default class WebPlatform extends VectorBasePlatform {
}
getAppVersion(): Promise<string> {
if (this.runningVersion !== null) {
return Promise.resolve(this.runningVersion);
let ver = process.env.VERSION;
// if version looks like semver with leading v, strip it
// (matches scripts/package.sh)
const semVerRegex = new RegExp("^v[0-9]+.[0-9]+.[0-9]+(-.+)?$");
if (semVerRegex.test(process.env.VERSION)) {
ver = process.env.VERSION.substr(1);
}
return this.getVersion();
return Promise.resolve(ver);
}
startUpdater() {
@ -147,12 +150,10 @@ export default class WebPlatform extends VectorBasePlatform {
}
pollForUpdate = () => {
return this.getVersion().then((ver) => {
if (this.runningVersion === null) {
this.runningVersion = ver;
} else if (this.runningVersion !== ver) {
if (this.shouldShowUpdate(ver)) {
showUpdateToast(this.runningVersion, ver);
return this.getMostRecentVersion().then((mostRecentVersion) => {
if (process.env.VERSION !== mostRecentVersion) {
if (this.shouldShowUpdate(mostRecentVersion)) {
showUpdateToast(process.env.VERSION, mostRecentVersion);
}
return { status: UpdateCheckStatus.Ready };
} else {