feat(android): add support new arch for android

Author: ngocle2497

.gitignore

@@ -11,6 +11,7 @@ jsconfig.json
 
 # Xcode
 #
+.cxx/
 build/
 *.pbxuser
 !default.pbxuser

README.md

@@ -5,7 +5,8 @@
 ### Why should we use react-native-keys over react-native-config?
 
 We should use **react-native-keys** instead of **react-native-config** because **react-native-keys** gives guarantee of undecryptable **envs** whereas **react-native-config** **envs** can be decompile and hack
-#### See the [ Migration from react-native-config](docs/react-native-config-migration-guide.md)
+
+#### See the [Migration from react-native-config](docs/react-native-config-migration-guide.md)
 
 <div align="center">
     <img align="right" width="250" style="margin-right: 55px;" src="./media/key.png">
@@ -67,7 +68,7 @@ Create a new file `keys.development.json` in the root of your React Native app a
 }
 ```
 
-**Note: Dont forgot to follow other steps for **Android** and **IOS** to use these keys**
+**Note: Dont forgot to follow other steps for**Android**and**IOS**to use these keys**
 
 later on you can define other enviroment files like `keys.staging.json` and `keys.production.json`
 
@@ -99,7 +100,7 @@ Keep in mind It's [basically impossible to prevent users from reverse engineerin
 Install the package:
 
 ```
-$ yarn add react-native-keys
+yarn add react-native-keys
 ```
 
 Link the library:
@@ -109,7 +110,7 @@ Link the library:
 or later. For earlier versions you need to manually link the module.)
 
 ```
