wsd: extract WOPI HTTP request construction

This hoists the common parts of the HTTPRequest
for all WOPI requests to avoid errors when changing them.

Change-Id: Ia02ef657a43b7a7d2fc13be3da012836fa0d7650
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/96372
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Tested-by: Jenkins
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>

1 files changed, 6 insertions, 0 deletions

diff --git a/wsd/Storage.hpp b/wsd/Storage.hpp
index b426d44bd..d7f6d70f9 100644
--- a/wsd/Storage.hpp
+++ b/wsd/Storage.hpp
@@ -505,6 +505,12 @@ public:
     std::chrono::duration<double> getWopiSaveDuration() const { return _wopiSaveDuration; }
 
 private:
+    /// Initialize an HTTPRequest instance with the common settings and headers.
+    /// Older Poco versions don't support copying HTTPRequest objects, so we can't generate them.
+    void initHttpRequest(Poco::Net::HTTPRequest& request, const Poco::URI& uri,
+                         const Authorization& auth, const std::string& cookies) const;
+
+private:
     // Time spend in loading the file from storage
     std::chrono::duration<double> _wopiLoadDuration;
     std::chrono::duration<double> _wopiSaveDuration;