From b239ae1029eef5a68bf98d21e81acd317abb0673 Mon Sep 17 00:00:00 2001 From: Sebastian Date: Thu, 28 Apr 2022 13:26:29 -0300 Subject: keep alive manifest v3 --- .../taler-wallet-webextension/src/platform/api.ts | 4 ++ .../src/platform/chrome.ts | 19 +++++- .../taler-wallet-webextension/src/platform/dev.ts | 1 + .../src/serviceWorkerTimerAPI.ts | 70 ---------------------- .../taler-wallet-webextension/src/wxBackend.ts | 12 ++-- 5 files changed, 30 insertions(+), 76 deletions(-) delete mode 100644 packages/taler-wallet-webextension/src/serviceWorkerTimerAPI.ts (limited to 'packages/taler-wallet-webextension/src') diff --git a/packages/taler-wallet-webextension/src/platform/api.ts b/packages/taler-wallet-webextension/src/platform/api.ts index cee628a6f..4713d5d00 100644 --- a/packages/taler-wallet-webextension/src/platform/api.ts +++ b/packages/taler-wallet-webextension/src/platform/api.ts @@ -56,6 +56,10 @@ export interface WalletVersion { * WebExtension APIs consistently. */ export interface PlatformAPI { + /** + * Garantee that the + */ + keepAlive(cb: VoidFunction): void; /** * FIXME: should not be needed * diff --git a/packages/taler-wallet-webextension/src/platform/chrome.ts b/packages/taler-wallet-webextension/src/platform/chrome.ts index d295168bf..900470dff 100644 --- a/packages/taler-wallet-webextension/src/platform/chrome.ts +++ b/packages/taler-wallet-webextension/src/platform/chrome.ts @@ -15,7 +15,7 @@ TALER; see the file COPYING. If not, see */ -import { classifyTalerUri, CoreApiResponse, TalerUriType } from "@gnu-taler/taler-util"; +import { classifyTalerUri, CoreApiResponse, Logger, TalerUriType } from "@gnu-taler/taler-util"; import { CrossBrowserPermissionsApi, MessageFromBackend, Permissions, PlatformAPI } from "./api.js"; const api: PlatformAPI = { @@ -38,10 +38,27 @@ const api: PlatformAPI = { sendMessageToWalletBackground, useServiceWorkerAsBackgroundProcess, containsTalerHeaderListener, + keepAlive, } export default api; +const logger = new Logger("chrome.ts"); + +function keepAlive(callback: any): void { + if (chrome.runtime && chrome.runtime.getManifest().manifest_version === 3) { + chrome.alarms.create("wallet-worker", { periodInMinutes: 1 }) + + chrome.alarms.onAlarm.addListener((a) => { + logger.trace(`kee p alive alarm: ${a.name}`) + callback() + }) + } else { + callback(); + } + +} + function isFirefox(): boolean { return false; } diff --git a/packages/taler-wallet-webextension/src/platform/dev.ts b/packages/taler-wallet-webextension/src/platform/dev.ts index e05ff29e9..59e6738c5 100644 --- a/packages/taler-wallet-webextension/src/platform/dev.ts +++ b/packages/taler-wallet-webextension/src/platform/dev.ts @@ -21,6 +21,7 @@ const frames = ["popup", "wallet"] const api: PlatformAPI = ({ isFirefox: () => false, + keepAlive: (cb: VoidFunction) => cb(), findTalerUriInActiveTab: async () => undefined, containsTalerHeaderListener: () => { return true }, getPermissionsApi: () => ({ diff --git a/packages/taler-wallet-webextension/src/serviceWorkerTimerAPI.ts b/packages/taler-wallet-webextension/src/serviceWorkerTimerAPI.ts deleted file mode 100644 index 15614ed18..000000000 --- a/packages/taler-wallet-webextension/src/serviceWorkerTimerAPI.ts +++ /dev/null @@ -1,70 +0,0 @@ -/* - This file is part of GNU Taler - (C) 2020 Taler Systems S.A. - - GNU Taler is free software; you can redistribute it and/or modify it under the - terms of the GNU General Public License as published by the Free Software - Foundation; either version 3, or (at your option) any later version. - - GNU Taler is distributed in the hope that it will be useful, but WITHOUT ANY - WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR - A PARTICULAR PURPOSE. See the GNU General Public License for more details. - - You should have received a copy of the GNU General Public License along with - GNU Taler; see the file COPYING. If not, see - */ - -/** - * Imports. - */ -import { Logger } from "@gnu-taler/taler-util"; -import { timer, TimerAPI, TimerHandle } from "@gnu-taler/taler-wallet-core"; - - -const logger = new Logger("ServiceWorkerTimerGroup.ts"); - -/** - * Implementation of [[TimerAPI]] using alarm API - */ -export class ServiceWorkerTimerAPI implements TimerAPI { - - - /** - * Call a function every time the delay given in milliseconds passes. - */ - every(delayMs: number, callback: VoidFunction): TimerHandle { - const seconds = delayMs / 1000; - const periodInMinutes = Math.round(seconds < 61 ? 1 : seconds / 60); - - logger.info(`creating a alarm every ${periodInMinutes} min, intended delay was ${delayMs}`) - // chrome.alarms.create("wallet-worker", { periodInMinutes }) - // chrome.alarms.onAlarm.addListener((a) => { - // logger.info(`alarm called, every: ${a.name}`) - // callback() - // }) - - const p = timer.every(delayMs, callback) - return p; - } - - /** - * Call a function after the delay given in milliseconds passes. - */ - after(delayMs: number, callback: VoidFunction): TimerHandle { - const seconds = delayMs / 1000; - const delayInMinutes = Math.round(seconds < 61 ? 1 : seconds / 60); - - logger.info(`creating a alarm after ${delayInMinutes} min, intended delay was ${delayMs}`) - chrome.alarms.create("wallet-worker", { delayInMinutes }) - // chrome.alarms.onAlarm.addListener((a) => { - // logger.info(`alarm called, after: ${a.name}`) - // callback(); - // }) - - - const p = timer.after(delayMs, callback); - - return p - } - -} diff --git a/packages/taler-wallet-webextension/src/wxBackend.ts b/packages/taler-wallet-webextension/src/wxBackend.ts index b0d690ef8..2ac8f6343 100644 --- a/packages/taler-wallet-webextension/src/wxBackend.ts +++ b/packages/taler-wallet-webextension/src/wxBackend.ts @@ -46,7 +46,6 @@ import { BrowserHttpLib } from "./browserHttpLib.js"; import { MessageFromBackend, platform } from "./platform/api.js"; import { SynchronousCryptoWorkerFactory } from "./serviceWorkerCryptoWorkerFactory.js"; import { ServiceWorkerHttpLib } from "./serviceWorkerHttpLib.js"; -import { ServiceWorkerTimerAPI } from "./serviceWorkerTimerAPI.js"; /** * Currently active wallet instance. Might be unloaded and @@ -196,7 +195,7 @@ async function reinitWallet(): Promise { if (platform.useServiceWorkerAsBackgroundProcess()) { httpLib = new ServiceWorkerHttpLib(); cryptoWorker = new SynchronousCryptoWorkerFactory(); - timer = new ServiceWorkerTimerAPI(); + timer = new SetTimeoutTimerAPI(); } else { httpLib = new BrowserHttpLib(); cryptoWorker = new BrowserCryptoWorkerFactory(); @@ -216,9 +215,12 @@ async function reinitWallet(): Promise { const message: MessageFromBackend = { type: x.type }; platform.sendMessageToAllChannels(message) }); - wallet.runTaskLoop().catch((e) => { - logger.error("error during wallet task loop", e); - }); + + platform.keepAlive(() => { + wallet.runTaskLoop().catch((e) => { + logger.error("error during wallet task loop", e); + }); + }) // Useful for debugging in the background page. if (typeof window !== "undefined") { (window as any).talerWallet = wallet; -- cgit v1.2.3