remove unused file_name_header field from Body::File

ducaale · Jun 17, 2024 · ca629c8 · ca629c8
1 parent 7841303
commit ca629c8
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 8 deletions.
diff --git a/src/main.rs b/src/main.rs
@@ -418,10 +418,6 @@ fn run(args: Cli) -> Result<i32> {
             Body::File {
                 file_name,
                 file_type,
-                // We could turn this into a Content-Disposition header, but
-                // that has no effect, so just ignore it
-                // (Additional precedent: HTTPie ignores file_type here)
-                file_name_header: _,
             } => request_builder.body(File::open(file_name)?).header(
                 CONTENT_TYPE,
                 file_type.unwrap_or_else(|| HeaderValue::from_static(JSON_CONTENT_TYPE)),

diff --git a/src/request_items.rs b/src/request_items.rs
@@ -220,7 +220,6 @@ pub enum Body {
     File {
         file_name: PathBuf,
         file_type: Option<HeaderValue>,
-        file_name_header: Option<String>,
     },
 }
 
@@ -423,7 +422,7 @@ impl RequestItems {
                     key,
                     file_name,
                     file_type,
-                    file_name_header,
+                    file_name_header: _,
                 } => {
                     assert!(key.is_empty());
                     if body.is_some() {
@@ -436,7 +435,6 @@ impl RequestItems {
                             .map(HeaderValue::from_str)
                             .transpose()?,
                         file_name: expand_tilde(file_name),
-                        file_name_header,
                     });
                 }
                 RequestItem::HttpHeader(..)

diff --git a/src/to_curl.rs b/src/to_curl.rs
@@ -428,7 +428,6 @@ pub fn translate(args: Cli) -> Result<Command> {
             Body::File {
                 file_name,
                 file_type,
-                file_name_header: _,
             } => {
                 if let Some(file_type) = file_type {
                     cmd.header("content-type", file_type.to_str()?);