fix(java): box OAuth native pointer as java.lang.Long; add slf4j-nop; fix Main.java import

sunli829 · sunli829 · commit a86109e112f0 · 2026-03-04T12:15:44.000+08:00
- Add JavaLong newtype (IntoJValue boxes i64 as java.lang.Long object) to fix
  WrongJValueType("object", "long") panic in async_util::execute
- Use JavaLong in oauthBuild return value so callback receives a proper object
- Remove stray java.io.ObjectInputFilter.Config import from test/Main.java
- Add slf4j-nop-1.7.30.jar to libs/ and all three classpaths in Makefile.toml
  to silence SLF4J StaticLoggerBinder warning at runtime
diff --git a/java/Makefile.toml b/java/Makefile.toml
@@ -9,7 +9,7 @@ args = [
   "-h",
   "c",
   "-cp",
-  "libs/native-lib-loader-2.4.0.jar;libs/slf4j-api-1.7.30.jar;libs/gson-2.10.1.jar",
+  "libs/native-lib-loader-2.4.0.jar;libs/slf4j-api-1.7.30.jar;libs/slf4j-nop-1.7.30.jar;libs/gson-2.10.1.jar",
   "-sourcepath",
   "javasrc/src/main/java",
   "-d",
@@ -22,7 +22,7 @@ cwd = "java"
 [tasks.compile-java-test]
 args = [
   "-cp",
-  "libs/native-lib-loader-2.4.0.jar;libs/slf4j-api-1.7.30.jar;libs/gson-2.10.1.jar",
+  "libs/native-lib-loader-2.4.0.jar;libs/slf4j-api-1.7.30.jar;libs/slf4j-nop-1.7.30.jar;libs/gson-2.10.1.jar",
   "-sourcepath",
   "javasrc/src/main/java",
   "-d",
@@ -162,7 +162,7 @@ command = "java"
 args = [
   "-Djava.library.path=target/debug",
   "-cp",
-  "java/classes;java/libs/native-lib-loader-2.4.0.jar;java/libs/slf4j-api-1.7.30.jar;java/libs/gson-2.10.1.jar",
+  "java/classes;java/libs/native-lib-loader-2.4.0.jar;java/libs/slf4j-api-1.7.30.jar;java/libs/slf4j-nop-1.7.30.jar;java/libs/gson-2.10.1.jar",
   "Main",
 ]
 dependencies = ["java", "compile-java-test"]
diff --git a/java/libs/slf4j-nop-1.7.30.jar b/java/libs/slf4j-nop-1.7.30.jar
diff --git a/java/src/oauth.rs b/java/src/oauth.rs
@@ -5,7 +5,7 @@ use jni::{
 };
 use longport::oauth::{OAuth, OAuthBuilder};
 
-use crate::{async_util, error::jni_result};
+use crate::{async_util, error::jni_result, types::JavaLong};
 
 // ── OAuth native handle
 // ───────────────────────────────────────────────────────
@@ -51,7 +51,7 @@ pub unsafe extern "system" fn Java_com_longport_SdkNative_oauthBuild(
                 })
                 .await
                 .map_err(|e| crate::error::JniError::Other(e.to_string()))?;
-            Ok(Box::into_raw(Box::new(oauth)) as jlong)
+            Ok(JavaLong::from(Box::into_raw(Box::new(oauth)) as jlong))
         })?;
         Ok(())
     })
