Consult about a logic problem in the code

[zishang520]

When I reviewed the code, the code here:

socket.io-redis-adapter/lib/index.ts

Lines 527 to 533 in ef5f0da

	if (
	!requestId ||
	!(this.requests.has(requestId) || this.ackRequests.has(requestId))
	) {
	debug("ignoring unknown request");
	return;
	}

If requestId does not exist in this.requests, but requestId exists in this.ackRequests, the code will continue to execute:

socket.io-redis-adapter/lib/index.ts

Line 537 in ef5f0da

const request = this.requests.get(requestId);

I'm wondering if there is a problem with this business logic? If there is no problem, is this.ackRequests.has(requestId) a redundant condition?