Merge pull request #78 from NordicSemiconductor/hotfix/fix-foreground-service

Hotfix/fix foreground service

app/build.gradle

@@ -1,13 +1,13 @@
 apply plugin: 'com.android.application'
 
 android {
-    compileSdkVersion 28
+    compileSdkVersion 29
-    buildToolsVersion "28.0.3"
+    buildToolsVersion "29.0.0"
 
     defaultConfig {
         applicationId "no.nordicsemi.android.nrftoolbox"
         minSdkVersion 18
-        targetSdkVersion 28
+        targetSdkVersion 29
         versionCode 69
         versionName "2.7.2"
         resConfigs "en"
@@ -42,9 +42,9 @@ dependencies {
     //noinspection GradleDependency
     implementation 'com.google.android.gms:play-services-wearable:10.2.0'
 
-    implementation 'androidx.appcompat:appcompat:1.1.0-alpha04'
+    implementation 'androidx.appcompat:appcompat:1.1.0-rc01'
-    implementation 'androidx.preference:preference:1.1.0-alpha04'
+    implementation 'androidx.preference:preference:1.1.0-rc01'
-    implementation 'com.google.android.material:material:1.1.0-alpha05'
+    implementation 'com.google.android.material:material:1.1.0-alpha09'
 
     implementation 'no.nordicsemi.android:log:2.2.0'
     implementation 'no.nordicsemi.android.support.v18:scanner:1.4.0'

app/src/main/AndroidManifest.xml

@@ -33,6 +33,7 @@
 	<uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
 	<uses-permission android:name="android.permission.NFC" />
 	<uses-permission android:name="no.nordicsemi.android.LOG" />
+	<uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
 
 	<uses-feature
 		android:name="android.hardware.bluetooth_le"

app/src/main/java/no/nordicsemi/android/nrftoolbox/cgms/CGMService.java

@@ -8,11 +8,12 @@ import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
+import android.os.Build;
+import android.util.SparseArray;
+
 import androidx.annotation.NonNull;
 import androidx.core.app.NotificationCompat;
 import androidx.localbroadcastmanager.content.LocalBroadcastManager;
-import android.util.SparseArray;
-
 import no.nordicsemi.android.log.Logger;
 import no.nordicsemi.android.nrftoolbox.FeaturesActivity;
 import no.nordicsemi.android.nrftoolbox.R;
@@ -147,7 +148,7 @@ public class CGMService extends BleProfileService implements CGMSManagerCallback
     @Override
     public void onDestroy() {
         // when user has disconnected from the sensor, we have to cancel the notification that we've created some milliseconds before using unbindService
-		cancelNotification();
+        stopForegroundService();
         unregisterReceiver(mDisconnectActionBroadcastReceiver);
 
         super.onDestroy();
@@ -155,14 +156,40 @@ public class CGMService extends BleProfileService implements CGMSManagerCallback
 
     @Override
     protected void onRebind() {
-		// when the activity rebinds to the service, remove the notification
+        startForegroundService();
-		cancelNotification();
     }
 
     @Override
     protected void onUnbind() {
-		// when the activity closes we need to show the notification that user is connected to the sensor
+        startForegroundService();
-		createNotification(R.string.csc_notification_connected_message, 0);
+    }
+
+    /**
+     * Sets the service as a foreground service
+     */
+    private void startForegroundService(){
+        // when the activity closes we need to show the notification that user is connected to the peripheral sensor
+        // We start the service as a foreground service as Android 8.0 (Oreo) onwards kills any running background services
+        final Notification notification = createNotification(R.string.uart_notification_connected_message, 0);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            startForeground(NOTIFICATION_ID, notification);
+        } else {
+            final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
+            nm.notify(NOTIFICATION_ID, notification);
+        }
+    }
+
+    /**
+     * Stops the service as a foreground service
+     */
+    private void stopForegroundService(){
+        // when the activity rebinds to the service, remove the notification and stop the foreground service
+        // on devices running Android 8.0 (Oreo) or above
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            stopForeground(true);
+        } else {
+            cancelNotification();
+        }
     }
 
     /**
@@ -172,7 +199,7 @@ public class CGMService extends BleProfileService implements CGMSManagerCallback
      *                     f.e. <code>&lt;string name="name"&gt;%s is connected&lt;/string&gt;</code>
      * @param defaults     signals that will be used to notify the user
      */
-	private void createNotification(final int messageResId, final int defaults) {
+    private Notification createNotification(final int messageResId, final int defaults) {
         final Intent parentIntent = new Intent(this, FeaturesActivity.class);
         parentIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
         final Intent targetIntent = new Intent(this, CGMSActivity.class);
@@ -189,9 +216,7 @@ public class CGMService extends BleProfileService implements CGMSManagerCallback
         builder.setShowWhen(defaults != 0).setDefaults(defaults).setAutoCancel(true).setOngoing(true);
         builder.addAction(new NotificationCompat.Action(R.drawable.ic_action_bluetooth, getString(R.string.csc_notification_action_disconnect), disconnectAction));
 
-		final Notification notification = builder.build();
+        return builder.build();
-		final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
-		nm.notify(NOTIFICATION_ID, notification);
     }
 
     /**

app/src/main/java/no/nordicsemi/android/nrftoolbox/csc/CSCService.java

@@ -30,10 +30,11 @@ import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
+import android.os.Build;
+
 import androidx.annotation.NonNull;
 import androidx.core.app.NotificationCompat;
 import androidx.localbroadcastmanager.content.LocalBroadcastManager;
-
 import no.nordicsemi.android.log.Logger;
 import no.nordicsemi.android.nrftoolbox.FeaturesActivity;
 import no.nordicsemi.android.nrftoolbox.R;
@@ -46,11 +47,17 @@ public class CSCService extends BleProfileService implements CSCManagerCallbacks
     private static final String TAG = "CSCService";
 
     public static final String BROADCAST_WHEEL_DATA = "no.nordicsemi.android.nrftoolbox.csc.BROADCAST_WHEEL_DATA";
-	/** Speed in meters per second. */
+    /**
+     * Speed in meters per second.
+     */
     public static final String EXTRA_SPEED = "no.nordicsemi.android.nrftoolbox.csc.EXTRA_SPEED";
-	/** Distance in meters. */
+    /**
+     * Distance in meters.
+     */
     public static final String EXTRA_DISTANCE = "no.nordicsemi.android.nrftoolbox.csc.EXTRA_DISTANCE";
-	/** Total distance in meters. */
+    /**
+     * Total distance in meters.
+     */
     public static final String EXTRA_TOTAL_DISTANCE = "no.nordicsemi.android.nrftoolbox.csc.EXTRA_TOTAL_DISTANCE";
 
     public static final String BROADCAST_CRANK_DATA = "no.nordicsemi.android.nrftoolbox.csc.BROADCAST_CRANK_DATA";
@@ -106,8 +113,7 @@ public class CSCService extends BleProfileService implements CSCManagerCallbacks
 
     @Override
     protected void onRebind() {
-		// when the activity rebinds to the service, remove the notification
+        stopForegroundService();
-		cancelNotification();
 
         if (isConnected()) {
             // This method will read the Battery Level value, if possible and then try to enable battery notifications (if it has NOTIFY property).
@@ -122,9 +128,7 @@ public class CSCService extends BleProfileService implements CSCManagerCallbacks
         // But we will still be receiving other values, if enabled.
         if (isConnected())
             mManager.disableBatteryLevelCharacteristicNotifications();
-
+        startForegroundService();
-		// when the activity closes we need to show the notification that user is connected to the sensor
-		createNotification(R.string.csc_notification_connected_message, 0);
     }
 
     @Override
@@ -154,16 +158,42 @@ public class CSCService extends BleProfileService implements CSCManagerCallbacks
         LocalBroadcastManager.getInstance(this).sendBroadcast(broadcast);
     }
 
+    /**
+     * Sets the service as a foreground service
+     */
+    private void startForegroundService(){
+        // when the activity closes we need to show the notification that user is connected to the peripheral sensor
+        // We start the service as a foreground service as Android 8.0 (Oreo) onwards kills any running background services
+        final Notification notification = createNotification(R.string.uart_notification_connected_message, 0);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            startForeground(NOTIFICATION_ID, notification);
+        } else {
+            final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
+            nm.notify(NOTIFICATION_ID, notification);
+        }
+    }
+
+    /**
+     * Stops the service as a foreground service
+     */
+    private void stopForegroundService(){
+        // when the activity rebinds to the service, remove the notification and stop the foreground service
+        // on devices running Android 8.0 (Oreo) or above
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            stopForeground(true);
+        } else {
+            cancelNotification();
+        }
+    }
+
     /**
      * Creates the notification
      *
-	 * @param messageResId
+     * @param messageResId the message resource id. The message must have one String parameter,<br />
-	 *            the message resource id. The message must have one String parameter,<br />
      *                     f.e. <code>&lt;string name="name"&gt;%s is connected&lt;/string&gt;</code>
      * @param defaults
-	 *            signals that will be used to notify the user
      */
-	private void createNotification(final int messageResId, final int defaults) {
+    private Notification createNotification(final int messageResId, final int defaults) {
         final Intent parentIntent = new Intent(this, FeaturesActivity.class);
         parentIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
         final Intent targetIntent = new Intent(this, CSCActivity.class);
@@ -180,9 +210,7 @@ public class CSCService extends BleProfileService implements CSCManagerCallbacks
         builder.setShowWhen(defaults != 0).setDefaults(defaults).setAutoCancel(true).setOngoing(true);
         builder.addAction(new NotificationCompat.Action(R.drawable.ic_action_bluetooth, getString(R.string.csc_notification_action_disconnect), disconnectAction));
 
-		final Notification notification = builder.build();
+        return builder.build();
-		final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
-		nm.notify(NOTIFICATION_ID, notification);
     }
 
     /**

app/src/main/java/no/nordicsemi/android/nrftoolbox/hts/HTSService.java

@@ -30,6 +30,8 @@ import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
+import android.os.Build;
+
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.core.app.NotificationCompat;
@@ -110,14 +112,12 @@ public class HTSService extends BleProfileService implements HTSManagerCallbacks
 
 	@Override
 	protected void onRebind() {
-		// when the activity rebinds to the service, remove the notification
+		stopForegroundService();
-		cancelNotification();
 	}
 
 	@Override
 	protected void onUnbind() {
-		// when the activity closes we need to show the notification that user is connected to the sensor
+		startForegroundService();
-		createNotification(R.string.hts_notification_connected_message, 0);
 	}
 
 	@Override
@@ -153,16 +153,42 @@ public class HTSService extends BleProfileService implements HTSManagerCallbacks
 		LocalBroadcastManager.getInstance(this).sendBroadcast(broadcast);
 	}
 
+	/**
+	 * Sets the service as a foreground service
+	 */
+	private void startForegroundService(){
+		// when the activity closes we need to show the notification that user is connected to the peripheral sensor
+		// We start the service as a foreground service as Android 8.0 (Oreo) onwards kills any running background services
+		final Notification notification = createNotification(R.string.uart_notification_connected_message, 0);
+		if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+			startForeground(NOTIFICATION_ID, notification);
+		} else {
+			final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
+			nm.notify(NOTIFICATION_ID, notification);
+		}
+	}
+
+	/**
+	 * Stops the service as a foreground service
+	 */
+	private void stopForegroundService(){
+		// when the activity rebinds to the service, remove the notification and stop the foreground service
+		// on devices running Android 8.0 (Oreo) or above
+		if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+			stopForeground(true);
+		} else {
+			cancelNotification();
+		}
+	}
+
 	/**
 	 * Creates the notification
-	 * 
 	 *  @param messageResId
 	 *            message resource id. The message must have one String parameter,<br />
 	 *            f.e. <code>&lt;string name="name"&gt;%s is connected&lt;/string&gt;</code>
 	 * @param defaults
-	 *            signals that will be used to notify the user
 	 */
