graph: Make error checking for EnvVarsMapping possible

Author: lutter

graph/src/env/mappings.rs

@@ -82,9 +82,11 @@ impl fmt::Debug for EnvVarsMapping {
     }
 }
 
-impl From<InnerMappingHandlers> for EnvVarsMapping {
-    fn from(x: InnerMappingHandlers) -> Self {
-        Self {
+impl TryFrom<InnerMappingHandlers> for EnvVarsMapping {
+    type Error = anyhow::Error;
+
+    fn try_from(x: InnerMappingHandlers) -> Result<Self, Self::Error> {
+        let vars = Self {
             entity_cache_dead_weight: x.entity_cache_dead_weight.0,
             entity_cache_size: x.entity_cache_size_in_kb * 1000,
 
@@ -102,7 +104,8 @@ impl From<InnerMappingHandlers> for EnvVarsMapping {
             allow_non_deterministic_ipfs: x.allow_non_deterministic_ipfs.0,
             disable_declared_calls: x.disable_declared_calls.0,
             store_errors_are_nondeterministic: x.store_errors_are_nondeterministic.0,
-        }
+        };
+        Ok(vars)
     }
 }
 

graph/src/env/mod.rs

@@ -268,7 +268,7 @@ impl EnvVars {
     pub fn from_env() -> Result<Self, anyhow::Error> {
         let inner = Inner::init_from_env()?;
         let graphql = InnerGraphQl::init_from_env()?.into();
-        let mapping_handlers = InnerMappingHandlers::init_from_env()?.into();
+        let mapping_handlers = InnerMappingHandlers::init_from_env()?.try_into()?;
         let store = InnerStore::init_from_env()?.try_into()?;
         let ipfs_request_timeout = match inner.ipfs_request_timeout {
             Some(timeout) => Duration::from_secs(timeout),