Check in bytes constructor

rylev · rylev · commit b337273f7d8f · 2023-10-25T19:11:32.000+02:00
Signed-off-by: Ryan Levick &lt;ryan.levick@fermyon.com&gt;
diff --git a/sdk/rust/src/http.rs b/sdk/rust/src/http.rs
@@ -289,7 +289,9 @@ impl HeaderValue {
     /// Construct a `HeaderValue` from a bag of bytes
     pub fn bytes(bytes: Vec<u8>) -> HeaderValue {
         HeaderValue {
-            inner: HeaderValueRep::Bytes(bytes),
+            inner: String::from_utf8(bytes)
+                .map(HeaderValueRep::String)
+                .unwrap_or_else(|e| HeaderValueRep::Bytes(e.into_bytes())),
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -289,7 +289,9 @@ impl HeaderValue {`
`289`	`289`	/// Construct a `HeaderValue` from a bag of bytes
`290`	`290`	`pub fn bytes(bytes: Vec<u8>) -> HeaderValue {`
`291`	`291`	`HeaderValue {`
`292`		`- inner: HeaderValueRep::Bytes(bytes),`
	`292`	`+ inner: String::from_utf8(bytes)`
	`293`	`+ .map(HeaderValueRep::String)`
	`294`	`+ .unwrap_or_else(\|e\| HeaderValueRep::Bytes(e.into_bytes())),`
`293`	`295`	`}`
`294`	`296`	`}`
`295`	`297`