-	private void createNotification(final int messageResId, final int defaults) {
+	private Notification createNotification(final int messageResId, final int defaults) {
 		final Intent parentIntent = new Intent(this, FeaturesActivity.class);
 		parentIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
 		final Intent targetIntent = new Intent(this, HTSActivity.class);
@@ -179,9 +205,7 @@ public class HTSService extends BleProfileService implements HTSManagerCallbacks
 		builder.setShowWhen(defaults != 0).setDefaults(defaults).setAutoCancel(true).setOngoing(true);
 		builder.addAction(new NotificationCompat.Action(R.drawable.ic_action_bluetooth, getString(R.string.hts_notification_action_disconnect), disconnectAction));
 
-		final Notification notification = builder.build();
+		return builder.build();
-		final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
-		nm.notify(NOTIFICATION_ID, notification);
 	}
 
 	/**

app/src/main/java/no/nordicsemi/android/nrftoolbox/profile/BleProfileService.java

@@ -33,12 +33,12 @@ import android.net.Uri;
 import android.os.Binder;
 import android.os.Handler;
 import android.os.IBinder;
-import androidx.annotation.NonNull;
-import androidx.annotation.StringRes;
-import androidx.localbroadcastmanager.content.LocalBroadcastManager;
 import android.util.Log;
 import android.widget.Toast;
 
+import androidx.annotation.NonNull;
+import androidx.annotation.StringRes;
+import androidx.localbroadcastmanager.content.LocalBroadcastManager;
 import no.nordicsemi.android.ble.BleManager;
 import no.nordicsemi.android.ble.BleManagerCallbacks;
 import no.nordicsemi.android.ble.utils.ILogger;
@@ -58,9 +58,13 @@ public abstract class BleProfileService extends Service implements BleManagerCal
     public static final String BROADCAST_BATTERY_LEVEL = "no.nordicsemi.android.nrftoolbox.BROADCAST_BATTERY_LEVEL";
     public static final String BROADCAST_ERROR = "no.nordicsemi.android.nrftoolbox.BROADCAST_ERROR";
 
-	/** The parameter passed when creating the service. Must contain the address of the sensor that we want to connect to */
+    /**
+     * The parameter passed when creating the service. Must contain the address of the sensor that we want to connect to
+     */
     public static final String EXTRA_DEVICE_ADDRESS = "no.nordicsemi.android.nrftoolbox.EXTRA_DEVICE_ADDRESS";
-	/** The key for the device name that is returned in {@link #BROADCAST_CONNECTION_STATE} with state {@link #STATE_CONNECTED}. */
+    /**
+     * The key for the device name that is returned in {@link #BROADCAST_CONNECTION_STATE} with state {@link #STATE_CONNECTED}.
+     */
     public static final String EXTRA_DEVICE_NAME = "no.nordicsemi.android.nrftoolbox.EXTRA_DEVICE_NAME";
     public static final String EXTRA_DEVICE = "no.nordicsemi.android.nrftoolbox.EXTRA_DEVICE";
     public static final String EXTRA_LOG_URI = "no.nordicsemi.android.nrftoolbox.EXTRA_LOG_URI";
@@ -142,6 +146,7 @@ public abstract class BleProfileService extends Service implements BleManagerCal
         /**
          * Sets whether the bound activity if changing configuration or not.
          * If <code>false</code>, we will turn off battery level notifications in onUnbind(..) method below.
+         *
          * @param changing true if the bound activity is finishing
          */
         public void setActivityIsChangingConfiguration(final boolean changing) {
@@ -187,6 +192,7 @@ public abstract class BleProfileService extends Service implements BleManagerCal
 
         /**
          * Returns the connection state of given device.
+         *
          * @return the connection state, as in {@link BleManager#getConnectionState()}.
          */
         public int getConnectionState() {
@@ -308,6 +314,7 @@ public abstract class BleProfileService extends Service implements BleManagerCal
 
     /**
      * Initializes the Ble Manager responsible for connecting to a single device.
+     *
      * @return a new BleManager object
      */
     @SuppressWarnings("rawtypes")
@@ -424,6 +431,7 @@ public abstract class BleProfileService extends Service implements BleManagerCal
     /**
      * This method should return false if the service needs to do some asynchronous work after if has disconnected from the device.
      * In that case the {@link #stopService()} method must be called when done.
+     *
      * @return true (default) to automatically stop the service when device is disconnected. False otherwise.
      */
     protected boolean stopWhenDisconnected() {
@@ -538,8 +546,7 @@ public abstract class BleProfileService extends Service implements BleManagerCal
     /**
      * Shows a message as a Toast notification. This method is thread safe, you can call it from any thread
      *
-	 * @param messageResId
+     * @param messageResId an resource id of the message to be shown
-	 *            an resource id of the message to be shown
      */
     protected void showToast(final int messageResId) {
         mHandler.post(() -> Toast.makeText(BleProfileService.this, messageResId, Toast.LENGTH_SHORT).show());
@@ -548,8 +555,7 @@ public abstract class BleProfileService extends Service implements BleManagerCal
     /**
      * Shows a message as a Toast notification. This method is thread safe, you can call it from any thread
      *
-	 * @param message
+     * @param message a message to be shown
-	 *            a message to be shown
      */
     protected void showToast(final String message) {
         mHandler.post(() -> Toast.makeText(BleProfileService.this, message, Toast.LENGTH_SHORT).show());

app/src/main/java/no/nordicsemi/android/nrftoolbox/proximity/ProximityService.java

@@ -34,11 +34,7 @@ import android.media.AudioManager;
 import android.media.MediaPlayer;
 import android.media.RingtoneManager;
 import android.net.Uri;
-import androidx.annotation.NonNull;
+import android.os.Build;
-import androidx.core.app.NotificationCompat;
-import androidx.core.app.NotificationManagerCompat;
-import androidx.core.content.ContextCompat;
-import androidx.localbroadcastmanager.content.LocalBroadcastManager;
 import android.text.TextUtils;
 import android.util.Log;
 
@@ -46,6 +42,11 @@ import java.io.IOException;
 import java.util.LinkedList;
 import java.util.List;
 
+import androidx.annotation.NonNull;
+import androidx.core.app.NotificationCompat;
+import androidx.core.app.NotificationManagerCompat;
+import androidx.core.content.ContextCompat;
+import androidx.localbroadcastmanager.content.LocalBroadcastManager;
 import no.nordicsemi.android.log.LogContract;
 import no.nordicsemi.android.nrftoolbox.FeaturesActivity;
 import no.nordicsemi.android.nrftoolbox.R;
@@ -449,8 +450,12 @@ public class ProximityService extends BleMulticonnectProfileService implements P
         }
 
         final Notification notification = builder.build();
-		final NotificationManagerCompat nm = NotificationManagerCompat.from(this);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
-		nm.notify(device.getAddress(), NOTIFICATION_ID, notification);
+            startForeground(NOTIFICATION_ID, notification);
+        } else {
+            final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
+            nm.notify(NOTIFICATION_ID, notification);
+        }
     }
 
     /**
@@ -475,9 +480,13 @@ public class ProximityService extends BleMulticonnectProfileService implements P
         builder.setTicker(getString(R.string.proximity_notification_link_loss_alert, name));
 
         final Notification notification = builder.build();
-		final NotificationManagerCompat nm = NotificationManagerCompat.from(this);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            startForeground(NOTIFICATION_ID, notification);
+        } else {
+            final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
             nm.notify(device.getAddress(), NOTIFICATION_ID, notification);
         }
+    }
 
     private NotificationCompat.Builder getNotificationBuilder() {
         final Intent parentIntent = new Intent(this, FeaturesActivity.class);
@@ -498,21 +507,34 @@ public class ProximityService extends BleMulticonnectProfileService implements P
      */
     private void cancelNotifications() {
         final NotificationManager nm = (NotificationManager) getSystemService(NOTIFICATION_SERVICE);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            stopForeground(true);
+        } else {
             nm.cancel(NOTIFICATION_ID);
+        }
 
         final List<BluetoothDevice> managedDevices = getManagedDevices();
         for (final BluetoothDevice device : managedDevices) {
+            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+                stopForeground(true);
+            } else {
                 nm.cancel(device.getAddress(), NOTIFICATION_ID);
             }
         }
+    }
 
     /**
      * Cancels the existing notification for given device. If there is no active notification this method does nothing
      */
     private void cancelNotification(final BluetoothDevice device) {
         final NotificationManager nm = (NotificationManager) getSystemService(NOTIFICATION_SERVICE);
+
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            stopForeground(true);
+        } else {
             nm.cancel(device.getAddress(), NOTIFICATION_ID);
         }
+    }
 
     private void initializeAlarm() {
         mDevicesWithAlarm = new LinkedList<>();

app/src/main/java/no/nordicsemi/android/nrftoolbox/rsc/RSCService.java

@@ -30,12 +30,13 @@ import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
+import android.os.Build;
 import android.os.Handler;
+
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.core.app.NotificationCompat;
 import androidx.localbroadcastmanager.content.LocalBroadcastManager;
-
 import no.nordicsemi.android.log.Logger;
 import no.nordicsemi.android.nrftoolbox.FeaturesActivity;
 import no.nordicsemi.android.nrftoolbox.R;
@@ -64,13 +65,21 @@ public class RSCService extends BleProfileService implements RSCManagerCallbacks
 
     private RSCManager mManager;
 
-	/** The last value of a cadence */
+    /**
+     * The last value of a cadence
+     */
     private float mCadence;
-	/** Trip distance in cm */
+    /**
+     * Trip distance in cm
+     */
     private long mDistance;
-	/** Stride length in cm */
+    /**
+     * Stride length in cm
+     */
     private Integer mStrideLength;
-	/** Number of steps in the trip */
+    /**
+     * Number of steps in the trip
+     */
     private int mStepsNumber;
     private boolean mTaskInProgress;
     private final Handler mHandler = new Handler();
@@ -110,7 +119,7 @@ public class RSCService extends BleProfileService implements RSCManagerCallbacks
     @Override
     public void onDestroy() {
         // when user has disconnected from the sensor, we have to cancel the notification that we've created some milliseconds before using unbindService
-		cancelNotification();
+        stopForegroundService();
         unregisterReceiver(mDisconnectActionBroadcastReceiver);
 
         super.onDestroy();
@@ -118,8 +127,7 @@ public class RSCService extends BleProfileService implements RSCManagerCallbacks
 
     @Override
     protected void onRebind() {
-		// when the activity rebinds to the service, remove the notification
+        stopForegroundService();
-		cancelNotification();
 
         if (isConnected()) {
             // This method will read the Battery Level value, if possible and then try to enable battery notifications (if it has NOTIFY property).
@@ -135,8 +143,7 @@ public class RSCService extends BleProfileService implements RSCManagerCallbacks
         if (isConnected())
             mManager.disableBatteryLevelCharacteristicNotifications();
 
-		// when the activity closes we need to show the notification that user is connected to the sensor
+        startForegroundService();
-		createNotification(R.string.rsc_notification_connected_message, 0);
     }
 
     private final Runnable mUpdateStridesTask = new Runnable() {
@@ -196,16 +203,42 @@ public class RSCService extends BleProfileService implements RSCManagerCallbacks
         LocalBroadcastManager.getInstance(this).sendBroadcast(broadcast);
     }
 
+    /**
+     * Sets the service as a foreground service
+     */
+    private void startForegroundService(){
+        // when the activity closes we need to show the notification that user is connected to the peripheral sensor
+        // We start the service as a foreground service as Android 8.0 (Oreo) onwards kills any running background services
+        final Notification notification = createNotification(R.string.uart_notification_connected_message, 0);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            startForeground(NOTIFICATION_ID, notification);
+        } else {
+            final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
+            nm.notify(NOTIFICATION_ID, notification);
+        }
+    }
+
+    /**
+     * Stops the service as a foreground service
+     */
+    private void stopForegroundService(){
+        // when the activity rebinds to the service, remove the notification and stop the foreground service
+        // on devices running Android 8.0 (Oreo) or above
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            stopForeground(true);
+        } else {
+            cancelNotification();
+        }
+    }
+
     /**
      * Creates the notification
      *
-	 * @param messageResId
+     * @param messageResId message resource id. The message must have one String parameter,<br />
-	 *            message resource id. The message must have one String parameter,<br />
      *                     f.e. <code>&lt;string name="name"&gt;%s is connected&lt;/string&gt;</code>
      * @param defaults
-	 *            signals that will be used to notify the user
      */
-	private void createNotification(final int messageResId, final int defaults) {
+    private Notification createNotification(final int messageResId, final int defaults) {
         final Intent parentIntent = new Intent(this, FeaturesActivity.class);
         parentIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
         final Intent targetIntent = new Intent(this, RSCActivity.class);
@@ -222,9 +255,7 @@ public class RSCService extends BleProfileService implements RSCManagerCallbacks
         builder.setShowWhen(defaults != 0).setDefaults(defaults).setAutoCancel(true).setOngoing(true);
         builder.addAction(new NotificationCompat.Action(R.drawable.ic_action_bluetooth, getString(R.string.rsc_notification_action_disconnect), disconnectAction));
 
-		final Notification notification = builder.build();
+        return builder.build();
-		final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
-		nm.notify(NOTIFICATION_ID, notification);
     }
 
     /**

app/src/main/java/no/nordicsemi/android/nrftoolbox/template/TemplateService.java

@@ -30,10 +30,11 @@ import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
+import android.os.Build;
+
 import androidx.annotation.NonNull;
 import androidx.core.app.NotificationCompat;
 import androidx.localbroadcastmanager.content.LocalBroadcastManager;
-
 import no.nordicsemi.android.log.Logger;
 import no.nordicsemi.android.nrftoolbox.FeaturesActivity;
 import no.nordicsemi.android.nrftoolbox.R;
@@ -96,7 +97,7 @@ public class TemplateService extends BleProfileService implements TemplateManage
     @Override
     public void onDestroy() {
         // when user has disconnected from the sensor, we have to cancel the notification that we've created some milliseconds before using unbindService
-		cancelNotification();
+        stopForegroundService();
         unregisterReceiver(mDisconnectActionBroadcastReceiver);
 
         super.onDestroy();
@@ -104,14 +105,12 @@ public class TemplateService extends BleProfileService implements TemplateManage
 
     @Override
     protected void onRebind() {
-		// when the activity rebinds to the service, remove the notification
+        stopForegroundService();
-		cancelNotification();
     }
 
     @Override
     protected void onUnbind() {
-		// when the activity closes we need to show the notification that user is connected to the sensor
+        startForegroundService();
-		createNotification(R.string.template_notification_connected_message, 0);
     }
 
     @Override
@@ -132,6 +131,34 @@ public class TemplateService extends BleProfileService implements TemplateManage
 
     }
 
+    /**
+     * Sets the service as a foreground service
+     */
+    private void startForegroundService(){
+        // when the activity closes we need to show the notification that user is connected to the peripheral sensor
+        // We start the service as a foreground service as Android 8.0 (Oreo) onwards kills any running background services
+        final Notification notification = createNotification(R.string.uart_notification_connected_message, 0);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            startForeground(NOTIFICATION_ID, notification);
+        } else {
+            final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
+            nm.notify(NOTIFICATION_ID, notification);
+        }
+    }
+
+    /**
+     * Stops the service as a foreground service
+     */
+    private void stopForegroundService(){
+        // when the activity rebinds to the service, remove the notification and stop the foreground service
+        // on devices running Android 8.0 (Oreo) or above
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            stopForeground(true);
+        } else {
+            cancelNotification();
+        }
+    }
+
     /**
      * Creates the notification.
      *
@@ -139,7 +166,7 @@ public class TemplateService extends BleProfileService implements TemplateManage
      *                     f.e. <code>&lt;string name="name"&gt;%s is connected&lt;/string&gt;</code>
      * @param defaults     signals that will be used to notify the user
      */
