github_mirrors/openappsec
3
0
Fork 1
mirror of https://github.com/openappsec/openappsec.git synced 2025-06-28 16:41:02 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix rate limit string handling

Browse Source
This commit is contained in:
Ned Wright 2023-12-26 13:10:04 +00:00
parent 596033391b
commit d91a1c4ca5
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
components/security_apps/rate_limit/rate_limit.cc
View File

@ -114,7 +114,7 @@ public:
} }
string application_uri = maybe_uri.unpack(); 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()) { for (const auto &rule : rate_limit_config.getRateLimitRules()) {
string full_rule_uri = application_uri + rule.getRateLimitUri(); string full_rule_uri = application_uri + rule.getRateLimitUri();
@ -227,7 +227,7 @@ public:
<< " seconds"; << " seconds";
string unique_key = asset_id + ":" + source_identifier + ":" + uri; 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); auto verdict = decide(unique_key);
if (verdict == RateLimitVedict::ACCEPT) { if (verdict == RateLimitVedict::ACCEPT) {