mirror/axum-htmx
1
0
Fork 0
mirror of https://github.com/robertwayne/axum-htmx synced 2024-12-27 02:39:32 +01:00
Code Issues Projects Releases Packages Wiki Activity

Remove unnecessary .to_string() call when creating a header from JSON data

Browse source 
The original version would double quote values, resulting in serde_json inserting
escape characters into the final string value, which prevented the header from
being parsed by htmx properly.
This commit is contained in:
Rob Wagner 2023-10-23 22:52:01 -04:00
parent 81b9c548b3
commit a1bac4b30e
No known key found for this signature in database
GPG key ID: 53CCB4497B15CF61
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/responders/serde.rs
View file

@ -193,8 +193,9 @@ pub(crate) fn events_to_header_value(events: Vec<HxEvent>) -> Result<HeaderValue
// encoded. // encoded.
let header_value = events let header_value = events
.into_iter() .into_iter()
.map(|e| (e.name, e.data.map(|d| d.to_string()).unwrap_or_default())) .map(|e| (e.name, e.data.unwrap_or_default()))
.collect::<HashMap<_, _>>(); .collect::<HashMap<String, Value>>();
serde_json::to_string(&header_value)? serde_json::to_string(&header_value)?
} else { } else {
// no event contains data, the event names can be put in the header // no event contains data, the event names can be put in the header