Skip to content

Commit 18ac9eb

Browse files
authored
fix: only delete endpoint which has expired in EndpointCache (#3752)
1 parent 9b68a90 commit 18ac9eb

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"type": "bugfix",
3+
"category": "EndpointCache",
4+
"description": "Only delete endpoint which has expired in EndpointCache"
5+
}

vendor/endpoint-cache/index.js

+6-3
Original file line numberDiff line numberDiff line change
@@ -29,13 +29,16 @@ var EndpointCache = /** @class */ (function () {
2929
var now = Date.now();
3030
var records = this.cache.get(keyString);
3131
if (records) {
32-
for (var i = 0; i < records.length; i++) {
32+
for (var i = records.length-1; i >= 0; i--) {
3333
var record = records[i];
3434
if (record.Expire < now) {
35-
this.cache.remove(keyString);
36-
return undefined;
35+
records.splice(i, 1);
3736
}
3837
}
38+
if (records.length === 0) {
39+
this.cache.remove(keyString);
40+
return undefined;
41+
}
3942
}
4043
return records;
4144
};

0 commit comments

Comments
 (0)