ByteRange::resolve()

src/server/byte_range.cpp

@@ -62,4 +62,18 @@ ByteRange ByteRange::parse(std::string rangeStr)
   return byteRange;
 }
 
+ByteRange ByteRange::resolve(int64_t contentSize) const
+{
+  if ( kind() == NONE )
+    return ByteRange(RESOLVED_FULL_CONTENT, 0, contentSize-1);
+
+  const int64_t resolved_first = first() < 0
+                               ? std::max(int64_t(0), contentSize + first())
+                               : first();
+
+  const int64_t resolved_last = std::min(contentSize-1, last());
+
+  return ByteRange(RESOLVED_PARTIAL_CONTENT, resolved_first, resolved_last);
+}
+
 } // namespace kiwix

src/server/byte_range.h

@@ -53,6 +53,7 @@ class ByteRange
     int64_t length() const { return last_ + 1 - first_; }
 
     static ByteRange parse(std::string rangeStr);
+    ByteRange resolve(int64_t contentSize) const;
 
   private: // data
     Kind kind_;

src/server/response.cpp

@@ -39,21 +39,6 @@ bool is_compressible_mime_type(const std::string& mimeType)
       || mimeType.find("application/json") != string::npos;
 }
 
-ByteRange resolve_byte_range(const kiwix::Entry& entry, ByteRange range)
-{
-  const int64_t entrySize = entry.getSize();
-
-  if ( range.kind() == ByteRange::NONE )
-    return ByteRange(ByteRange::RESOLVED_FULL_CONTENT, 0, entrySize-1);
-
-  const int64_t resolved_first = range.first() < 0
-                               ? std::max(int64_t(0), entrySize + range.first())
-                               : range.first();
-
-  const int64_t resolved_last = std::min(entrySize-1, range.last());
-
-  return ByteRange(ByteRange::RESOLVED_PARTIAL_CONTENT, resolved_first, resolved_last);
-}
 
 } // unnamed namespace
 
@@ -338,7 +323,7 @@ void Response::set_entry(const Entry& entry, const RequestContext& request) {
     set_content(content);
     set_compress(true);
   } else {
-    m_byteRange = resolve_byte_range(entry, request.get_range());
+    m_byteRange = request.get_range().resolve(entry.getSize());
   }
 }