makes acquireLock and releaseLock more consistent

konstantinabl · konstantinabl · commit b6c39a2507ab · 2025-12-02T14:09:18.000+02:00
Signed-off-by: Konstantina Blazhukova &lt;konstantina.blajukova@gmail.com&gt;
diff --git a/packages/relay/src/lib/services/lockService/LockService.ts b/packages/relay/src/lib/services/lockService/LockService.ts
@@ -31,10 +31,9 @@ export class LockService {
    * @returns A promise that resolves to a unique session key, or null if acquisition fails (fail open).
    */
   async acquireLock(address: string): Promise<string | undefined> {
-    if (!ConfigService.get('ENABLE_NONCE_ORDERING')) {
-      return;
+    if (ConfigService.get('ENABLE_NONCE_ORDERING')) {
+      return await this.strategy.acquireLock(address);
     }
-    return await this.strategy.acquireLock(address);
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -31,10 +31,9 @@ export class LockService {`
`31`	`31`	`* @returns A promise that resolves to a unique session key, or null if acquisition fails (fail open).`
`32`	`32`	`*/`
`33`	`33`	`async acquireLock(address: string): Promise<string \| undefined> {`
`34`		`- if (!ConfigService.get('ENABLE_NONCE_ORDERING')) {`
`35`		`- return;`
	`34`	`+ if (ConfigService.get('ENABLE_NONCE_ORDERING')) {`
	`35`	`+ return await this.strategy.acquireLock(address);`
`36`	`36`	`}`
`37`		`- return await this.strategy.acquireLock(address);`
`38`	`37`	`}`
`39`	`38`
`40`	`39`	`/**`