Empty post bodies (#2050)

* Cleaning optional post bodies. Fixes #2039

* Only trim once.

* Using .map() instead.

* Revert "Using .map() instead."

This reverts commit a2f4907209.
This commit is contained in:
Dessalines 2022-01-21 08:38:01 -05:00 committed by GitHub
parent f23fed70bc
commit 16271b0a4c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -176,7 +176,16 @@ pub fn clean_url_params(mut url: Url) -> Url {
} }
pub fn clean_optional_text(text: &Option<String>) -> Option<String> { pub fn clean_optional_text(text: &Option<String>) -> Option<String> {
text.as_ref().map(|t| t.trim().to_string()) if let Some(text) = text {
let trimmed = text.trim();
if trimmed.is_empty() {
None
} else {
Some(trimmed.to_owned())
}
} else {
None
}
} }
#[cfg(test)] #[cfg(test)]