Display error/warning for too large input #72

src/_locales/en/messages.json

@@ -80,6 +80,10 @@
     "message": "Permission request failed.",
     "description": "Error shown when a permission request failed. This indicates a technical error and does NOT mean the user declined the permission."
   },
+  "errorQrCodeOverflow": {
+    "message": "Cannot generate QR code, the provided text is too much.",
+    "description": "Error shown when QR code cannot be generated due to overflow in input size."
+  },
   "requestDownloadPermissionForQr": {
     "message": "To continue to save the QR code, we need the download permission.",
     "description": "Shown, when the user is asked to allow the download permission to save the QR code."

src/popup/modules/QrLib/kjua.js

@@ -3,10 +3,12 @@
  *
  * @module QrLib/kjua
  * @requires /common/modules/Logger
+ * @requires qrerr
  */
 /* globals kjua */
 
 import * as Logger from "/common/modules/Logger/Logger.js";
+import * as QrErr from "./qrerr.js";
 
 /**
  * The type of QR code this library generates.
@@ -112,7 +114,13 @@ export function set(tag, value) {
  */
 export function getQr() {
     Logger.logInfo("generated new qr kjua code", kjuaOptions);
-    return kjua(kjuaOptions);
+
+    try {
+        return kjua(kjuaOptions);
+    } catch (err) {
+        throw err.message.startsWith("code length overflow.")
+          ? new QrErr.DataOverflowError() : err;
+    }
 }
 
 /**

src/popup/modules/QrLib/qrerr.js

@@ -0,0 +1,12 @@
+/**
+ * Errors in QR code generation
+ *
+ * @module QrLib/qrerr
+ */
+
+export class DataOverflowError extends Error {
+  constructor(...params) {
+    super(...params);
+    this.message = 'data overflow error';
+  }
+}

src/popup/modules/QrLib/qrgen.js

@@ -7,6 +7,7 @@
 /* globals qrcodegen */
 
 import * as Logger from "/common/modules/Logger/Logger.js";
+import * as QrErr from "./qrerr.js";
 
 const QRC = qrcodegen.QrCode;
 
@@ -101,7 +102,13 @@ export function set(tag, value) {
 export function getQr() {
     Logger.logInfo("generated new QrGen qr code");
 
-    const qrElem = QRC.encodeText(qrText, qrErrorCorrection);
+    try {
+        const qrElem = QRC.encodeText(qrText, qrErrorCorrection);
+    } catch (err) {
+        throw (err === 'Data too long')
+            ? QrErr.DataOverflowError : err;
+    }
+
     const svgString = qrElem.toSvgString(qrQuietZone);
 
     return getSvgElement(svgString);

src/popup/modules/UserInterface.js

@@ -109,8 +109,18 @@ const refreshQrCode = throttle(() => {
         CommonMessages.hideError();
     }
 
-    QrCreator.setTextInternal(text);
-    QrCreator.generate();
+    try {
+        QrCreator.setTextInternal(text);
+        QrCreator.generate();
+    } catch (e) {
+        // Error thrown from qrcodegen & kjua wrapper when code too long
+        if (e === "Data too long") {
+            CommonMessages.showError("errorQrCodeOverflow");
+            Logger.logError("Data exceeds maximum size:", text.length);
+        } else {
+          throw e;
+        }
+    }
 }, QR_CODE_REFRESH_TIMEOUT);
 
 /**