nacos-sdk-rust-binding-node 0.1.0-PRE → 0.1.0-PRE2

package/Cargo.toml

@@ -21,7 +21,7 @@ napi-derive = "2"
 #nacos-sdk = { version = "0.2.3", features = ["async"] }
 nacos-sdk = { git = "https://github.com/nacos-group/nacos-sdk-rust.git", features = ["async"] }
 
-tracing-subscriber = "0.3"
+tracing-subscriber = { version = "0.3", features = ["env-filter", "local-time"] }
 tracing-appender = "0.2"
 lazy_static = "1.4.0"
 

package/index.d.ts

@@ -107,6 +107,12 @@ export class NacosConfigClient {
    * If it fails, pay attention to err
    */
   addListener(dataId: string, group: string, listener: (err: Error | null, value: NacosConfigResponse) => any): Promise<void>
+  /**
+   * Remove NacosConfigChangeListener callback func, but noop....
+   * The logic is not implemented internally, and only APIs are provided as compatibility.
+   * Users maybe do not need it? Not removing the listener is not a big problem, Sorry!
+   */
+  removeListener(dataId: string, group: string, listener: (err: Error | null, value: NacosConfigResponse) => any): Promise<void>
 }
 /** Client api of Nacos Naming. */
 export class NacosNamingClient {
@@ -147,4 +153,10 @@ export class NacosNamingClient {
    * If it fails, pay attention to err
    */
   subscribe(serviceName: string, group: string, clusters: Array<string> | undefined | null, listener: (err: Error | null, value: Array<NacosServiceInstance>) => any): Promise<void>
+  /**
+   * Remove NacosNamingEventListener callback func, but noop....
+   * The logic is not implemented internally, and only APIs are provided as compatibility.
+   * Users maybe do not need it? Not removing the subscription is not a big problem, Sorry!
+   */
+  unSubscribe(serviceName: string, group: string, clusters: Array<string> | undefined | null, listener: (err: Error | null, value: Array<NacosServiceInstance>) => any): Promise<void>
 }

package/package.json

@@ -7,7 +7,7 @@
     "binding",
     "ffi"
   ],
-  "version": "0.1.0-PRE",
+  "version": "0.1.0-PRE2",
   "bugs": "https://github.com/opc-source/nacos-sdk-rust-binding-node/issues",
   "license": "MIT",
   "repository": {
@@ -55,19 +55,19 @@
   },
   "packageManager": "yarn@3.4.1",
   "optionalDependencies": {
-    "nacos-sdk-rust-binding-node-win32-x64-msvc": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-darwin-x64": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-linux-x64-gnu": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-darwin-arm64": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-android-arm64": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-linux-arm64-gnu": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-linux-arm64-musl": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-win32-arm64-msvc": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-linux-arm-gnueabihf": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-linux-x64-musl": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-freebsd-x64": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-win32-ia32-msvc": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-android-arm-eabi": "0.1.0-PRE",
-    "nacos-sdk-rust-binding-node-darwin-universal": "0.1.0-PRE"
+    "nacos-sdk-rust-binding-node-win32-x64-msvc": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-darwin-x64": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-linux-x64-gnu": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-darwin-arm64": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-android-arm64": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-linux-arm64-gnu": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-linux-arm64-musl": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-win32-arm64-msvc": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-linux-arm-gnueabihf": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-linux-x64-musl": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-freebsd-x64": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-win32-ia32-msvc": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-android-arm-eabi": "0.1.0-PRE2",
+    "nacos-sdk-rust-binding-node-darwin-universal": "0.1.0-PRE2"
   }
 }

package/src/config.rs

@@ -141,6 +141,19 @@ impl NacosConfigClient {
       .map_err(|nacos_err| Error::from_reason(nacos_err.to_string()))?;
     Ok(())
   }
+
+  /// Remove NacosConfigChangeListener callback func, but noop....
+  /// The logic is not implemented internally, and only APIs are provided as compatibility.
+  /// Users maybe do not need it? Not removing the listener is not a big problem, Sorry!
+  #[napi]
+  pub async fn remove_listener(
+    &self,
+    _data_id: String,
+    _group: String,
+    _listener: ThreadsafeFunction<NacosConfigResponse>,
+  ) -> Result<()> {
+    Ok(())
+  }
 }
 
 #[napi(object)]

package/src/lib.rs

@@ -19,6 +19,7 @@ lazy_static::lazy_static! {
 
       tracing_subscriber::fmt()
         .with_writer(non_blocking)
+        .with_timer(tracing_subscriber::fmt::time::LocalTime::rfc_3339())
         .with_level(true)
         .with_thread_names(true)
         .with_thread_ids(true)

package/src/naming.rs

@@ -221,6 +221,20 @@ impl NacosNamingClient {
       .map_err(|nacos_err| Error::from_reason(nacos_err.to_string()))?;
     Ok(())
   }
+
+  /// Remove NacosNamingEventListener callback func, but noop....
+  /// The logic is not implemented internally, and only APIs are provided as compatibility.
+  /// Users maybe do not need it? Not removing the subscription is not a big problem, Sorry!
+  #[napi]
+  pub async fn un_subscribe(
+    &self,
+    _service_name: String,
+    _group: String,
+    _clusters: Option<Vec<String>>,
+    _listener: ThreadsafeFunction<Vec<NacosServiceInstance>>,
+  ) -> Result<()> {
+    Ok(())
+  }
 }
 
 pub struct NacosNamingEventListener {