aboutsummaryrefslogtreecommitdiff
path: root/zenserver/projectstore.cpp
diff options
context:
space:
mode:
authorStefan Boberg <[email protected]>2021-10-13 21:25:52 +0200
committerStefan Boberg <[email protected]>2021-10-13 21:25:52 +0200
commit52e3a11d0eda8a131c449173857cdd7c1a3d95cd (patch)
treea9c82c0229e32d37a49c58ec5a4ef7f567d0c8c0 /zenserver/projectstore.cpp
parentMerge branch 'main' of https://github.com/EpicGames/zen (diff)
downloadzen-52e3a11d0eda8a131c449173857cdd7c1a3d95cd.tar.xz
zen-52e3a11d0eda8a131c449173857cdd7c1a3d95cd.zip
projectstore: Removing support for requesting attachments by hash using offset/length (not needed and we will remove this path entirely)
Diffstat (limited to 'zenserver/projectstore.cpp')
-rw-r--r--zenserver/projectstore.cpp60
1 files changed, 5 insertions, 55 deletions
diff --git a/zenserver/projectstore.cpp b/zenserver/projectstore.cpp
index f14bc7896..39348b436 100644
--- a/zenserver/projectstore.cpp
+++ b/zenserver/projectstore.cpp
@@ -1232,66 +1232,16 @@ HttpProjectService::HttpProjectService(CasStore& Store, ProjectStore* Projects)
const auto& OplogId = Req.GetCapture(2);
const auto& HashString = Req.GetCapture(3);
- bool IsOffset = false;
- uint64_t Offset = 0;
- uint64_t Size = ~(0ull);
-
- auto QueryParms = Req.ServerRequest().GetQueryParams();
-
- if (auto OffsetParm = QueryParms.GetValue("offset"); OffsetParm.empty() == false)
- {
- if (auto OffsetVal = ParseInt<uint64_t>(OffsetParm))
- {
- Offset = OffsetVal.value();
- IsOffset = true;
- }
- else
- {
- return HttpReq.WriteResponse(HttpResponseCode::BadRequest);
- }
- }
-
- if (auto SizeParm = QueryParms.GetValue("size"); SizeParm.empty() == false)
- {
- if (auto SizeVal = ParseInt<uint64_t>(SizeParm))
- {
- Size = SizeVal.value();
- IsOffset = true;
- }
- else
- {
- return HttpReq.WriteResponse(HttpResponseCode::BadRequest);
- }
- }
-
ZEN_DEBUG("oplog hash - {} / {} / {}", ProjectId, OplogId, HashString);
- IoHash Hash = IoHash::FromHexString(HashString);
- IoBuffer Value = m_CasStore.FindChunk(Hash);
- if (!Value)
- {
- return HttpReq.WriteResponse(HttpResponseCode::NotFound);
- }
+ IoHash Hash = IoHash::FromHexString(HashString);
- if (IsOffset)
+ if (IoBuffer Value = m_CasStore.FindChunk(Hash))
{
- if (Offset > Value.Size())
- {
- Offset = Value.Size();
- }
-
- if ((Offset + Size) > Value.Size())
- {
- Size = Value.Size() - Offset;
- }
-
- // Send only a subset of data
- IoBuffer InnerValue(Value, Offset, Size);
-
- return HttpReq.WriteResponse(HttpResponseCode::OK, HttpContentType::kBinary, InnerValue);
+ return HttpReq.WriteResponse(HttpResponseCode::OK, HttpContentType::kBinary, Value);
}
- return HttpReq.WriteResponse(HttpResponseCode::OK, HttpContentType::kBinary, Value);
+ return HttpReq.WriteResponse(HttpResponseCode::NotFound);
},
HttpVerb::kGet);
@@ -1325,7 +1275,7 @@ HttpProjectService::HttpProjectService(CasStore& Store, ProjectStore* Projects)
if (!m_CasStore.FindChunk(FileHash))
{
- ZEN_DEBUG("NEED: {}", FileHash);
+ ZEN_DEBUG("prep - NEED: {}", FileHash);
NeedList.push_back(FileHash);
}