Skip to content

[rust] Modified api.mustache to ensure correct handling of Vec<PathBuf> while adding files to multipart form #22942

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
wing328 merged 3 commits into from
Feb 12, 2026
+87 −39
Merged

[rust] Modified api.mustache to ensure correct handling of Vec<PathBuf> while adding files to multipart form #22942

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
fc2295b
Fixes #22900
MercMayhem Feb 11, 2026
10160b2
fixed non async block generation when Vec<PathBuf> (files) are present
MercMayhem Feb 11, 2026
76bd2e5
updated samples with reqwest multipart form change
MercMayhem Feb 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
56 changes: 52 additions & 4 deletions modules/openapi-generator/src/main/resources/rust/reqwest/api.mustache
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -404,37 +404,85 @@ pub {{#supportAsync}}async {{/supportAsync}}fn {{{operationId}}}(configuration:
{{^supportAsync}}
{{#required}}
{{^isNullable}}
{{#isArray}}
for value in &{{{vendorExtensions.x-rust-param-identifier}}} {
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot Feb 11, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2: Async multipart handling for required nullable file arrays iterates over &Option<Vec<PathBuf>> without unwrapping, producing &Vec<PathBuf> loop values that cannot be opened as paths; this causes compile errors for required nullable file arrays.

Prompt for AI agents 
Check if this issue is valid — if so, understand the root cause and fix it. At modules/openapi-generator/src/main/resources/rust/reqwest/api.mustache, line 408:

<comment>Async multipart handling for required nullable file arrays iterates over `&Option<Vec<PathBuf>>` without unwrapping, producing `&Vec<PathBuf>` loop values that cannot be opened as paths; this causes compile errors for required nullable file arrays.</comment>

<file context>
@@ -404,37 +404,85 @@ pub {{#supportAsync}}async {{/supportAsync}}fn {{{operationId}}}(configuration:
     {{#required}}
     {{^isNullable}}
+    {{#isArray}}
+    for value in &{{{vendorExtensions.x-rust-param-identifier}}} {
+        multipart_form = multipart_form.file("{{{baseName}}}", value)?;
+    }
</file context>
Fix with Cubic

multipart_form = multipart_form.file("{{{baseName}}}", value)?;
}
{{/isArray}}
{{^isArray}}
multipart_form = multipart_form.file("{{{baseName}}}", {{{vendorExtensions.x-rust-param-identifier}}})?;
{{/isArray}}
{{/isNullable}}
{{#isNullable}}
{{#isArray}}
match {{{vendorExtensions.x-rust-param-identifier}}} {
Some(param_value) => {
for value in param_value {
multipart_form = multipart_form.file("{{{baseName}}}", value)?;
}
},
None => { unimplemented!("Required nullable form file param not supported"); },
}
{{/isArray}}
{{^isArray}}
match {{{vendorExtensions.x-rust-param-identifier}}} {
Some(param_value) => { multipart_form = multipart_form.file("{{{baseName}}}", param_value)?; },
None => { unimplemented!("Required nullable form file param not supported"); },
}
{{/isArray}}
{{/isNullable}}
{{/required}}
{{^required}}
if let Some(ref param_value) = {{{vendorExtensions.x-rust-param-identifier}}} {
{{#isArray}}
for value in param_value {
multipart_form = multipart_form.file("{{{baseName}}}", value)?;
}
{{/isArray}}
{{^isArray}}
multipart_form = multipart_form.file("{{{baseName}}}", param_value)?;
{{/isArray}}
}
{{/required}}
{{/supportAsync}}
{{#supportAsync}}
{{^required}}
if let Some(ref param_value) = {{{vendorExtensions.x-rust-param-identifier}}} {
let file = TokioFile::open(param_value).await?;
{{#isArray}}
for value in param_value {
let file = TokioFile::open(value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("{{{baseName}}}", file_part);
}
{{/isArray}}
{{^isArray}}
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("{{{baseName}}}", file_part);
{{/isArray}}
}
{{/required}}
{{#required}}
{{#isArray}}
for value in &{{{vendorExtensions.x-rust-param-identifier}}} {
let file = TokioFile::open(value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_name = value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("{{{baseName}}}", file_part);
}
{{/required}}
{{#required}}
{{/isArray}}
{{^isArray}}
let file = TokioFile::open(&{{{vendorExtensions.x-rust-param-identifier}}}).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = {{{vendorExtensions.x-rust-param-identifier}}}.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("{{{baseName}}}", file_part);
{{/isArray}}
{{/required}}
{{/supportAsync}}
{{/isFile}}
Expand Down
20 changes: 10 additions & 10 deletions samples/client/others/rust/reqwest/multipart-async/src/apis/default_api.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,11 +94,11 @@ pub async fn upload_multiple_fields(configuration: &configuration::Configuration
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("primaryFile", file_part);
if let Some(ref param_value) = params.thumbnail {
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("thumbnail", file_part);
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("thumbnail", file_part);
}
req_builder = req_builder.multipart(multipart_form);

Expand Down Expand Up @@ -141,11 +141,11 @@ pub async fn upload_optional_file(configuration: &configuration::Configuration,
multipart_form = multipart_form.text("metadata", param_value.to_string());
}
if let Some(ref param_value) = params.file {
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
}
req_builder = req_builder.multipart(multipart_form);

Expand Down
10 changes: 5 additions & 5 deletions samples/client/petstore/rust/reqwest/petstore-async-middleware/src/apis/pet_api.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -570,11 +570,11 @@ pub async fn upload_file(configuration: &configuration::Configuration, params: U
multipart_form = multipart_form.text("additionalMetadata", param_value.to_string());
}
if let Some(ref param_value) = params.file {
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
}
req_builder = req_builder.multipart(multipart_form);

Expand Down
10 changes: 5 additions & 5 deletions samples/client/petstore/rust/reqwest/petstore-async-tokensource/src/apis/pet_api.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -581,11 +581,11 @@ pub async fn upload_file(configuration: &configuration::Configuration, params: U
multipart_form = multipart_form.text("additionalMetadata", param_value.to_string());
}
if let Some(ref param_value) = params.file {
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
}
req_builder = req_builder.multipart(multipart_form);

Expand Down
10 changes: 5 additions & 5 deletions samples/client/petstore/rust/reqwest/petstore-async/src/apis/pet_api.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -570,11 +570,11 @@ pub async fn upload_file(configuration: &configuration::Configuration, params: U
multipart_form = multipart_form.text("additionalMetadata", param_value.to_string());
}
if let Some(ref param_value) = params.file {
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
}
req_builder = req_builder.multipart(multipart_form);

Expand Down
10 changes: 5 additions & 5 deletions samples/client/petstore/rust/reqwest/petstore-avoid-box/src/apis/pet_api.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -570,11 +570,11 @@ pub async fn upload_file(configuration: &configuration::Configuration, params: U
multipart_form = multipart_form.text("additionalMetadata", param_value.to_string());
}
if let Some(ref param_value) = params.file {
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
}
req_builder = req_builder.multipart(multipart_form);

Expand Down
10 changes: 5 additions & 5 deletions samples/client/petstore/rust/reqwest/petstore-serde-path-to-error/src/apis/pet_api.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -496,11 +496,11 @@ pub async fn upload_file(configuration: &configuration::Configuration, pet_id: i
multipart_form = multipart_form.text("additionalMetadata", param_value.to_string());
}
if let Some(ref param_value) = p_form_file {
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
let file = TokioFile::open(param_value).await?;
let stream = FramedRead::new(file, BytesCodec::new());
let file_name = param_value.file_name().map(|n| n.to_string_lossy().to_string()).unwrap_or_default();
let file_part = reqwest::multipart::Part::stream(reqwest::Body::wrap_stream(stream)).file_name(file_name);
multipart_form = multipart_form.part("file", file_part);
}
req_builder = req_builder.multipart(multipart_form);

Expand Down
Loading