diff options
| author | Dan Engelbrecht <[email protected]> | 2022-06-08 11:20:18 +0200 |
|---|---|---|
| committer | GitHub <[email protected]> | 2022-06-08 11:20:18 +0200 |
| commit | 5faa4ecce9b59d46f617eeb693c50e118e7ff6fb (patch) | |
| tree | a9d846c38fb864a14c2c15b961f39e5839be8ea2 /zenserver/cache/structuredcache.cpp | |
| parent | Merge pull request #123 from EpicGames/de/fix-gc-over-multiple-namespaces (diff) | |
| parent | Merge branch 'main' into de/add-log-for-invalid-names (diff) | |
| download | zen-5faa4ecce9b59d46f617eeb693c50e118e7ff6fb.tar.xz zen-5faa4ecce9b59d46f617eeb693c50e118e7ff6fb.zip | |
Merge pull request #125 from EpicGames/de/add-log-for-invalid-names
add warnings on illegal namespace/bucket names
Diffstat (limited to 'zenserver/cache/structuredcache.cpp')
| -rw-r--r-- | zenserver/cache/structuredcache.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/zenserver/cache/structuredcache.cpp b/zenserver/cache/structuredcache.cpp index 79d15a204..4590df48b 100644 --- a/zenserver/cache/structuredcache.cpp +++ b/zenserver/cache/structuredcache.cpp @@ -94,15 +94,18 @@ namespace { { if (Name.empty()) { + ZEN_WARN("Namspace is invalid, empty namespace is not allowed"); return {}; } if (Name.length() > 64) { + ZEN_WARN("Namspace '{}' is invalid, length exceeds 64 charaters", Name); return {}; } if (Name.find_first_not_of(ValidNamespaceNameCharacters) != std::string::npos) { + ZEN_WARN("Namspace '{}' is invalid, invalid characters detected", Name); return {}; } return ToLower(Name); @@ -112,10 +115,12 @@ namespace { { if (Name.empty()) { + ZEN_WARN("Bucket name is invalid, empty bucket name is not allowed"); return {}; } if (Name.find_first_not_of(ValidBucketNameCharacters) != std::string::npos) { + ZEN_WARN("Bucket name '{}' is invalid, invalid characters detected", Name); return {}; } return ToLower(Name); |