Skip to content

Commit 6b683b1

Browse files
authored
Merge pull request #139 from dentiny/hjiang/fix-header-inclusion
Fix header inclusion
2 parents 6f7c73f + 76a4467 commit 6b683b1

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

src/include/httpfs_curl_client.hpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
#pragma once
22

3+
#include <curl/curl.h>
4+
35
#include "duckdb/common/http_util.hpp"
46

57
namespace duckdb {
@@ -8,7 +10,6 @@ class FileOpener;
810
struct FileOpenerInfo;
911
class HTTPState;
1012

11-
#include <curl/curl.h>
1213
class CURLHandle {
1314
public:
1415
CURLHandle(const string &token, const string &cert_path);

src/s3fs.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ static HTTPHeaders create_s3_header(string url, string query, string host, strin
8989
if (use_sse_kms) {
9090
signed_headers += ";x-amz-server-side-encryption;x-amz-server-side-encryption-aws-kms-key-id";
9191
}
92-
auto canonical_request = method + "\n" + S3FileSystem::UrlEncode(url) + "\n" + query;
92+
auto canonical_request = method + "\n" + S3FileSystem::UrlEncode(url) + "\n" + query;
9393
if (content_type.length() > 0) {
9494
canonical_request += "\ncontent-type:" + content_type;
9595
}

0 commit comments

Comments
 (0)