Issue workflow progress
Progress of the issue based on the
Contributor Workflow
Describe the bug
The code in response-cache-redis is
if (ttl === Infinity) {
pipeline.set(responseId, JSON.stringify(result));
} else {
// set the ttl in milliseconds
pipeline.set(responseId, JSON.stringify(result), 'PX', ttl);
}
and it clearly set the ttl in milliseconds. The ttl in the response cache is set in seconds though.
To Reproduce Steps to reproduce the behavior:
Expected behavior
Seconds should be used
Environment:
- OS: any
- NodeJS: any
@envelop/* versions: any
Additional context
Issue workflow progress
Progress of the issue based on the
Contributor Workflow
minimal reproduction available on
Stackblitz.
@envelop/*packages that you are using.Describe the bug
The code in response-cache-redis is
and it clearly set the ttl in milliseconds. The ttl in the response cache is set in seconds though.
To Reproduce Steps to reproduce the behavior:
Expected behavior
Seconds should be used
Environment:
@envelop/*versions: any@envelop/core:anyAdditional context