Skip to content

Commit

Permalink
HttpRequest: Add support for sending custom headers
Browse files Browse the repository at this point in the history
  • Loading branch information
leoetlino committed Jun 13, 2017
1 parent 8f87433 commit ba3f16e
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 12 deletions.
37 changes: 28 additions & 9 deletions Source/Core/Common/HttpRequest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
#include <curl/curl.h>

#include "Common/Logging/Log.h"
#include "Common/ScopeGuard.h"
#include "Common/StringUtil.h"

namespace Common
{
Expand All @@ -23,7 +25,8 @@ class HttpRequest::Impl final
Impl();

bool IsValid() const;
Response Fetch(const std::string& url, Method method, const u8* payload, size_t size);
Response Fetch(const std::string& url, Method method, const Headers& headers, const u8* payload,
size_t size);

private:
std::unique_ptr<CURL, decltype(&curl_easy_cleanup)> m_curl{curl_easy_init(), curl_easy_cleanup};
Expand All @@ -40,20 +43,22 @@ bool HttpRequest::IsValid() const
return m_impl->IsValid();
}

HttpRequest::Response HttpRequest::Get(const std::string& url)
HttpRequest::Response HttpRequest::Get(const std::string& url, const Headers& headers)
{
return m_impl->Fetch(url, Impl::Method::GET, nullptr, 0);
return m_impl->Fetch(url, Impl::Method::GET, headers, nullptr, 0);
}

HttpRequest::Response HttpRequest::Post(const std::string& url, const std::vector<u8>& payload)
HttpRequest::Response HttpRequest::Post(const std::string& url, const std::vector<u8>& payload,
const Headers& headers)
{
return m_impl->Fetch(url, Impl::Method::POST, payload.data(), payload.size());
return m_impl->Fetch(url, Impl::Method::POST, headers, payload.data(), payload.size());
}

HttpRequest::Response HttpRequest::Post(const std::string& url, const std::string& payload)
HttpRequest::Response HttpRequest::Post(const std::string& url, const std::string& payload,
const Headers& headers)
{
return m_impl->Fetch(url, Impl::Method::POST, reinterpret_cast<const u8*>(payload.data()),
payload.size());
return m_impl->Fetch(url, Impl::Method::POST, headers,
reinterpret_cast<const u8*>(payload.data()), payload.size());
}

HttpRequest::Impl::Impl()
Expand Down Expand Up @@ -85,7 +90,8 @@ static size_t CurlCallback(char* data, size_t size, size_t nmemb, void* userdata
}

HttpRequest::Response HttpRequest::Impl::Fetch(const std::string& url, Method method,
const u8* payload, size_t size)
const Headers& headers, const u8* payload,
size_t size)
{
curl_easy_setopt(m_curl.get(), CURLOPT_POST, method == Method::POST);
curl_easy_setopt(m_curl.get(), CURLOPT_URL, url.c_str());
Expand All @@ -95,6 +101,19 @@ HttpRequest::Response HttpRequest::Impl::Fetch(const std::string& url, Method me
curl_easy_setopt(m_curl.get(), CURLOPT_POSTFIELDSIZE, size);
}

curl_slist* list = nullptr;
Common::ScopeGuard list_guard{[&list] { curl_slist_free_all(list); }};
for (const std::pair<std::string, std::optional<std::string>>& header : headers)
{
if (!header.second)
list = curl_slist_append(list, (header.first + ":").c_str());
else if (header.second->empty())
list = curl_slist_append(list, (header.first + ";").c_str());
else
list = curl_slist_append(list, (header.first + ": " + *header.second).c_str());
}
curl_easy_setopt(m_curl.get(), CURLOPT_HTTPHEADER, list);

std::vector<u8> buffer;
curl_easy_setopt(m_curl.get(), CURLOPT_WRITEFUNCTION, CurlCallback);
curl_easy_setopt(m_curl.get(), CURLOPT_WRITEDATA, &buffer);
Expand Down
9 changes: 6 additions & 3 deletions Source/Core/Common/HttpRequest.h
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

#pragma once

#include <map>
#include <memory>
#include <optional>
#include <string>
Expand All @@ -21,9 +22,11 @@ class HttpRequest final
bool IsValid() const;

using Response = std::optional<std::vector<u8>>;
Response Get(const std::string& url);
Response Post(const std::string& url, const std::vector<u8>& payload);
Response Post(const std::string& url, const std::string& payload);
using Headers = std::map<std::string, std::optional<std::string>>;
Response Get(const std::string& url, const Headers& headers = {});
Response Post(const std::string& url, const std::vector<u8>& payload,
const Headers& headers = {});
Response Post(const std::string& url, const std::string& payload, const Headers& headers = {});

private:
class Impl;
Expand Down

0 comments on commit ba3f16e

Please sign in to comment.