-	private void createNotification(final int messageResId, final int defaults) {
+    private Notification createNotification(final int messageResId, final int defaults) {
         final Intent parentIntent = new Intent(this, FeaturesActivity.class);
         parentIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
         final Intent targetIntent = new Intent(this, TemplateActivity.class);
@@ -156,9 +183,7 @@ public class TemplateService extends BleProfileService implements TemplateManage
         builder.setShowWhen(defaults != 0).setDefaults(defaults).setAutoCancel(true).setOngoing(true);
         builder.addAction(new NotificationCompat.Action(R.drawable.ic_action_bluetooth, getString(R.string.template_notification_action_disconnect), disconnectAction));
 
-		final Notification notification = builder.build();
+        return builder.build();
-		final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
-		nm.notify(NOTIFICATION_ID, notification);
     }
 
     /**

app/src/main/java/no/nordicsemi/android/nrftoolbox/uart/UARTService.java

@@ -30,9 +30,7 @@ import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
-import androidx.annotation.NonNull;
+import android.os.Build;
-import androidx.core.app.NotificationCompat;
-import androidx.localbroadcastmanager.content.LocalBroadcastManager;
 import android.text.TextUtils;
 import android.util.Log;
 
@@ -42,6 +40,9 @@ import com.google.android.gms.wearable.Node;
 import com.google.android.gms.wearable.NodeApi;
 import com.google.android.gms.wearable.Wearable;
 
+import androidx.annotation.NonNull;
+import androidx.core.app.NotificationCompat;
+import androidx.localbroadcastmanager.content.LocalBroadcastManager;
 import no.nordicsemi.android.log.Logger;
 import no.nordicsemi.android.nrftoolbox.FeaturesActivity;
 import no.nordicsemi.android.nrftoolbox.R;
@@ -57,13 +58,21 @@ public class UARTService extends BleProfileService implements UARTManagerCallbac
     public static final String BROADCAST_UART_RX = "no.nordicsemi.android.nrftoolbox.uart.BROADCAST_UART_RX";
     public static final String EXTRA_DATA = "no.nordicsemi.android.nrftoolbox.uart.EXTRA_DATA";
 
-	/** A broadcast message with this action and the message in {@link Intent#EXTRA_TEXT} will be sent t the UART device. */
+    /**
+     * A broadcast message with this action and the message in {@link Intent#EXTRA_TEXT} will be sent t the UART device.
+     */
     public final static String ACTION_SEND = "no.nordicsemi.android.nrftoolbox.uart.ACTION_SEND";
-	/** A broadcast message with this action is triggered when a message is received from the UART device. */
+    /**
+     * A broadcast message with this action is triggered when a message is received from the UART device.
+     */
     private final static String ACTION_RECEIVE = "no.nordicsemi.android.nrftoolbox.uart.ACTION_RECEIVE";
-	/** Action send when user press the DISCONNECT button on the notification. */
+    /**
+     * Action send when user press the DISCONNECT button on the notification.
+     */
     public final static String ACTION_DISCONNECT = "no.nordicsemi.android.nrftoolbox.uart.ACTION_DISCONNECT";
-	/** A source of an action. */
+    /**
+     * A source of an action.
+     */
     public final static String EXTRA_SOURCE = "no.nordicsemi.android.nrftoolbox.uart.EXTRA_SOURCE";
     public final static int SOURCE_NOTIFICATION = 0;
     public final static int SOURCE_WEARABLE = 1;
@@ -116,7 +125,7 @@ public class UARTService extends BleProfileService implements UARTManagerCallbac
     @Override
     public void onDestroy() {
         // when user has disconnected from the sensor, we have to cancel the notification that we've created some milliseconds before using unbindService
-		cancelNotification();
+        stopForegroundService();
         unregisterReceiver(mDisconnectActionBroadcastReceiver);
         unregisterReceiver(mIntentBroadcastReceiver);
 
@@ -127,14 +136,12 @@ public class UARTService extends BleProfileService implements UARTManagerCallbac
 
     @Override
     protected void onRebind() {
-		// when the activity rebinds to the service, remove the notification
+        stopForegroundService();
-		cancelNotification();
     }
 
     @Override
     protected void onUnbind() {
-		// when the activity closes we need to show the notification that user is connected to the sensor
+        startForegroundService();
-		createNotification(R.string.uart_notification_connected_message, 0);
     }
 
     @Override
@@ -190,6 +197,7 @@ public class UARTService extends BleProfileService implements UARTManagerCallbac
 
     /**
      * Sends the given message to all connected wearables. If the path is equal to {@link Constants.UART#DEVICE_DISCONNECTED} the service will be stopped afterwards.
+     *
      * @param path    message path
      * @param message the message
      */
@@ -216,16 +224,42 @@ public class UARTService extends BleProfileService implements UARTManagerCallbac
         }
     }
 
+    /**
+     * Sets the service as a foreground service
+     */
+    private void startForegroundService() {
+        // when the activity closes we need to show the notification that user is connected to the peripheral sensor
+        // We start the service as a foreground service as Android 8.0 (Oreo) onwards kills any running background services
+        final Notification notification = createNotification(R.string.uart_notification_connected_message, 0);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            startForeground(NOTIFICATION_ID, notification);
+        } else {
+            final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
+            nm.notify(NOTIFICATION_ID, notification);
+        }
+    }
+
+    /**
+     * Stops the service as a foreground service
+     */
+    private void stopForegroundService() {
+        // when the activity rebinds to the service, remove the notification and stop the foreground service
+        // on devices running Android 8.0 (Oreo) or above
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            stopForeground(true);
+        } else {
+            cancelNotification();
+        }
+    }
+
     /**
      * Creates the notification
      *
-	 * @param messageResId
+     * @param messageResId message resource id. The message must have one String parameter,<br />
-	 *            message resource id. The message must have one String parameter,<br />
      *                     f.e. <code>&lt;string name="name"&gt;%s is connected&lt;/string&gt;</code>
-	 * @param defaults
+     * @param defaults     signals that will be used to notify the user
-	 *            signals that will be used to notify the user
      */
-	private void createNotification(final int messageResId, final int defaults) {
+    protected Notification createNotification(final int messageResId, final int defaults) {
         final Intent parentIntent = new Intent(this, FeaturesActivity.class);
         parentIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
         final Intent targetIntent = new Intent(this, UARTActivity.class);
@@ -243,9 +277,7 @@ public class UARTService extends BleProfileService implements UARTManagerCallbac
         builder.setShowWhen(defaults != 0).setDefaults(defaults).setAutoCancel(true).setOngoing(true);
         builder.addAction(new NotificationCompat.Action(R.drawable.ic_action_bluetooth, getString(R.string.uart_notification_action_disconnect), disconnectAction));
 
-		final Notification notification = builder.build();
+        return builder.build();
-		final NotificationManager nm = (NotificationManager) getSystemService(Context.NOTIFICATION_SERVICE);
-		nm.notify(NOTIFICATION_ID, notification);
     }
 
     /**