fix: bug fixes (64bit#104)

* fix: request creation failures should be bubbled up

* ISSUE 103: Serialize None to null for content field in ChatCompletionRequestMessage

Author: 64bit

async-openai/src/client.rs

@@ -197,8 +197,9 @@ impl<C: Config> Client<C> {
         let client = self.http_client.clone();
 
         backoff::future::retry(self.backoff.clone(), || async {
+            let request = request_maker().await.map_err(backoff::Error::Permanent)?;
             let response = client
-                .execute(request_maker().await?)
+                .execute(request)
                 .await
                 .map_err(OpenAIError::Reqwest)
                 .map_err(backoff::Error::Permanent)?;

async-openai/src/types/types.rs

@@ -723,7 +723,8 @@ pub struct ChatCompletionRequestMessage {
     pub role: Role,
     /// The contents of the message.
     /// `content` is required for all messages except assistant messages with function calls.
-    #[serde(skip_serializing_if = "Option::is_none")]
+    // DON'T SKIP: https://github.com/64bit/async-openai/issues/103
+    //#[serde(skip_serializing_if = "Option::is_none")]
     pub content: Option<String>,
     /// The name of the author of this message. `name` is required if role is function,
     /// and it should be the name of the function whose response is in the `content`.