Is this a bug about this.del function

[songjinzhong]

node-cache/index.js

Line 53 in 27a1eee

if (!isNaN(oldRecord.expire) && oldRecord.expire < Date.now()) {

When a record is about to expire, we take the initiative to delete it. And due to the running time of the program, oldRecord.expire < Date.now() === true, so the record will not be deleted.

[AsynchronousToddler]

Yeah, that seems silly to me. If you’re manually invoking a delete function then jr shouldn’t matter wether or not the record is expiring. It should delete it regardless

[IvanAdmaers]

I think it’s an interesting idea to have a method that won't delete a key if it isn't expired. But it’s really strange when you don't have an opportunity to delete something that doesn’t expired. In my similar package memory-cache-pro you can use the ‘delete(key)’ method to delete something and it doesn’t check the key expiration time