diff --git a/java/src/types/classes.rs b/java/src/types/classes.rs
@@ -132,9 +132,7 @@ impl_java_class!(
 impl_java_class!(
     "com/longport/quote/PrePostQuote",
     longport::quote::PrePostQuote,
-    [
-        last_done, timestamp, volume, turnover, high, low, prev_close
-    ]
+    [last_done, timestamp, volume, turnover, high, low, prev_close]
 );
 
 impl_java_class!(
diff --git a/java/src/types/datetime.rs b/java/src/types/datetime.rs
@@ -1,18 +1,18 @@
 use std::borrow::Cow;
 
 use jni::{
-    JNIEnv,
     errors::Result,
     objects::{JValue, JValueOwned},
+    JNIEnv,
 };
 use time::{Date, Month, OffsetDateTime, PrimitiveDateTime, Time};
 
 use crate::{
     init::{
-        TIME_INSTANT_CLASS, TIME_LOCALDATE_CLASS, TIME_LOCALDATETIME_CLASS, TIME_LOCALTIME_CLASS,
+        TIME_INSTANT_CLASS, TIME_LOCALDATETIME_CLASS, TIME_LOCALDATE_CLASS, TIME_LOCALTIME_CLASS,
         TIME_OFFSETDATETIME_CLASS, TIME_ZONE_ID,
     },
-    types::{ClassLoader, FromJValue, IntoJValue, JSignature, get_field},
+    types::{get_field, ClassLoader, FromJValue, IntoJValue, JSignature},
 };
 
 impl ClassLoader for OffsetDateTime {
diff --git a/java/src/types/decimal.rs b/java/src/types/decimal.rs
@@ -1,6 +1,6 @@
 use std::borrow::Cow;
 
-use jni::{JNIEnv, errors::Result, objects::JValueOwned};
+use jni::{errors::Result, objects::JValueOwned, JNIEnv};
 use longport::Decimal;
 
 use crate::{
diff --git a/java/src/types/enum_types.rs b/java/src/types/enum_types.rs
@@ -1,9 +1,9 @@
 use std::borrow::Cow;
 
 use jni::{
-    JNIEnv,
     errors::Result,
     objects::{JObject, JValueOwned},
+    JNIEnv,
 };
 use longport_java_macros::impl_java_enum;
 
@@ -348,9 +348,7 @@ impl_java_enum!(
 impl_java_enum!(
     "com/longport/trade/OrderType",
     longport::trade::OrderType,
-    [
-        Unknown, LO, ELO, MO, AO, ALO, ODD, LIT, MIT, TSLPAMT, TSLPPCT, TSMAMT, TSMPCT, SLO
-    ]
+    [Unknown, LO, ELO, MO, AO, ALO, ODD, LIT, MIT, TSLPAMT, TSLPPCT, TSMAMT, TSMPCT, SLO]
 );
 
 impl_java_enum!(
diff --git a/java/src/types/mod.rs b/java/src/types/mod.rs
@@ -12,10 +12,10 @@ mod sub_flags;
 use std::borrow::Cow;
 
 use jni::{
-    JNIEnv,
     errors::Result,
     objects::{GlobalRef, JObject, JValueOwned},
     strings::JNIString,
+    JNIEnv,
 };
 
 pub(crate) use self::{
@@ -25,7 +25,7 @@ pub(crate) use self::{
     },
     object_array::ObjectArray,
     primary_array::PrimaryArray,
-    primary_types::JavaInteger,
+    primary_types::{JavaInteger, JavaLong},
 };
 
 pub(crate) trait ClassLoader {
diff --git a/java/src/types/object_array.rs b/java/src/types/object_array.rs
@@ -1,9 +1,9 @@
 use std::borrow::Cow;
 
 use jni::{
-    JNIEnv,
     errors::Result,
     objects::{JObject, JObjectArray, JValueOwned},
+    JNIEnv,
 };
 
 use crate::types::{ClassLoader, FromJValue, IntoJValue, JSignature};
diff --git a/java/src/types/optional.rs b/java/src/types/optional.rs
@@ -1,9 +1,9 @@
 use std::borrow::Cow;
 
 use jni::{
-    JNIEnv,
     errors::Result,
     objects::{JObject, JValueOwned},
+    JNIEnv,
 };
 
 use crate::types::{ClassLoader, FromJValue, IntoJValue, JSignature};
diff --git a/java/src/types/primary_array.rs b/java/src/types/primary_array.rs
@@ -1,10 +1,10 @@
 use std::borrow::Cow;
 
 use jni::{
-    JNIEnv,
     errors::Result,
     objects::{JValueOwned, ReleaseMode},
     sys::{jboolean, jint, jlong},
+    JNIEnv,
 };
 
 use crate::types::{FromJValue, IntoJValue, JSignature};
diff --git a/java/src/types/primary_types.rs b/java/src/types/primary_types.rs
@@ -1,9 +1,9 @@
 use std::borrow::Cow;
 
-use jni::{JNIEnv, errors::Result, objects::JValueOwned};
+use jni::{errors::Result, objects::JValueOwned, JNIEnv};
 
 use crate::{
-    init::INTEGER_CLASS,
+    init::{INTEGER_CLASS, LONG_CLASS},
     types::{FromJValue, IntoJValue, JSignature},
 };
 
@@ -87,6 +87,47 @@ impl IntoJValue for f64 {
     }
 }
 
+pub(crate) struct JavaLong(i64);
+
+impl From<i64> for JavaLong {
+    #[inline]
+    fn from(value: i64) -> Self {
+        JavaLong(value)
+    }
+}
+
+impl From<JavaLong> for i64 {
+    #[inline]
+    fn from(value: JavaLong) -> Self {
+        value.0
+    }
+}
+
+impl JSignature for JavaLong {
+    fn signature() -> Cow<'static, str> {
+        "Ljava/lang/Long;".into()
+    }
+}
+
+impl FromJValue for JavaLong {
+    fn from_jvalue(env: &mut JNIEnv, value: JValueOwned) -> Result<Self> {
+        let obj = value.l()?;
+        let value = env.call_method(obj, "longValue", "()J", &[])?;
+        Ok(JavaLong(value.j()?))
+    }
+}
+
+impl IntoJValue for JavaLong {
+    fn into_jvalue<'a>(self, env: &mut JNIEnv<'a>) -> Result<JValueOwned<'a>> {
+        let obj = env.new_object(
+            LONG_CLASS.get().unwrap(),
+            "(J)V",
+            &[JValueOwned::from(self.0).borrow()],
+        )?;
+        Ok(JValueOwned::from(obj))
+    }
+}
+
 pub(crate) struct JavaInteger(i32);
 
 impl From<i32> for JavaInteger {
diff --git a/java/src/types/string.rs b/java/src/types/string.rs
@@ -1,7 +1,7 @@
 use jni::{
-    JNIEnv,
     errors::Result,
     objects::{JString, JValueOwned},
+    JNIEnv,
 };
 
 use crate::{
diff --git a/java/src/types/sub_flags.rs b/java/src/types/sub_flags.rs
@@ -1,6 +1,6 @@
 use std::borrow::Cow;
 
-use jni::{JNIEnv, errors::Result, objects::JValueOwned};
+use jni::{errors::Result, objects::JValueOwned, JNIEnv};
 use longport::quote::SubFlags;
 
 use crate::types::{IntoJValue, JSignature};
diff --git a/java/test/Main.java b/java/test/Main.java
@@ -17,4 +17,4 @@ public static void main(String[] args) throws Exception {
             Thread.sleep(30000);
         }
     }
-}
+}
