From 33077299086f85646f0f8456fb4d3c874c738841 Mon Sep 17 00:00:00 2001 From: Jon Carstens Date: Tue, 11 Nov 2025 16:38:38 -0700 Subject: [PATCH] Support location output parsed as CSV Newer versions of Req will attempt to decode the body based on the path name if the content-type is `application/octet-stream`. Since this is being hardcoded to `*-manifest.csv`, that means it will be decoded and fail the `String` function calls that were previously used in `ReqAthena.S3.get_locations/2`. This forces `decode_body: false` for this specific request since it is expected to only be a newline delineated string --- lib/req_athena/s3.ex | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/req_athena/s3.ex b/lib/req_athena/s3.ex index daa76f7..51fb815 100644 --- a/lib/req_athena/s3.ex +++ b/lib/req_athena/s3.ex @@ -8,6 +8,7 @@ defmodule ReqAthena.S3 do manifest_csv_location = output_location <> "-manifest.csv" req_s3 + |> Req.merge(decode_body: false) |> get_body(manifest_csv_location) |> String.trim() |> String.split("\n")