refactor(cubesql): Use correct naming for checkSqlAuth

Author: ovr

packages/cubejs-backend-native/src/auth.rs

@@ -17,14 +17,14 @@ use crate::channel::call_js_with_channel_as_callback;
 #[derive(Debug)]
 pub struct NodeBridgeAuthService {
     channel: Arc<Channel>,
-    check_auth: Arc<Root<JsFunction>>,
+    check_sql_auth: Arc<Root<JsFunction>>,
 }
 
 impl NodeBridgeAuthService {
-    pub fn new(channel: Channel, check_auth: Root<JsFunction>) -> Self {
+    pub fn new(channel: Channel, check_sql_auth: Root<JsFunction>) -> Self {
         Self {
             channel: Arc::new(channel),
-            check_auth: Arc::new(check_auth),
+            check_sql_auth: Arc::new(check_sql_auth),
         }
     }
 }
@@ -36,14 +36,14 @@ pub struct TransportRequest {
 }
 
 #[derive(Debug, Serialize)]
-struct CheckAuthRequest {
+struct CheckSQLAuthRequest {
     request: TransportRequest,
     user: Option<String>,
     password: Option<String>,
 }
 
 #[derive(Debug, Deserialize)]
-struct CheckAuthResponse {
+struct CheckSQLAuthResponse {
     password: Option<String>,
     superuser: bool,
     #[serde(rename = "securityContext", skip_serializing_if = "Option::is_none")]
@@ -76,17 +76,17 @@ impl SqlAuthService for NodeBridgeAuthService {
 
         let request_id = Uuid::new_v4().to_string();
 
-        let extra = serde_json::to_string(&CheckAuthRequest {
+        let extra = serde_json::to_string(&CheckSQLAuthRequest {
             request: TransportRequest {
                 id: format!("{}-span-1", request_id),
                 meta: None,
             },
             user: user.clone(),
             password: password.clone(),
         })?;
-        let response: CheckAuthResponse = call_js_with_channel_as_callback(
+        let response: CheckSQLAuthResponse = call_js_with_channel_as_callback(
             self.channel.clone(),
-            self.check_auth.clone(),
+            self.check_sql_auth.clone(),
             Some(extra),
         )
         .await?;

packages/cubejs-backend-native/src/node_export.rs

@@ -48,8 +48,8 @@ impl SQLInterface {
 
 fn register_interface<C: NodeConfiguration>(mut cx: FunctionContext) -> JsResult<JsPromise> {
     let options = cx.argument::<JsObject>(0)?;
-    let check_auth = options
-        .get::<JsFunction, _, _>(&mut cx, "checkAuth")?
+    let check_sql_auth = options
+        .get::<JsFunction, _, _>(&mut cx, "checkSqlAuth")?
         .root(&mut cx);
     let transport_sql_api_load = options
         .get::<JsFunction, _, _>(&mut cx, "sqlApiLoad")?
@@ -101,7 +101,7 @@ fn register_interface<C: NodeConfiguration>(mut cx: FunctionContext) -> JsResult
         transport_sql_generator,
         transport_can_switch_user_for_session,
     );
-    let auth_service = NodeBridgeAuthService::new(cx.channel(), check_auth);
+    let auth_service = NodeBridgeAuthService::new(cx.channel(), check_sql_auth);
 
     std::thread::spawn(move || {
         let config = C::new(NodeConfigurationFactoryOptions {