Add hash to albatross-client file for cache-bust

nimiq · Nov 29, 2024 · 36d1aea · 36d1aea
1 parent 133e7a5
commit 36d1aea
Show file tree

Hide file tree

Showing 8 changed files with 41 additions and 11 deletions.
diff --git a/demos/index.html b/demos/index.html
@@ -182,7 +182,7 @@ <h2>Result</h2>
     crossorigin="anonymous"></script>
 
 <script type="module">
-    import init, * as Nimiq from '/albatross-client/web/index.js';
+    import init, * as Nimiq from '/albatross-client/web/index.js?v=<%= htmlWebpackPlugin.options.albatrossClientHash %>';
 
     /** @type {Promise<Nimiq> | undefined} */
     let initPromise;

diff --git a/public/cashlink.html b/public/cashlink.html
@@ -22,7 +22,7 @@
     <link rel="icon" href="<%= htmlWebpackPlugin.options.domain %>/favicon.ico">
 
     <script type="module">
-        import init, * as Nimiq from '/albatross-client/web/index.js';
+        import init, * as Nimiq from '/albatross-client/web/index.js?v=<%= htmlWebpackPlugin.options.albatrossClientHash %>';
 
         /** @type {Promise<Nimiq> | undefined} */
         let initPromise;

diff --git a/public/export.html b/public/export.html
@@ -13,7 +13,7 @@
     </style>
 
     <script type="module">
-        import init, * as Nimiq from '/albatross-client/web/index.js';
+        import init, * as Nimiq from '/albatross-client/web/index.js?v=<%= htmlWebpackPlugin.options.albatrossClientHash %>';
 
         /** @type {Promise<Nimiq> | undefined} */
         let initPromise;

diff --git a/public/iframe.html b/public/iframe.html
@@ -6,7 +6,7 @@
     <meta name="robots" content="noindex">
     <title>Nimiq Hub IFrame</title>
     <script type="module">
-      import init, * as Nimiq from '/albatross-client/web/index.js';
+      import init, * as Nimiq from '/albatross-client/web/index.js?v=<%= htmlWebpackPlugin.options.albatrossClientHash %>';
 
       /** @type {Promise<Nimiq> | undefined} */
       let initPromise;

diff --git a/public/index.html b/public/index.html
@@ -20,7 +20,7 @@
     <link rel="icon" href="<%= htmlWebpackPlugin.options.domain %>/favicon.ico">
 
     <script type="module">
-        import init, * as Nimiq from '/albatross-client/web/index.js';
+        import init, * as Nimiq from '/albatross-client/web/index.js?v=<%= htmlWebpackPlugin.options.albatrossClientHash %>';
 
         /** @type {Promise<Nimiq> | undefined} */
         let initPromise;

diff --git a/src/i18n/en.po b/src/i18n/en.po
@@ -404,7 +404,7 @@ msgid "Connect your Ledger hardware wallet."
 msgstr ""
 
 #: src/views/Migrate.vue:143
-#: src/views/Migrate.vue:285
+#: src/views/Migrate.vue:283
 msgid "Connecting to Keyguard..."
 msgstr ""
 
@@ -1189,7 +1189,7 @@ msgstr ""
 msgid "Unsupported Request"
 msgstr ""
 
-#: src/components/CheckoutCardNimiq.vue:329
+#: src/components/CheckoutCardNimiq.vue:327
 msgid "Updating balances"
 msgstr ""
 
@@ -1198,7 +1198,7 @@ msgid "Updating Keyguard..."
 msgstr ""
 
 #: src/views/Migrate.vue:142
-#: src/views/Migrate.vue:284
+#: src/views/Migrate.vue:282
 msgid "Updating your Accounts"
 msgstr ""
 

diff --git a/src/iframe.ts b/src/iframe.ts
@@ -26,7 +26,29 @@ import { deriveAddressesFromXPub } from './lib/bitcoin/BitcoinUtils';
 import { detectLanguage, setLanguage } from './i18n/i18n-setup';
 
 class IFrameApi {
-    public static run() {
+    private static async waitForLoadAlbatross(): Promise<void> {
+        // Wait for loadAlbatross to be available, with timeout
+        const maxWaitTime = 10000; // 10 seconds
+        const startTime = Date.now();
+
+        while (!window.loadAlbatross) {
+            if (Date.now() - startTime > maxWaitTime) {
+                throw new Error('Timeout waiting for loadAlbatross to be available');
+            }
+            await new Promise(resolve => setTimeout(resolve, 50)); // Wait 50ms before checking again
+        }
+    }
+
+    public static async run() {
+        // Wait for loadAlbatross to be available
+        try {
+            await IFrameApi.waitForLoadAlbatross();
+            window.Nimiq = await window.loadAlbatross();
+        } catch (e) {
+            console.error('Failed to load Albatross:', e);
+            throw e;
+        }
+
         const rpcServer = new RpcServer(Config.privilegedOrigins);
 
         // Register handlers
@@ -38,8 +60,7 @@ class IFrameApi {
     }
 
     public static async list(): Promise<Account[]> {
-        window.Nimiq = await window.loadAlbatross();
-
+        // Albatross is already loaded in run(), no need to load it again
         let wallets: WalletInfoEntry[];
         if (BrowserDetection.isIOS() || BrowserDetection.isSafari()) {
             /*

diff --git a/vue.config.js b/vue.config.js
@@ -40,6 +40,10 @@ const bitcoinJsIntegrityHash = `sha256-${createHash('sha256')
     .update(fs.readFileSync(path.join(__dirname, 'public/bitcoin/BitcoinJS.min.js')))
     .digest('base64')}`;
 
+const albatrossClientHash = `sha256-${createHash('sha256')
+    .update(fs.readFileSync(path.join(__dirname, 'node_modules/@nimiq/albatross-wasm/web/index.js')))
+    .digest('base64')}`;
+
 // Accessible within client code via process.env.VUE_APP_BITCOIN_JS_INTEGRITY_HASH,
 // see https://cli.vuejs.org/guide/mode-and-env.html#using-env-variables-in-client-side-code
 process.env.VUE_APP_BITCOIN_JS_INTEGRITY_HASH = bitcoinJsIntegrityHash;
@@ -129,6 +133,7 @@ const pages = {
         // insert browser warning html template
         browserWarningTemplate,
         browserWarningIntegrityHash,
+        albatrossClientHash,
         domain,
         cdnDomain,
         // output as dist/index.html
@@ -142,6 +147,7 @@ const pages = {
         entry: 'src/iframe.ts',
         // the source template
         template: 'public/iframe.html',
+        albatrossClientHash,
         // output as dist/iframe.html
         filename: 'iframe.html',
         // chunks to include on this page, by default includes
@@ -156,6 +162,7 @@ const pages = {
         // insert browser warning html template
         browserWarningTemplate,
         browserWarningIntegrityHash,
+        albatrossClientHash,
         domain,
         cdnDomain,
         // output as dist/cashlink/index.html
@@ -169,6 +176,7 @@ const pages = {
         entry: 'src/export.ts',
         // the source template
         template: 'public/export.html',
+        albatrossClientHash,
         cdnDomain,
         // output as dist/iframe.html
         filename: 'export.html',
@@ -183,6 +191,7 @@ if (buildName === 'local' || buildName === 'testnet') {
         template: 'demos/index.html',
         cdnDomain,
         bitcoinJsIntegrityHash,
+        albatrossClientHash,
         // output as dist/demos.html
         filename: 'demos.html',
         // chunks to include on this page, by default includes