Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Commit 031f10e

Browse files
authored
Merge pull request #1497 from majirosstefan/number-generator-fix
replaced java.util.Random with java.security.SecureRandom in RNPushNo…
2 parents 0bfc74b + 5d7a228 commit 031f10e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

android/src/main/java/com/dieam/reactnativepushnotification/modules/RNPushNotification.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@
2727
import com.facebook.react.bridge.WritableMap;
2828

2929
import java.io.IOException;
30+
import java.security.SecureRandom;
3031
import java.util.HashMap;
3132
import java.util.Map;
32-
import java.util.Random;
3333

3434
import android.util.Log;
3535

@@ -43,7 +43,7 @@ public class RNPushNotification extends ReactContextBaseJavaModule implements Ac
4343
public static final String LOG_TAG = "RNPushNotification";// all logging should use this tag
4444

4545
private RNPushNotificationHelper mRNPushNotificationHelper;
46-
private final Random mRandomNumberGenerator = new Random(System.currentTimeMillis());
46+
private final SecureRandom mRandomNumberGenerator = new SecureRandom();
4747
private RNPushNotificationJsDelivery mJsDelivery;
4848

4949
public RNPushNotification(ReactApplicationContext reactContext) {
@@ -61,6 +61,8 @@ public RNPushNotification(ReactApplicationContext reactContext) {
6161
mRNPushNotificationHelper.checkOrCreateDefaultChannel();
6262
}
6363

64+
65+
6466
@Override
6567
public String getName() {
6668
return "RNPushNotification";

0 commit comments

Comments
 (0)