fix bug with empty body

Signed-off-by: Justin Pflueger <justin.pflueger@fermyon.com>
jpflueger · Mar 28, 2023 · 1ba697f · 1ba697f
1 parent 531cac4
commit 1ba697f
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/scripts/test-auth0.sh b/scripts/test-auth0.sh
@@ -2,6 +2,8 @@
 
 source .envrc
 
+# set -x
+
 AUTH_BODY=$(jq --null-input \
   --arg client_id "${AUTH0_CLIENT_ID}" \
   --arg client_secret "${AUTH0_CLIENT_SECRET}" \

diff --git a/src/config.rs b/src/config.rs
@@ -28,6 +28,7 @@ impl Default for Config {
     fn default() -> Self {
         let jwks_uri = config::get(JWKS_URI).expect("Missing required config item 'jwks_uri'");
 
+        //TODO: add logging for parse errors
         let issuers = config_get_set(ISSUERS).ok();
         let audiences = config_get_set(AUDIENCES).ok();
         let max_validity: Option<Duration> = config_get_duration(MAX_VALIDITY_SECS).ok();

diff --git a/src/request.rs b/src/request.rs
@@ -12,8 +12,9 @@ impl TryInto<VerificationRequest> for Option<&Bytes> {
     type Error = anyhow::Error;
     fn try_into(self) -> Result<VerificationRequest, Self::Error> {
         match self {
-            Some(b) => serde_json::from_slice::<VerificationRequest>(b).map_err(|e| e.into()),
             None => Ok(Default::default()),
+            Some(b) if b.len() == 0 => Ok(Default::default()),
+            Some(b) => serde_json::from_slice::<VerificationRequest>(b).map_err(|e| e.into()),
         }
     }
 }