IRedisCollection<T>.UpdateAsync does not set the TTL value

[delepster]

When using the UpdateAsync method to update an entry, the TTL gets updated to no limit even if I pass a TTL value. As a workaround, I need to delete the entry and recreate it:

await this._sessionCollection.UpdateAsync(new CachedServerSideSession(session), session.Expires - DateTime.UtcNow ?? TimeSpan.Zero);

TTL is set to no limit.

await this.DeleteSessionAsync(session.Key, cancellationToken);
await this.CreateSessionAsync(session, cancellationToken);

TTL has the right value.

[slorello89]

Hi @delepster - I'm running this test - and it seems to be sending the right thing:

1752242152.524145 [0 lua] "JSON.SET" "Redis.OM.Unit.Tests.RediSearchTests.Person:01JZWWAHXCC8H6PA4ZQXBCQSCD" "$.Age" "33"
1752242152.524615 [0 lua] "PEXPIRE" "Redis.OM.Unit.Tests.RediSearchTests.Person:01JZWWAHXCC8H6PA4ZQXBCQSCD" "3600000

• are you sure you're not sending something strange? - I'd think that if session.Expires is null in this case it would throw an error. Can I suggest moving the timespan computation outside the method call and logging it to see what it's passing, you might also consider checking what Redis is seeing with the MONITOR command.

[delepster]

I'll try that

[delepster]

The session.Expires value is not null and the TTL is overriden and is now no limit

[slorello89]

Can you share a minimal reproduction of this issue, along with a run of what it's sending from the monitor is the redis-cli?

[delepster]

Here is the value of the TTL I pass (just under 14 days). The use case is only to update an entry and at the same time, reset the TTL.

The DeleteSession merely just calls DeleteAsync and CreateSession calls InsertAsync with the same TTL value as for the update.

Also, I am not sure what you mean by:

sending from the monitor is the redis-cli

[delepster]

Another thing is when the TTL is set to no limit, the model gets corrupted:

it goes from something like this:

to

Since I am caching a model from an external library, I needed to derive the class so it becomes a [Document] along with the properties that need to be [Indexed]. Weird enough is the PEXPIRE becoming a property instead of the actual TTL value. Here is the update command:

var ttl = session.Expires - DateTime.UtcNow ?? TimeSpan.Zero;
await this._sessionCollection.UpdateAsync(new CachedServerSideSession(session), ttl);