support node 24

Author: joeferner

.devcontainer/Dockerfile

@@ -1,4 +1,4 @@
-FROM mcr.microsoft.com/devcontainers/javascript-node:22
+FROM mcr.microsoft.com/devcontainers/javascript-node:24
 
 # need to run update and install at the same time to prevent stale update layer
 RUN apt-get update && apt-get install -y gdb clang-format openjdk-21-jdk maven

.github/workflows/master.yml

@@ -9,7 +9,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        nodeVersion: [ 20, 22 ]
+        nodeVersion: [ 20, 22, 24 ]
         jdkVersion: [ openjdk9, openjdk10, openjdk11, openjdk21 ]
     steps:
       - name: Checkout source code

.github/workflows/pull-requests.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        nodeVersion: [ 20, 22 ]
+        nodeVersion: [ 20, 22, 24 ]
         jdkVersion: [ openjdk9, openjdk10, openjdk11, openjdk21 ]
     steps:
       - name: Checkout source code

src-cpp/javaObject.cpp

@@ -325,7 +325,7 @@ NAN_INDEX_GETTER(JavaObject::indexGetter) {
   assertNoException(env);
   if ((jint)index >= arrayLength) {
     info.GetReturnValue().SetUndefined();
-    return;
+    return v8::Intercepted::kYes;
   }
 
   jmethodID array_get = env->GetStaticMethodID(arrayClass, "get", "(Ljava/lang/Object;I)Ljava/lang/Object;");