BlockStorage/repertory
1
0
Fork 0
Code Issues 7 Pull Requests Releases 6 Wiki Activity

[bug] S3 provider should limit max key size to 1024 #31

Browse Source
This commit is contained in:
Scott E. Graves 2024-12-31 12:55:52 -06:00
parent 75d2d39e7c
commit 81712e7a99
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
repertory/librepertory/include/types/repertory.hpp
View File

@ -41,6 +41,7 @@ constexpr const auto default_task_wait_ms{100U};
constexpr const auto default_timeout_ms{60000U};
constexpr const auto max_orphaned_file_retention_days{std::uint16_t(31U)};
constexpr const auto max_ring_buffer_file_size{std::uint16_t(1024U)};
constexpr const auto max_s3_object_name_length{1024U};
constexpr const auto min_cache_size_bytes{
std::uint64_t(100UL * 1024UL * 1024UL)};
constexpr const auto min_download_timeout_secs{std::uint8_t(5U)};

2
repertory/librepertory/src/providers/s3/s3_provider.cpp
View File

@ -44,7 +44,7 @@ namespace {
const std::string &object_name)
-> repertory::api_error {
request.path = object_name;
if (request.path.substr(1U).size() > 1024U) {
if (request.path.substr(1U).size() > max_s3_object_name_length) {
return repertory::api_error::name_too_long;
}