Merge pull request #349 from Authenticator-Extension/5.3.1-patches

mymindstorm · web-flow · commit 43158a2b878c · 2019-07-04T10:43:15.000-05:00
5.3.1 patches
diff --git a/manifest-chrome.json b/manifest-chrome.json
@@ -2,7 +2,7 @@
     "manifest_version": 2,
     "name": "__MSG_extName__",
     "short_name": "__MSG_extShortName__",
-    "version": "5.3.1",
+    "version": "5.3.2",
     "default_locale": "en",
     "description": "__MSG_extDesc__",
     "icons": {
diff --git a/manifest-firefox.json b/manifest-firefox.json
@@ -2,7 +2,7 @@
     "manifest_version": 2,
     "name": "__MSG_extName__",
     "short_name": "__MSG_extShortName__",
-    "version": "5.3.1",
+    "version": "5.3.2",
     "default_locale": "en",
     "description": "__MSG_extDesc__",
     "applications": {
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/models/otp.ts b/src/models/otp.ts
@@ -57,8 +57,10 @@ class OTPEntry implements OTP {
       return;
     }
     this.generate();
-    this.counter++;
-    await this.update(encryption);
+    if (this.secret !== 'Encrypted') {
+      this.counter++;
+      await this.update(encryption);
+    }
     return;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -57,8 +57,10 @@ class OTPEntry implements OTP {`
`57`	`57`	`return;`
`58`	`58`	`}`
`59`	`59`	`this.generate();`
`60`		`- this.counter++;`
`61`		`- await this.update(encryption);`
	`60`	`+ if (this.secret !== 'Encrypted') {`
	`61`	`+ this.counter++;`
	`62`	`+ await this.update(encryption);`
	`63`	`+ }`
`62`	`64`	`return;`
`63`	`65`	`}`
`64`	`66`