Merge pull request duckdb#13 from dentiny/hjiang/populate-http-status

Mytherin · web-flow · commit e21cc3968dc9 · 2025-02-04T19:24:33.000+01:00
[Easy] Populate http status to exception
diff --git a/extension/httpfs/httpfs.cpp b/extension/httpfs/httpfs.cpp
@@ -181,7 +181,7 @@ HTTPFileSystem::RunRequestWithRetry(const std::function<duckdb_httplib_openssl::
 			} else if (err == duckdb_httplib_openssl::Error::Success) {
 				throw HTTPException(response, "Request returned HTTP %d for HTTP %s to '%s'", status, method, url);
 			} else {
-				throw IOException("%s error for HTTP %s to '%s'", to_string(err), method, url);
+				throw IOException("%s error for HTTP %s to '%s' with status %d", to_string(err), method, url, status);
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,7 @@ HTTPFileSystem::RunRequestWithRetry(const std::function<duckdb_httplib_openssl::`
`181`	`181`	`} else if (err == duckdb_httplib_openssl::Error::Success) {`
`182`	`182`	`throw HTTPException(response, "Request returned HTTP %d for HTTP %s to '%s'", status, method, url);`
`183`	`183`	`} else {`
`184`		`- throw IOException("%s error for HTTP %s to '%s'", to_string(err), method, url);`
	`184`	`+ throw IOException("%s error for HTTP %s to '%s' with status %d", to_string(err), method, url, status);`
`185`	`185`	`}`
`186`	`186`	`}`
`187`	`187`	`}`