HeadlessJsTaskService is expected to run on UI thread

Is this a bug report?

Yes

Have you read the Contributing Guidelines?

Yes

Environment

react-native -v: react-native-cli: 2.0.1 react-native: 0.48.1
node -v: v7.5.0
npm -v: 5.3.0

Target Platform: Android

Development Operating System: macOS

Build tools:

Steps to Reproduce

Received a push notification
Starting a Service that extends HeadlessJsTaskService.java
App throws exception com.facebook.react.bridge.AssertionException: Expected to run on UI thread!

Expected Behavior

I expect no UI thread requirement for a background service (same as Android native SDK)

Actual Behavior

App throws an exception

starting a headless task is expected to run on UI thread: https://github.com/facebook/react-native/blob/1e8f3b11027fe0a7514b4fc97d0798d3c64bc895/ReactAndroid/src/main/java/com/facebook/react/HeadlessJsTaskService.java#L89-L96

@chirag04

@chirag04 thanks for the note. However, as I understand, one of the common uses of this is performing tasks triggered by a push notification (or a job schedule). These are clearly not required to run in a UI thread as these are background activities.

@oriharel

@oriharel according to the documentation, HeadlessJS is meant for tasks that are needed to be ran when your app in the background. In which case there are no operations on the main thread.

By default, your app will crash if you try to run a task while the app is in the foreground. This is to prevent developers from shooting themselves in the foot by doing a lot of work in a task and slowing the UI. You can pass a fourth boolean argument to control this behaviour.

Exactly. This is why I don't understand the "startTask must be triggered from the UI thread" requirement. Unless there is some limitation forcing the React context to be spawned from the UI thread.

That means you can not trigger a headless task from another headless task (threads of threads).

In the documentation, it is clearly stated -

Now, whenever you start your service, e.g. as a periodic task or in response to some system event / broadcast, JS will spin up, run your task, then spin down.

Which means, even if a BroadcastReceiver is triggered, say from a push notification, the Headless task should be able to perform.

But this is not the case, because there is an underline requirement that the onReactContextInitialized event handler will be called from the main thread (AKA UI thread). A requirement that most of the time not fulfilled (for example, when a user swiped out the app from the recent apps menu).

So in fact - I DO consider this a bug and updated the form.

+1

I think there's a bit of confusion here. The "UI" thread is simply the "main" thread of an Android application. It's the thread that Services naturally run on. For various reasons, the React context is always initialized on this thread, from where it spins off two others: JS and modules (IIRC). This is why startTask needs to be run on the UI/main thread, because it might need to initialize the context.

If you're on a background thread it should be fairly easy to post something to the main one and call startTask there. Is there a reason why that wouldn't work?

You're right, everything I said about the UI thread actually reffers to the main thread. The Service "onStartCommand" does indeed happen on that thread. However, inside startTask there is an asynchronic call to create the React context. The callback that is triggered asynchronically is no longer on that thread. And that is already a private area so we can't override this in our inheriting class.

An updated pull request here.

re #15915 HeadlessJsTaskService is expected to run on UI thread

+1

Issue is now fixed and merged to React Native.

I have used latest version i.e 0.48.4 of react-native, still, I don't see your changes in File HeadlessJsTaskService.java.

I don't think it's released yet

Hi. I tried to fix this file manually, but i got another error.

My code

BackgroundJob.register({
        jobKey: exactJobKey,
        job: () => process()
    });

    BackgroundJob.schedule({
        jobKey: exactJobKey,
        period: 900000,
        override: true,
        allowExecutionInForeground: true,
        persist: true
    });

Check your AppRegistry in js code. You need to register a js file before you call it from Java

Also check your HeadlessJsTaskService implementation in Java. Make sure you return new HeadlessJsTaskConfig with a name "exactJobKey".

Thanks. Looks like this issue is fixed in 0.49.5.

Navigation Menu

HeadlessJsTaskService is expected to run on UI thread #15915

Is this a bug report?

Have you read the Contributing Guidelines?

Environment

Steps to Reproduce

Expected Behavior

Actual Behavior

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

HeadlessJsTaskService is expected to run on UI thread #15915

Description

Is this a bug report?

Have you read the Contributing Guidelines?

Environment

Steps to Reproduce

Expected Behavior

Actual Behavior

Activity

chirag04 commented on Sep 12, 2017

oriharel commented on Sep 12, 2017

cmdshepard commented on Sep 12, 2017

oriharel commented on Sep 13, 2017

D1no commented on Sep 13, 2017

oriharel commented on Sep 13, 2017

haimyyy commented on Sep 13, 2017

ycarmel commented on Sep 13, 2017

blankg commented on Sep 13, 2017

foghina commented on Sep 13, 2017

foghina commented on Sep 13, 2017

oriharel commented on Sep 14, 2017

oriharel commented on Sep 14, 2017

narodejesus commented on Sep 20, 2017

oriharel commented on Sep 20, 2017

sparsh commented on Sep 28, 2017

oriharel commented on Sep 28, 2017

artemukolov commented on Oct 26, 2017

oriharel commented on Oct 26, 2017

oriharel commented on Oct 26, 2017

artemukolov commented on Oct 30, 2017

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions