react-native-ble-manager 7.6.2 → 8.0.2

package/.github/ISSUE_TEMPLATE/bug_report.md

@@ -0,0 +1,38 @@
+---
+name: Bug report
+about: Create a report to help us improve
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+**Before open an issue**
+- Check the closed issues, your question maybe is not new.
+- We can't debug or reproduce hardware issue.
+- If the library is not working 99% you did something wrong, in the code, installation or in the phone permissions.
+
+**Describe the bug**
+A clear and concise description of what the bug is.
+
+**To Reproduce**
+Steps to reproduce the behavior:
+1. Go to '...'
+2. Click on '....'
+3. Scroll down to '....'
+4. See error
+
+**Expected behavior**
+A clear and concise description of what you expected to happen.
+
+**Screenshots**
+If applicable, add screenshots to help explain your problem.
+
+**Smartphone (please complete the following information):**
+ - Device: [e.g. iPhone6]
+ - OS: [e.g. iOS8.1]
+ - react-native-ble-manager version: 
+ - react-native version:
+
+**Additional context**
+Add any other context about the problem here.

package/README.md

@@ -22,23 +22,45 @@ RN 0.30-0.39 supported until 2.4.3
 
 ```shell
 npm i --save react-native-ble-manager
-npx react-native link
 ```
+The library support the react native autolink feature.
 
 
 ##### Android - Update Manifest
 
 ```xml
 // file: android/app/src/main/AndroidManifest.xml
-...
-    <uses-permission android:name="android.permission.BLUETOOTH"/>
-    <uses-permission android:name="android.permission.BLUETOOTH_ADMIN"/>
-    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
+<!-- Add xmlns:tools -->
+<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:tools="http://schemas.android.com/tools"
+    package="YOUR_PACKAGE_NAME">
+
+    <uses-permission android:name="android.permission.BLUETOOTH" android:maxSdkVersion="30" />
+    <uses-permission android:name="android.permission.BLUETOOTH_ADMIN" android:maxSdkVersion="30" />
+
+    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" android:maxSdkVersion="28"/>
+    <uses-permission-sdk-23 android:name="android.permission.ACCESS_FINE_LOCATION" tools:targetApi="Q"/>
+
+    <!-- Only when targeting Android 12 or higher -->
+    <!-- Please make sure you read the following documentation to have a
+         better understanging of the new permissions.
+         https://developer.android.com/guide/topics/connectivity/bluetooth/permissions#assert-never-for-location
+         -->
+
+    <!-- If your app doesn't use Bluetooth scan results to derive physical location information,
+         you can strongly assert that your app
+         doesn't derive physical location. -->
+    <uses-permission android:name="android.permission.BLUETOOTH_CONNECT" 
+                     android:usesPermissionFlags="neverForLocation"
+                     tools:targetApi="s" />
+
+    <!-- Needed only if your app looks for Bluetooth devices. -->
+    <uses-permission android:name="android.permission.BLUETOOTH_SCAN" />
+    <!-- Needed only if your app makes the device discoverable to Bluetooth devices. -->
+    <uses-permission android:name="android.permission.BLUETOOTH_ADVERTISE" />
 ...
 ```
 
-In Android API 29 >= you need to use "ACCESS_FINE_LOCATION" instead of "ACCESS_COARSE_LOCATION".
-
 If you need communication while the app is not in the foreground you need the "ACCESS_BACKGROUND_LOCATION" permission.
 
 ##### iOS - Update Info.plist
@@ -53,6 +75,7 @@ In iOS >= 13 you need to add the `NSBluetoothAlwaysUsageDescription` string key.
 - Android API >= 29 require the ACCESS_FINE_LOCATION permission to scan for peripherals.
    React-Native 0.63.X started targeting Android API 29.
 - Before write, read or start notification you need to call `retrieveServices` method
+- Because location and bluetooth permissions are runtime permissions, you **must** request these permissions at runtime along with declaring them    in your manifest.
 
 ## Example
 
@@ -758,6 +781,14 @@ A peripheral was disconnected.
 - `peripheral` - `String` - the id of the peripheral
 - `status` - `Number` - [Android only] disconnect [`reasons`](<https://developer.android.com/reference/android/bluetooth/BluetoothGattCallback.html#onConnectionStateChange(android.bluetooth.BluetoothGatt,%20int,%20int)>)
 
+### BleManagerPeripheralDidBond
+
+A bond with a peripheral was established
+
+**Arguments**
+
+Object with information about the device
+
 ### BleManagerCentralManagerWillRestoreState [iOS only]
 
 This is fired when [`centralManager:WillRestoreState:`](https://developer.apple.com/documentation/corebluetooth/cbcentralmanagerdelegate/1518819-centralmanager) is called (app relaunched in the background to handle a bluetooth event).

package/android/src/main/java/it/innove/BleManager.java

@@ -193,7 +193,7 @@ class BleManager extends ReactContextBaseJavaModule {
             for (Iterator<Map.Entry<String, Peripheral>> iterator = peripherals.entrySet().iterator(); iterator
                     .hasNext(); ) {
                 Map.Entry<String, Peripheral> entry = iterator.next();
-                if (!entry.getValue().isConnected()) {
+                if (!(entry.getValue().isConnected() || entry.getValue().isConnecting())) {
                     iterator.remove();
                 }
             }

package/android/src/main/java/it/innove/Peripheral.java

@@ -33,6 +33,8 @@ import java.util.Arrays;
 import java.util.Iterator;
 import java.util.List;
 import java.util.UUID;
+import java.util.Queue;
+import java.util.concurrent.ConcurrentLinkedQueue;
 
 import static android.os.Build.VERSION_CODES.LOLLIPOP;
 import static com.facebook.react.common.ReactConstants.TAG;
@@ -43,12 +45,15 @@ import static com.facebook.react.common.ReactConstants.TAG;
 public class Peripheral extends BluetoothGattCallback {
 
 	private static final String CHARACTERISTIC_NOTIFICATION_CONFIG = "00002902-0000-1000-8000-00805f9b34fb";
+	public static final int GATT_INSUFFICIENT_AUTHENTICATION = 5;
+	public static final int GATT_AUTH_FAIL = 137;
 
 	private final BluetoothDevice device;
 	private final Map<String, NotifyBufferContainer> bufferedCharacteristics;
 	protected byte[] advertisingDataBytes = new byte[0];
 	protected int advertisingRSSI;
 	private boolean connected = false;
+	private boolean connecting = false;
 	private ReactContext reactContext;
 
 	private BluetoothGatt gatt;
@@ -61,6 +66,11 @@ public class Peripheral extends BluetoothGattCallback {
 	private Callback registerNotifyCallback;
 	private Callback requestMTUCallback;
 
+	private final Queue<Runnable> commandQueue = new ConcurrentLinkedQueue<>();
+	private final Handler mainHandler = new Handler(Looper.getMainLooper());
+	private Runnable discoverServicesRunnable;
+	private boolean commandQueueBusy = false;
+
 	private List<byte[]> writeQueue = new ArrayList<>();
 
 	public Peripheral(BluetoothDevice device, int advertisingRSSI, byte[] scanRecord, ReactContext reactContext) {
@@ -95,6 +105,7 @@ public class Peripheral extends BluetoothGattCallback {
 		if (!connected) {
 			BluetoothDevice device = getDevice();
 			this.connectCallback = callback;
+			this.connecting = true;
 			if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
 				Log.d(BleManager.LOG_TAG, " Is Or Greater than M $mBluetoothDevice");
 				gatt = device.connectGatt(activity, false, this, BluetoothDevice.TRANSPORT_LE);
@@ -127,6 +138,9 @@ public class Peripheral extends BluetoothGattCallback {
 		connectCallback = null;
 		connected = false;
 		clearBuffers();
+		commandQueue.clear();
+		commandQueueBusy = false;
+
 		if (gatt != null) {
 			try {
 				gatt.disconnect();
@@ -240,6 +254,10 @@ public class Peripheral extends BluetoothGattCallback {
 		return connected;
 	}
 
+	public boolean isConnecting() {
+		return connecting;
+	}
+
 	public BluetoothDevice getDevice() {
 		return device;
 	}
@@ -272,11 +290,28 @@ public class Peripheral extends BluetoothGattCallback {
 
 		if (status != BluetoothGatt.GATT_SUCCESS) {
 		    gatt.close();
+		    // change the state to ensure the connection is teared down properly in the code below
+		    newState = BluetoothProfile.STATE_DISCONNECTED;
 		}
 
+		connecting = false;
 		if (newState == BluetoothProfile.STATE_CONNECTED) {
 			connected = true;
 
+			discoverServicesRunnable = new Runnable() {
+				@Override
+				public void run() {
+					try {
+						gatt.discoverServices();
+					} catch (NullPointerException e) {
+						Log.d(BleManager.LOG_TAG, "onConnectionStateChange connected but gatt of Run method was null");
+					}
+					discoverServicesRunnable = null;
+				}
+			};
+
+			mainHandler.post(discoverServicesRunnable);
+
 			sendConnectionEvent(device, "BleManagerConnectPeripheral", status);
 
 			if (connectCallback != null) {
@@ -287,9 +322,11 @@ public class Peripheral extends BluetoothGattCallback {
 
 		} else if (newState == BluetoothProfile.STATE_DISCONNECTED) {
 
-			this.disconnect(true);
+			if (discoverServicesRunnable != null) {
+				mainHandler.removeCallbacks(discoverServicesRunnable);
+				discoverServicesRunnable = null;
+			}
 
-			sendConnectionEvent(device, "BleManagerDisconnectPeripheral", status);
 			List<Callback> callbacks = Arrays.asList(writeCallback, retrieveServicesCallback, readRSSICallback,
 					readCallback, registerNotifyCallback, requestMTUCallback);
 			for (Callback currentCallback : callbacks) {
@@ -308,6 +345,10 @@ public class Peripheral extends BluetoothGattCallback {
 			readRSSICallback = null;
 			registerNotifyCallback = null;
 			requestMTUCallback = null;
+			commandQueue.clear();
+			commandQueueBusy = false;
+
+			this.disconnect(true);
 		}
 
 	}
@@ -367,54 +408,60 @@ public class Peripheral extends BluetoothGattCallback {
 	@Override
 	public void onCharacteristicRead(BluetoothGatt gatt, BluetoothGattCharacteristic characteristic, int status) {
 		super.onCharacteristicRead(gatt, characteristic, status);
-		Log.d(BleManager.LOG_TAG, "onCharacteristicRead " + characteristic);
 
-		if (readCallback != null) {
-
-			if (status == BluetoothGatt.GATT_SUCCESS) {
-				byte[] dataValue = characteristic.getValue();
-
-				if (readCallback != null) {
-					readCallback.invoke(null, BleManager.bytesToWritableArray(dataValue));
-				}
-			} else {
-				readCallback.invoke("Error reading " + characteristic.getUuid() + " status=" + status, null);
+		if (status != BluetoothGatt.GATT_SUCCESS) {
+			if (status == GATT_AUTH_FAIL || status == GATT_INSUFFICIENT_AUTHENTICATION) {
+				Log.d(BleManager.LOG_TAG, "Read needs bonding");
+				// *not* doing completedCommand()
+				return;
 			}
-
+			sendBackrError(readCallback, "Error reading " + characteristic.getUuid() + " status=" + status);
+		} else if (readCallback != null) {
+			final byte[] dataValue = copyOf(characteristic.getValue());
+			final Callback callback = readCallback;
 			readCallback = null;
-
+			mainHandler.post(new Runnable() {
+					@Override
+					public void run() {
+						callback.invoke(null, BleManager.bytesToWritableArray(dataValue));
+					}
+				});
 		}
+
+		completedCommand();
 	}
 
 	@Override
 	public void onCharacteristicWrite(BluetoothGatt gatt, BluetoothGattCharacteristic characteristic, int status) {
 		super.onCharacteristicWrite(gatt, characteristic, status);
 
-		if (writeCallback != null) {
-
-			if (writeQueue.size() > 0) {
-				byte[] data = writeQueue.get(0);
-				writeQueue.remove(0);
-				doWrite(characteristic, data);
-			} else {
-
-				if (status == BluetoothGatt.GATT_SUCCESS) {
-					writeCallback.invoke();
-				} else {
-					Log.e(BleManager.LOG_TAG, "Error onCharacteristicWrite:" + status);
-					writeCallback.invoke("Error writing status: " + status);
-				}
-
-				writeCallback = null;
+		if (writeQueue.size() > 0) {
+			byte[] data = writeQueue.get(0);
+			writeQueue.remove(0);
+			doWrite(characteristic, data, writeCallback);
+		} else if (status != BluetoothGatt.GATT_SUCCESS) {
+			if (status == GATT_AUTH_FAIL || status == GATT_INSUFFICIENT_AUTHENTICATION) {
+				Log.d(BleManager.LOG_TAG, "Write needs bonding");
+				// *not* doing completedCommand()
+				return;
 			}
-		} else {
-			Log.e(BleManager.LOG_TAG, "No callback on write");
+			sendBackrError(writeCallback, "Error writing " + characteristic.getUuid() + " status=" + status);
+		} else if (writeCallback != null) {
+			final Callback callback = writeCallback;
+			mainHandler.post(new Runnable() {
+					@Override
+					public void run() {
+						callback.invoke();
+					}
+				});
+			writeCallback = null;
 		}
+
+		completedCommand();
 	}
 
 	@Override
 	public void onDescriptorWrite(BluetoothGatt gatt, BluetoothGattDescriptor descriptor, int status) {
-		super.onDescriptorWrite(gatt, descriptor, status);
 		if (registerNotifyCallback != null) {
 			if (status == BluetoothGatt.GATT_SUCCESS) {
 				registerNotifyCallback.invoke();
@@ -428,6 +475,8 @@ public class Peripheral extends BluetoothGattCallback {
 		} else {
 			Log.e(BleManager.LOG_TAG, "onDescriptorWrite with no callback");
 		}
+
+		completedCommand();
 	}
 
 	@Override
@@ -454,85 +503,90 @@ public class Peripheral extends BluetoothGattCallback {
 			entry.getValue().resetBuffer();
 	}
 
-	private void setNotify(UUID serviceUUID, UUID characteristicUUID, Boolean notify, Integer buffer,
-			Callback callback) {
-		if (!isConnected()) {
+	private void setNotify(UUID serviceUUID, UUID characteristicUUID, final Boolean notify, Callback callback) {
+		if (! isConnected() || gatt == null) {
 			callback.invoke("Device is not connected", null);
 			return;
 		}
-		Log.d(BleManager.LOG_TAG, "setNotify");
 
-		if (gatt == null) {
-			callback.invoke("BluetoothGatt is null");
+		BluetoothGattService service = gatt.getService(serviceUUID);
+		final BluetoothGattCharacteristic characteristic = findNotifyCharacteristic(service, characteristicUUID);
+
+		if (characteristic == null) {
+			callback.invoke("Characteristic " + characteristicUUID + " not found");			
 			return;
 		}
-		BluetoothGattService service = gatt.getService(serviceUUID);
-		BluetoothGattCharacteristic characteristic = findNotifyCharacteristic(service, characteristicUUID);
 
-		if (characteristic != null) {
-			if (gatt.setCharacteristicNotification(characteristic, notify)) {
+		if (! gatt.setCharacteristicNotification(characteristic, notify)) {
+			callback.invoke("Failed to register notification for " + characteristicUUID);
+			return;
+		}
 
-				if (buffer > 1) {
-					Log.d(BleManager.LOG_TAG, "Characteristic buffering " + characteristicUUID + " count:" + buffer);
-					String key = this.bufferedCharacteristicsKey(serviceUUID.toString(), characteristicUUID.toString());
-					this.bufferedCharacteristics.put(key, new NotifyBufferContainer(key, buffer));
-				}
+		final BluetoothGattDescriptor descriptor = characteristic.getDescriptor(UUIDHelper.uuidFromString(CHARACTERISTIC_NOTIFICATION_CONFIG));
+		if (descriptor == null) {
+			callback.invoke("Set notification failed for " + characteristicUUID);
+			return;
+		}
+
+		// Prefer notify over indicate
+		byte[] value;
+		if ((characteristic.getProperties() & BluetoothGattCharacteristic.PROPERTY_NOTIFY) != 0) {
+			Log.d(BleManager.LOG_TAG, "Characteristic " + characteristicUUID + " set NOTIFY");
+			value = notify ? BluetoothGattDescriptor.ENABLE_NOTIFICATION_VALUE : BluetoothGattDescriptor.DISABLE_NOTIFICATION_VALUE;
+		} else if ((characteristic.getProperties() & BluetoothGattCharacteristic.PROPERTY_INDICATE) != 0) {
+			Log.d(BleManager.LOG_TAG, "Characteristic " + characteristicUUID + " set INDICATE");
+			value = notify ? BluetoothGattDescriptor.ENABLE_INDICATION_VALUE : BluetoothGattDescriptor.DISABLE_NOTIFICATION_VALUE;
+		} else {
+			String msg = "Characteristic " + characteristicUUID + " does not have NOTIFY or INDICATE property set";
+			Log.d(BleManager.LOG_TAG, msg);
+			callback.invoke(msg);
+			return;
+		}
+		final byte[] finalValue = notify ? value : BluetoothGattDescriptor.DISABLE_NOTIFICATION_VALUE;
+		final Callback finalCallback = callback;
 
-				BluetoothGattDescriptor descriptor = characteristic
-						.getDescriptor(UUIDHelper.uuidFromString(CHARACTERISTIC_NOTIFICATION_CONFIG));
-				if (descriptor != null) {
-
-					// Prefer notify over indicate
-					if ((characteristic.getProperties() & BluetoothGattCharacteristic.PROPERTY_NOTIFY) != 0) {
-						Log.d(BleManager.LOG_TAG, "Characteristic " + characteristicUUID + " set NOTIFY");
-						descriptor.setValue(notify ? BluetoothGattDescriptor.ENABLE_NOTIFICATION_VALUE
-								: BluetoothGattDescriptor.DISABLE_NOTIFICATION_VALUE);
-					} else if ((characteristic.getProperties() & BluetoothGattCharacteristic.PROPERTY_INDICATE) != 0) {
-						Log.d(BleManager.LOG_TAG, "Characteristic " + characteristicUUID + " set INDICATE");
-						descriptor.setValue(notify ? BluetoothGattDescriptor.ENABLE_INDICATION_VALUE
-								: BluetoothGattDescriptor.DISABLE_NOTIFICATION_VALUE);
-					} else {
-						Log.d(BleManager.LOG_TAG, "Characteristic " + characteristicUUID
-								+ " does not have NOTIFY or INDICATE property set");
+		boolean result = commandQueue.add(new Runnable() {
+				@Override
+				public void run() {
+					if (! isConnected()) {
+						completedCommand();
+						return;
 					}
 
+					boolean result = false;
 					try {
-						registerNotifyCallback = callback;
-						if (gatt.writeDescriptor(descriptor)) {
-							Log.d(BleManager.LOG_TAG, "setNotify complete");
-						} else {
-							registerNotifyCallback = null;
-							callback.invoke(
-									"Failed to set client characteristic notification for " + characteristicUUID);
-						}
-					} catch (Exception e) {
-						Log.d(BleManager.LOG_TAG, "Error on setNotify", e);
-						callback.invoke("Failed to set client characteristic notification for " + characteristicUUID
-								+ ", error: " + e.getMessage());
+						result = gatt.setCharacteristicNotification(characteristic, notify);
+						// Then write to descriptor
+						descriptor.setValue(finalValue);
+						registerNotifyCallback = finalCallback;
+						result = gatt.writeDescriptor(descriptor);
+					} catch(Exception e) {
+						Log.d(BleManager.LOG_TAG, "Exception in setNotify", e);
 					}
 
-				} else {
-					callback.invoke("Set notification failed for " + characteristicUUID);
+					if (! result) {
+						sendBackrError(registerNotifyCallback, "writeDescriptor failed for descriptor: " + descriptor.getUuid());
+						registerNotifyCallback = null;
+						completedCommand();
+					}
 				}
+			});
 
-			} else {
-				callback.invoke("Failed to register notification for " + characteristicUUID);
-			}
-
+		if (result) {
+			nextCommand();
 		} else {
-			callback.invoke("Characteristic " + characteristicUUID + " not found");
+			Log.e(BleManager.LOG_TAG, "Could not enqueue setNotify command");
 		}
-
 	}
 
 	public void registerNotify(UUID serviceUUID, UUID characteristicUUID, Integer buffer, Callback callback) {
 		Log.d(BleManager.LOG_TAG, "registerNotify");
-		this.setNotify(serviceUUID, characteristicUUID, true, buffer, callback);
+		this.setNotify(serviceUUID, characteristicUUID, true, callback);
 	}
 
 	public void removeNotify(UUID serviceUUID, UUID characteristicUUID, Callback callback) {
 		Log.d(BleManager.LOG_TAG, "removeNotify");
-		this.setNotify(serviceUUID, characteristicUUID, false, 1, callback);
+		this.setNotify(serviceUUID, characteristicUUID, false, callback);
 	}
 
 	// Some devices reuse UUIDs across characteristics, so we can't use
@@ -572,30 +626,100 @@ public class Peripheral extends BluetoothGattCallback {
 
 	public void read(UUID serviceUUID, UUID characteristicUUID, Callback callback) {
 
-		if (!isConnected()) {
+		if (!isConnected() || gatt == null) {
 			callback.invoke("Device is not connected", null);
 			return;
 		}
-		if (gatt == null) {
-			callback.invoke("BluetoothGatt is null", null);
-			return;
-		}
 
 		BluetoothGattService service = gatt.getService(serviceUUID);
-		BluetoothGattCharacteristic characteristic = findReadableCharacteristic(service, characteristicUUID);
+		final BluetoothGattCharacteristic characteristic = findReadableCharacteristic(service, characteristicUUID);
 
 		if (characteristic == null) {
 			callback.invoke("Characteristic " + characteristicUUID + " not found.", null);
+			return;
+		}
+
+		final Callback reactcb = callback;
+		boolean result = commandQueue.add(new Runnable() {
+				@Override
+				public void run() {
+					readCallback = reactcb;
+					if (! gatt.readCharacteristic(characteristic)) {
+						readCallback = null;
+						sendBackrError(reactcb, "Read failed");
+						completedCommand();
+					}
+				}
+			});
+
+		if (result) {
+			nextCommand();
 		} else {
-			readCallback = callback;
-			if (!gatt.readCharacteristic(characteristic)) {
-				readCallback = null;
-				callback.invoke("Read failed", null);
+			Log.d(BleManager.LOG_TAG, "Could not queue read characteristic command");
+		}
+	}
+
+	private void sendBackrError(final Callback callback, final String message) {
+		new Handler(Looper.getMainLooper()).post(new Runnable() {
+				@Override
+				public void run() {
+					callback.invoke(message, null);
+				}
+			});
+	}
+
+	private byte[] copyOf(byte[] source) {
+		if (source == null) return new byte[0];
+		final int sourceLength = source.length;
+		final byte[] copy = new byte[sourceLength];
+		System.arraycopy(source, 0, copy, 0, sourceLength);
+		return copy;
+	}
+
+	private void completedCommand() {
+		commandQueue.poll();
+		commandQueueBusy = false;
+		nextCommand();
+	}
+
+	private void nextCommand() {
+		synchronized (this) {
+			if (commandQueueBusy) {
+				Log.d(BleManager.LOG_TAG, "Command queue busy");
+				return;
+			}
+
+			final Runnable nextCommand = commandQueue.peek();
+			if (nextCommand == null) {
+				Log.d(BleManager.LOG_TAG, "Command queue empty");
+				return;
+			}
+
+			// Check if we still have a valid gatt object
+			if (gatt == null) {
+				Log.d(BleManager.LOG_TAG, "Error, gatt is null");
+				commandQueue.clear();
+				commandQueueBusy = false;
+				return;
 			}
+
+			// Execute the next command in the queue
+			commandQueueBusy = true;
+			mainHandler.post(new Runnable() {
+					@Override
+					public void run() {
+						try {
+							nextCommand.run();
+						} catch (Exception ex) {
+							Log.d(BleManager.LOG_TAG, "Error, command exception");
+							completedCommand();
+						}
+					}
+				});
 		}
 	}
 
-	public void readRSSI(Callback callback) {
+	public void readRSSI(final Callback callback) {
 		if (!isConnected()) {
 			callback.invoke("Device is not connected", null);
 			return;
@@ -605,11 +729,26 @@ public class Peripheral extends BluetoothGattCallback {
 			return;
 		}
 
-		readRSSICallback = callback;
+		final boolean result = commandQueue.add(new Runnable() {
+			@Override
+			public void run() {
+			    if (isConnected()) {
+				readRSSICallback = callback;
+				if (! gatt.readRemoteRssi()) {
+				    readCallback = null;
+				    sendBackrError(callback, "Read RSSI failed");
+				    completedCommand();
+				}
+			    } else {
+				completedCommand();
+			    }
+			}
+		    });
 
-		if (!gatt.readRemoteRssi()) {
-			readCallback = null;
-			callback.invoke("Read RSSI failed", null);
+		if (result) {
+		    nextCommand();
+		} else {
+		    Log.d(BleManager.LOG_TAG, "Could not queue readRemoteRssi command");
 		}
 	}
 
@@ -667,112 +806,109 @@ public class Peripheral extends BluetoothGattCallback {
 		return null;
 	}
 
-	public boolean doWrite(BluetoothGattCharacteristic characteristic, byte[] data) {
-		characteristic.setValue(data);
+	public boolean doWrite(final BluetoothGattCharacteristic characteristic, byte[] data, final Callback callback) {
+		final byte[] copyOfData = copyOf(data);
+		boolean result = commandQueue.add(new Runnable() {
+				@Override
+				public void run() {
+					characteristic.setValue(copyOfData);
+					if (characteristic.getWriteType() == BluetoothGattCharacteristic.WRITE_TYPE_DEFAULT)
+						writeCallback = callback;
+					else
+						writeCallback = null;
+					if (!gatt.writeCharacteristic(characteristic)) {
+						sendBackrError(writeCallback, "Write failed");
+						writeCallback = null;
+						completedCommand();
+					}
+				}
+			});
 
-		if (!gatt.writeCharacteristic(characteristic)) {
-			Log.d(BleManager.LOG_TAG, "Error on doWrite");
-			return false;
+		if (result) {
+			nextCommand();
+		} else {
+			Log.d(BleManager.LOG_TAG, "Could not queue write characteristic command");
 		}
-		return true;
+
+		return result;
 	}
 
-	public void write(UUID serviceUUID, UUID characteristicUUID, byte[] data, Integer maxByteSize,
-			Integer queueSleepTime, Callback callback, int writeType) {
-		if (!isConnected()) {
+	public void write(UUID serviceUUID, UUID characteristicUUID, byte[] data, Integer maxByteSize, Integer queueSleepTime, Callback callback, int writeType) {
+		if (!isConnected() || gatt == null) {
 			callback.invoke("Device is not connected", null);
 			return;
 		}
-		if (gatt == null) {
-			callback.invoke("BluetoothGatt is null");
-		} else {
-			BluetoothGattService service = gatt.getService(serviceUUID);
-			BluetoothGattCharacteristic characteristic = findWritableCharacteristic(service, characteristicUUID,
-					writeType);
 
-			if (characteristic == null) {
-				callback.invoke("Characteristic " + characteristicUUID + " not found.");
-			} else {
-				characteristic.setWriteType(writeType);
+		BluetoothGattService service = gatt.getService(serviceUUID);
+		BluetoothGattCharacteristic characteristic = findWritableCharacteristic(service, characteristicUUID, writeType);
 
-				if (writeQueue.size() > 0) {
-					callback.invoke("You have already an queued message");
-					return;
-				}
+		if (characteristic == null) {
+			callback.invoke("Characteristic " + characteristicUUID + " not found.");
+			return;
+		}
 
-				if (writeCallback != null) {
-					callback.invoke("You're already writing");
-					return;
-				}
+		characteristic.setWriteType(writeType);
 
-				if (writeQueue.size() == 0 && writeCallback == null) {
+		if (data.length <= maxByteSize) {
+			if (! doWrite(characteristic, data, callback)) {
+				callback.invoke("Write failed");
+				writeCallback = null;
+			} else {
+				if (BluetoothGattCharacteristic.WRITE_TYPE_NO_RESPONSE == writeType) {
+					callback.invoke();
+				}
+			}
+		} else {
+			int dataLength = data.length;
+			int count = 0;
+			byte[] firstMessage = null;
+			List<byte[]> splittedMessage = new ArrayList<>();
+
+			while (count < dataLength && (dataLength - count > maxByteSize)) {
+				if (count == 0) {
+					firstMessage = Arrays.copyOfRange(data, count, count + maxByteSize);
+				} else {
+					byte[] splitMessage = Arrays.copyOfRange(data, count, count + maxByteSize);
+					splittedMessage.add(splitMessage);
+				}
+				count += maxByteSize;
+			}
+			if (count < dataLength) {
+				// Other bytes in queue
+				byte[] splitMessage = Arrays.copyOfRange(data, count, data.length);
+				splittedMessage.add(splitMessage);
+			}
 
-					if (BluetoothGattCharacteristic.WRITE_TYPE_DEFAULT == writeType) {
-						writeCallback = callback;
+			if (BluetoothGattCharacteristic.WRITE_TYPE_DEFAULT == writeType) {
+				writeQueue.addAll(splittedMessage);
+				if (! doWrite(characteristic, firstMessage, callback)) {
+					writeQueue.clear();
+					writeCallback = null;
+					callback.invoke("Write failed");
+				}
+			} else {
+				try {
+					boolean writeError = false;
+					if (! doWrite(characteristic, firstMessage, callback)) {
+						writeError = true;
+						callback.invoke("Write failed");
 					}
-
-					if (data.length > maxByteSize) {
-						int dataLength = data.length;
-						int count = 0;
-						byte[] firstMessage = null;
-						List<byte[]> splittedMessage = new ArrayList<>();
-
-						while (count < dataLength && (dataLength - count > maxByteSize)) {
-							if (count == 0) {
-								firstMessage = Arrays.copyOfRange(data, count, count + maxByteSize);
-							} else {
-								byte[] splitMessage = Arrays.copyOfRange(data, count, count + maxByteSize);
-								splittedMessage.add(splitMessage);
-							}
-							count += maxByteSize;
-						}
-						if (count < dataLength) {
-							// Other bytes in queue
-							byte[] splitMessage = Arrays.copyOfRange(data, count, data.length);
-							splittedMessage.add(splitMessage);
-						}
-
-						if (BluetoothGattCharacteristic.WRITE_TYPE_DEFAULT == writeType) {
-							writeQueue.addAll(splittedMessage);
-							if (!doWrite(characteristic, firstMessage)) {
-								writeQueue.clear();
-								writeCallback = null;
+					if (! writeError) {
+						Thread.sleep(queueSleepTime);
+						for (byte[] message : splittedMessage) {
+							if (! doWrite(characteristic, message, callback)) {
+								writeError = true;
 								callback.invoke("Write failed");
+								break;
 							}
-						} else {
-							try {
-								boolean writeError = false;
-								if (!doWrite(characteristic, firstMessage)) {
-									writeError = true;
-									callback.invoke("Write failed");
-								}
-								if (!writeError) {
-									Thread.sleep(queueSleepTime);
-									for (byte[] message : splittedMessage) {
-										if (!doWrite(characteristic, message)) {
-											writeError = true;
-											callback.invoke("Write failed");
-											break;
-										}
-										Thread.sleep(queueSleepTime);
-									}
-									if (!writeError) {
-										callback.invoke();
-									}
-								}
-							} catch (InterruptedException e) {
-								callback.invoke("Error during writing");
-							}
+							Thread.sleep(queueSleepTime);
 						}
-					} else if (doWrite(characteristic, data)) {
-						Log.d(BleManager.LOG_TAG, "Write completed");
-						if (BluetoothGattCharacteristic.WRITE_TYPE_NO_RESPONSE == writeType) {
+						if (! writeError) {
 							callback.invoke();
 						}
-					} else {
-						callback.invoke("Write failed");
-						writeCallback = null;
 					}
+				} catch (InterruptedException e) {
+					callback.invoke("Error during writing");
 				}
 			}
 		}

package/ios/BleManager.m

@@ -720,14 +720,6 @@ RCT_EXPORT_METHOD(requestMTU:(NSString *)deviceUUID mtu:(NSInteger)mtu callback:
     callback(@[@"Not supported"]);
 }
 
-RCT_EXPORT_METHOD(addListener : (NSString *)eventName) {
-    // Keep: Required for RN built in Event Emitter Calls.
-}
-
-RCT_EXPORT_METHOD(removeListeners : (NSInteger)count) {
-    // Keep: Required for RN built in Event Emitter Calls.
-}
-
 - (void)peripheral:(CBPeripheral *)peripheral didWriteValueForCharacteristic:(CBCharacteristic *)characteristic error:(NSError *)error {
     NSLog(@"didWrite");
     

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-ble-manager",
-  "version": "7.6.2",
+  "version": "8.0.2",
   "description": "A BLE module for react native.",
   "main": "BleManager",
   "types": "./index.d.ts",

package/ISSUE_TEMPLATE.md

@@ -1,33 +0,0 @@
-### Before open an issue
-- Check the closed issues, your question maybe is not new.
-- We can't debug your physical devices.
-- If the library is not working 99% you did something wrong, in the code or in the phone permissions.
-
-### Version
-
-Tell us which versions you are using:
-
-- react-native-ble-manager v?.?.?
-- react-native v?.?
-- iOS/Android v.?
-
-### Expected behaviour
-
-Tell us what should happen
-
-### Actual behaviour
-
-Tell us what happens instead
-
-### Steps to reproduce
-
-1.
-2.
-3.
-
-### Stack trace and console log
-
-Hint: it would help a lot if you enable the debugger ("Pause on exceptions" in the "Source" panel of Chrome dev tools) and spot the place where the error is thrown
-
-```
-```