-$ react-native link react-native-keys
+react-native link react-native-keys
 ```
 
 if cocoapods are used in the project then pod has to be installed as well:
@@ -124,12 +125,12 @@ if cocoapods are used in the project then pod has to be installed as well:
 
 - Manual Link (iOS)
 
-  1.  In XCode, in the project navigator, right click `Libraries` ➜ `Add Files to [your project's name]`
-  2.  Go to `node_modules` ➜ `react-native-keys` and add `Keys.xcodeproj`
-  3.  Expand the `Keys.xcodeproj` ➜ `Products` folder
-  4.  In the project navigator, select your project. Add `Keys.a` to your project's `Build Phases` ➜ `Link Binary With Libraries`
-  5.  And go the Build Settings tab. Make sure All is toggled on (instead of Basic)
-  6.  Look for Header Search Paths and add `$(SRCROOT)/../node_modules/react-native-keys/ios/**` as `non-recursive`
+  1. In XCode, in the project navigator, right click `Libraries` ➜ `Add Files to [your project's name]`
+  2. Go to `node_modules` ➜ `react-native-keys` and add `Keys.xcodeproj`
+  3. Expand the `Keys.xcodeproj` ➜ `Products` folder
+  4. In the project navigator, select your project. Add `Keys.a` to your project's `Build Phases` ➜ `Link Binary With Libraries`
+  5. And go the Build Settings tab. Make sure All is toggled on (instead of Basic)
+  6. Look for Header Search Paths and add `$(SRCROOT)/../node_modules/react-native-keys/ios/**` as `non-recursive`
 
 - Manual Link (Android)
 
@@ -144,8 +145,8 @@ if cocoapods are used in the project then pod has to be installed as well:
 
   ```diff
   dependencies {
-  	implementation "com.facebook.react:react-native:+"  // From node_modules
-  +	implementation project(':react-native-keys')
+   implementation "com.facebook.react:react-native:+"  // From node_modules
+  + implementation project(':react-native-keys')
   }
   ```
 
@@ -156,9 +157,9 @@ if cocoapods are used in the project then pod has to be installed as well:
 
   @Override
   protected List<ReactPackage> getPackages() {
-  	   return Arrays.asList(
-          		new MainReactPackage()
-  +      		new KeysPackage()
+      return Arrays.asList(
+            new MainReactPackage()
+  +        new KeysPackage()
       );
   }
   ```
@@ -175,7 +176,7 @@ you can only read jni key into java file.like this
 URL url = new URL(BuildConfig.API_URL);  // https://example.com
 ```
 
-You can also read them from your Gradle configuration:
+You can also read them from your Gradle configuration(only public keys):
 
 ```groovy
 defaultConfig {
@@ -202,7 +203,7 @@ versionCode project.keys.get("VERSION_CODE").toInteger()
 ```java
 import static com.reactnativekeysjsi.KeysModule.getSecureFor;
 
-String secureValue=getSecureFor("BRANCH_KEY");   // key_test_omQ7YYKiq57vOqEJsdcsdfeEsiWkwxE
+String secureValue = getSecureFor("BRANCH_KEY");   // key_test_omQ7YYKiq57vOqEJsdcsdfeEsiWkwxE
 ```
 
 ### iOS
@@ -265,10 +266,19 @@ ios/tmp.xcconfig
 
 - Go to _Edit scheme..._ -> _Build_ -> _Pre-actions_, click _+_ and select _New Run Script Action_. Paste below code which will generate KEYS keys on native ios side (into node*modules) Make sure to select your target under \_Provide build settings from*, so `$SRCROOT` environment variables is available to the script.
 
-```
+```sh
 "${SRCROOT}/../node_modules/react-native-keys/keysIOS.js"
 ```
 
+Alternatively, you can define a map in `Pre-actions` associating builds with env files:
+
+```sh
+    export KEYSFILE = "path_to_env"
+   "${SRCROOT}/../node_modules/react-native-keys/keysIOS.js"
+```
+
+ call, and use build cases in lowercase, like:
+
 ### Different environments
 
 Save config for different environments in different files: `keys.staging.json`, `keys.production.json`, etc.
@@ -278,9 +288,9 @@ By default react-native-keys will read from `keys.development.json`, but you can
 The simplest approach is to tell it what file to read with an environment variable, like:
 
 ```
-$ KEYSFILE=keys.staging.json react-native run-ios           # bash
-$ SET KEYSFILE=keys.staging.json && react-native run-ios    # windows
-$ env:KEYSFILE="keys.staging.json"; react-native run-ios    # powershell
+KEYSFILE=keys.staging.json react-native run-ios           # bash
+SET KEYSFILE=keys.staging.json && react-native run-ios    # windows
+env:KEYSFILE="keys.staging.json"; react-native run-ios    # powershell
 ```
 
 This also works for `run-android`. Alternatively, there are platform-specific options below.
@@ -289,13 +299,13 @@ This also works for `run-android`. Alternatively, there are platform-specific op
 
 The same environment variable can be used to assemble releases with a different config:
 
-```
-$ cd android && KEYSFILE=keys.staging.json ./gradlew assembleRelease
+```sh
+cd android && KEYSFILE=keys.staging.json ./gradlew assembleRelease
 ```
 
 Alternatively, you can define a map in `build.gradle` associating builds with env files. Do it before the `apply from` call, and use build cases in lowercase, like:
 
-```
+```groovy
 project.ext.keyFiles = [
   debug: "keys.development.json",
   release: "keys.staging.json",
@@ -309,7 +319,7 @@ apply from: project(':react-native-keys').projectDir.getPath() + "/RNKeys.gradle
 In `android/app/build.gradle`, if you use `applicationIdSuffix` or `applicationId` that is different from the package name indicated in `AndroidManifest.xml` in `<manifest package="...">` tag, for example, to support different build variants:
 Add this in `android/app/build.gradle`
 
-```
+```groovy
 defaultConfig {
     ...
     resValue "string", "build_config_package", "YOUR_PACKAGE_NAME_IN_ANDROIDMANIFEST_XML"

android/CMakeLists.txt

@@ -7,97 +7,88 @@ set(CMAKE_CXX_STANDARD 17)
 set(BUILD_DIR ${CMAKE_SOURCE_DIR}/build)
 
 if(${REACT_NATIVE_MINOR_VERSION} GREATER_EQUAL 71)
-        # Consume shared libraries and headers from prefabs
-        find_package(fbjni REQUIRED CONFIG)
-        find_package(ReactAndroid REQUIRED CONFIG)
+    # Consume shared libraries and headers from prefabs
+    find_package(ReactAndroid REQUIRED CONFIG)
 
-        include_directories(
-                ${PACKAGE_NAME}
-                "${NODE_MODULES_DIR}/react-native/React"
-                "${NODE_MODULES_DIR}/react-native/React/Base"
-                "${NODE_MODULES_DIR}/react-native/ReactCommon/jsi"
-                "./androidcpp"
-                "../cpp"
-                ${FOLLY_INCLUDE_DIR}
-                "."
-                "${NODE_MODULES_DIR}/react-native/ReactAndroid/src/main/jni/react/turbomodule"
-        )
+    include_directories(
+        ${PACKAGE_NAME}
+        "./androidcpp"
+        "../cpp"
+    )
 
-        file(GLOB LIBRN_DIR "${BUILD_DIR}/react-native-0*/jni/${ANDROID_ABI}")
-        message(STATUS "LIBRN_DIR: ${LIBRN_DIR}")
+    file(GLOB LIBRN_DIR "${BUILD_DIR}/react-native-0*/jni/${ANDROID_ABI}")
+    message(STATUS "LIBRN_DIR: ${LIBRN_DIR}")
 
-        add_library(
-                ${PACKAGE_NAME}
-                SHARED
-                ./androidcpp/mediator.cpp
-                ./cpp-adapter.cpp
-                ../cpp/crypto.cpp
-                ../cpp/decryptor.cpp
-        )
+    add_library(
+        ${PACKAGE_NAME}
+        SHARED
+        ./androidcpp/mediator.cpp
+        ./cpp-adapter.cpp
+        ../cpp/crypto.cpp
+        ../cpp/decryptor.cpp
+    )
 
-        find_library(
-                LOG_LIB
-                log
-        )
+    find_library(
+        LOG_LIB
+        log
+    )
 
-        find_library(
-                REACT_NATIVE_JNI_LIB
-                reactnativejni
-                PATHS ${LIBRN_DIR}
-                NO_CMAKE_FIND_ROOT_PATH
-        )
+    find_library(
+        REACT_NATIVE_JNI_LIB
+        reactnativejni
+        PATHS ${LIBRN_DIR}
+        NO_CMAKE_FIND_ROOT_PATH
+    )
 
-        set_target_properties(
-                ${PACKAGE_NAME} PROPERTIES
-                CXX_STANDARD 17
-                CXX_EXTENSIONS OFF
-                POSITION_INDEPENDENT_CODE ON
-        )
+    set_target_properties(
+        ${PACKAGE_NAME} PROPERTIES
+        CXX_STANDARD 17
+        CXX_EXTENSIONS OFF
+        POSITION_INDEPENDENT_CODE ON
+    )
 
-        find_package(openssl REQUIRED CONFIG)
+    find_package(openssl REQUIRED CONFIG)
 
-        target_link_libraries(
-                ${PACKAGE_NAME}
-                ReactAndroid::turbomodulejsijni
-                fbjni::fbjni
-                ${LOG_LIB}
-                ReactAndroid::jsi
-                ReactAndroid::reactnativejni
-                ReactAndroid::react_nativemodule_core
-                android
-                openssl::crypto
-        )
+    target_link_libraries(
+        ${PACKAGE_NAME}
+        ${LOG_LIB}
+        ReactAndroid::jsi
+        ReactAndroid::reactnativejni
+        ReactAndroid::react_nativemodule_core
+        android
+        openssl::crypto
+    )
 else()
-        add_library(
-                ${PACKAGE_NAME}
-                SHARED
-                ../../react-native/ReactCommon/jsi/jsi/jsi.cpp
-                ./androidcpp/mediator.cpp
-                ./cpp-adapter.cpp
-                ../cpp/crypto.cpp
-                ../cpp/decryptor.cpp
-        )
+    add_library(
+        ${PACKAGE_NAME}
+        SHARED
+        ../../react-native/ReactCommon/jsi/jsi/jsi.cpp
+        ./androidcpp/mediator.cpp
+        ./cpp-adapter.cpp
+        ../cpp/crypto.cpp
+        ../cpp/decryptor.cpp
+    )
 
-        include_directories(
-                ../../react-native/React
-                ../../react-native/React/Base
-                ../../react-native/ReactCommon/jsi
-                ./cpp
-                ../cpp
-        )
+    include_directories(
+        ../../react-native/React
+        ../../react-native/React/Base
+        ../../react-native/ReactCommon/jsi
+        ./cpp
+        ../cpp
+    )
 
-        set_target_properties(
-                ${PACKAGE_NAME} PROPERTIES
-                CXX_STANDARD 17
-                CXX_EXTENSIONS OFF
-                POSITION_INDEPENDENT_CODE ON
-        )
+    set_target_properties(
+        ${PACKAGE_NAME} PROPERTIES
+        CXX_STANDARD 17
+        CXX_EXTENSIONS OFF
+        POSITION_INDEPENDENT_CODE ON
+    )
 
-        find_package(openssl REQUIRED CONFIG)
+    find_package(openssl REQUIRED CONFIG)
 
-        target_link_libraries(
-                ${PACKAGE_NAME}
-                android
-                openssl::crypto
-        )
+    target_link_libraries(
+        ${PACKAGE_NAME}
+        android
+        openssl::crypto
+    )
 endif()

android/androidcpp/mediator.cpp

@@ -10,13 +10,13 @@ std::string jstring2string(JNIEnv *env, jstring jStr)
 
     const jclass stringClass = env->GetObjectClass(jStr);
     const jmethodID getBytes = env->GetMethodID(stringClass, "getBytes", "(Ljava/lang/String;)[B");
-    const jbyteArray stringJbytes = (jbyteArray) env->CallObjectMethod(jStr, getBytes,
-                                                                       env->NewStringUTF("UTF-8"));
+    const jbyteArray stringJbytes = (jbyteArray)env->CallObjectMethod(jStr, getBytes,
+                                                                      env->NewStringUTF("UTF-8"));
 
-    size_t length = (size_t) env->GetArrayLength(stringJbytes);
+    size_t length = (size_t)env->GetArrayLength(stringJbytes);
     jbyte *pBytes = env->GetByteArrayElements(stringJbytes, NULL);
 
-    std::string ret = std::string((char *) pBytes, length);
+    std::string ret = std::string((char *)pBytes, length);
     env->ReleaseByteArrayElements(stringJbytes, pBytes, JNI_ABORT);
 
     env->DeleteLocalRef(stringJbytes);
@@ -25,8 +25,9 @@ std::string jstring2string(JNIEnv *env, jstring jStr)
 }
 
 extern "C" JNIEXPORT jstring JNICALL
-Java_com_reactnativekeysjsi_KeysModule_getJniJsonStringifyData(JNIEnv * env, jobject thiz,jstring key) {
+Java_com_reactnativekeysjsi_KeysModule_getJniJsonStringifyData(JNIEnv *env, jobject thiz, jstring key)
+{
     auto *crypto = new Crypto();
-    std::string _key{jstring2string(env,key)};
+    std::string _key{jstring2string(env, key)};
     return env->NewStringUTF(crypto->getJniJsonStringifyData(_key).c_str());
 }