artin/OBP-API
1
0
Fork 0
mirror of https://github.com/OpenBankProject/OBP-API.git synced 2026-02-06 13:26:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

TODO in RateLimitingUtil.scala

Browse Source
This commit is contained in:
simonredfern 2025-12-27 06:29:30 +01:00
parent b083fb7bb0
commit f63197fe48
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
obp-api/src/main/scala/code/api/util/RateLimitingUtil.scala
View File

@ -151,6 +151,14 @@ object RateLimitingUtil extends MdcLoggable {
(limit) match {
case l if l > 0 => // Redis is available and limit is set
val key = createUniqueKey(consumerKey, period)
// TODO: Check if we can remove redundant EXISTS check. GET returns None when key does not exist.
// Check This would reduce Redis operations from 2 to 1 (25% reduction per request).
// Simplified code:
// val currentValue = Redis.use(JedisMethod.GET, key)
// currentValue match {
// case Some(value) => value.toLong + 1 <= limit
// case None => true // Key does not exist, first call
// }
val exists = Redis.use(JedisMethod.EXISTS,key).map(_.toBoolean).get
exists match {
case true =>