From d91a1c4ca5501a76c643e29db92bd18430f34b35 Mon Sep 17 00:00:00 2001 From: Ned Wright Date: Tue, 26 Dec 2023 13:10:04 +0000 Subject: [PATCH] Fix rate limit string handling --- components/security_apps/rate_limit/rate_limit.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/components/security_apps/rate_limit/rate_limit.cc b/components/security_apps/rate_limit/rate_limit.cc index 5ef7aed..2025c9b 100755 --- a/components/security_apps/rate_limit/rate_limit.cc +++ b/components/security_apps/rate_limit/rate_limit.cc @@ -114,7 +114,7 @@ public: } string application_uri = maybe_uri.unpack(); - if (application_uri.back() == '/') application_uri.pop_back(); + if (!application_uri.empty() && application_uri.back() == '/') application_uri.pop_back(); for (const auto &rule : rate_limit_config.getRateLimitRules()) { string full_rule_uri = application_uri + rule.getRateLimitUri(); @@ -227,7 +227,7 @@ public: << " seconds"; string unique_key = asset_id + ":" + source_identifier + ":" + uri; - if (unique_key.back() == '/') unique_key.pop_back(); + if (!unique_key.empty() && unique_key.back() == '/') unique_key.pop_back(); auto verdict = decide(unique_key); if (verdict == RateLimitVedict::ACCEPT) {