Skip to content

Commit

Permalink
Remove get_config from the Node-API library (#3919)
Browse files Browse the repository at this point in the history
  • Loading branch information
aqrln committed Apr 26, 2023
1 parent 993890f commit e9d4b38
Showing 1 changed file with 1 addition and 44 deletions.
45 changes: 1 addition & 44 deletions query-engine/query-engine-node-api/src/functions.rs
Original file line number Diff line number Diff line change
@@ -1,11 +1,7 @@
use crate::error::ApiError;
use napi::{bindgen_prelude::*, JsUnknown};
use napi_derive::napi;
use request_handlers::dmmf;
use std::{
collections::{BTreeMap, HashMap},
sync::Arc,
};
use std::sync::Arc;

#[derive(serde::Serialize, Clone, Copy)]
#[napi(object)]
Expand Down Expand Up @@ -38,45 +34,6 @@ pub fn dmmf(datamodel_string: String) -> napi::Result<String> {
Ok(serde_json::to_string(&dmmf)?)
}

#[napi]
pub fn get_config(js_env: Env, options: JsUnknown) -> napi::Result<JsUnknown> {
#[derive(serde::Deserialize)]
#[serde(rename_all = "camelCase")]
struct GetConfigOptions {
datamodel: String,
#[serde(default)]
ignore_env_var_errors: bool,
#[serde(default)]
datasource_overrides: BTreeMap<String, String>,
#[serde(default)]
env: HashMap<String, String>,
}

let options: GetConfigOptions = js_env.from_js_value(options)?;

let GetConfigOptions {
datamodel,
ignore_env_var_errors,
datasource_overrides,
env,
} = options;

let overrides: Vec<(_, _)> = datasource_overrides.into_iter().collect();
let mut config = psl::parse_configuration(&datamodel).map_err(|errors| ApiError::conversion(errors, &datamodel))?;

config
.resolve_datasource_urls_query_engine(
&overrides,
|key| env.get(key).map(ToString::to_string),
ignore_env_var_errors,
)
.map_err(|errors| ApiError::conversion(errors, &datamodel))?;

let serialized = psl::get_config::config_to_mcf_json_value(&config);

js_env.to_js_value(&serialized)
}

#[napi]
pub fn debug_panic(panic_message: Option<String>) -> napi::Result<()> {
let user_facing = user_facing_errors::Error::from_panic_payload(Box::new(
Expand Down

0 comments on commit e9d4b38

Please sign in to comment.