Skip to content

Commit d71b576

Browse files
authored
fix: resolve remaining RCON cvar generation retry issues (#126)
Co-authored-by: Flegma <Flegma@users.noreply.github.com>
1 parent 230296f commit d71b576

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

src/rcon/rcon.service.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,9 @@ export class RconService {
124124
this.genreateCvars(serverId).catch(() => {});
125125
}
126126
} else if (version?.current === true && version?.cvars === false) {
127-
this.genreateCvars(serverId).catch(() => {});
127+
if ((await this.cache.has("cvars")) === false) {
128+
this.genreateCvars(serverId).catch(() => {});
129+
}
128130
}
129131
} catch {
130132
try {
@@ -243,14 +245,14 @@ export class RconService {
243245
return;
244246
}
245247

246-
this.logger.log(`generating cvars for build: ${buildId}`);
247-
248248
const hasLock = await this.aquireCvarsLock(buildId);
249249
if (!hasLock) {
250250
this.logger.warn(`unable to aquire cvars lock for build: ${buildId}`);
251251
return;
252252
}
253253

254+
this.logger.log(`generating cvars for build: ${buildId}`);
255+
254256
try {
255257
const rcon = await this.connect(serverId);
256258
if (!rcon) {
@@ -375,7 +377,7 @@ export class RconService {
375377
const lockKey = `cvars:lock:${buildId}`;
376378
const result = await this.redisManager
377379
.getConnection()
378-
.set(lockKey, 1, "EX", 60, "NX");
380+
.set(lockKey, 1, "EX", 300, "NX");
379381
if (result === null) {
380382
return false;
381383
}

0 commit comments

Comments
 (0)