Which package(s)
@nativescript/background-http@6.0.2 — Android only.
Environment
- @nativescript/core 9.0.20, CLI 9.0.6, @nativescript/android 9.0.4
- net.gotev:uploadservice 4.9.2 (the plugin's default)
- Reported by production users on Android; not reproducible on iOS (see below)
Issue
Task.cache lives only in the JS runtime, but the gotev upload service is a
foreground service that outlives it. If the OS kills the app process while an
upload is in flight, the service keeps going (or is restarted) and eventually
broadcasts progress/completion. The GlobalRequestObserver registered by
init() then dispatches into a fresh runtime whose Task.cache is empty,
Task.fromId() returns undefined, and the handlers dereference it right away.
Two reports from the same upload, in the order they arrived:
Calling js method onProgress failed
TypeError: Cannot read properties of undefined (reading 'setTotalUpload')
at onProgressReceiverProgress(vendor.mjs)
at onProgress(vendor.mjs)
at com.tns.gen.net.gotev.uploadservice.observer.request.RequestObserverDelegate.onProgress(RequestObserverDelegate.java:21)
at net.gotev.uploadservice.observer.request.BaseRequestObserver.onReceive(BaseRequestObserver.kt:29)
Calling js method onSuccess failed
TypeError: Cannot read properties of undefined (reading 'setUpload')
at onProgressReceiverCompleted(vendor.mjs)
at onSuccess(vendor.mjs)
at com.tns.gen.net.gotev.uploadservice.observer.request.RequestObserverDelegate.onSuccess(RequestObserverDelegate.java:32)
at net.gotev.uploadservice.observer.request.BaseRequestObserver.onReceive(BaseRequestObserver.kt:31)
Note the upload itself succeeded — only the JS callback blew up.
Root cause
packages/background-http/index.android.ts on main:
function onProgressReceiverProgress(context: Context, uploadInfo: UploadInfo) {
const uploadId = uploadInfo.getUploadId();
const task = Task.fromId(uploadId); // may be undefined
const totalBytes = uploadInfo.getTotalBytes();
const currentBytes = uploadInfo.getUploadedBytes();
task.setTotalUpload(totalBytes);
Task.fromId (L228) is a plain Task.cache[id] lookup, and all four handlers
use the result unguarded: onProgressReceiverProgress (L42),
onProgressReceiverCompleted (L58), onProgressReceiverCancelled (L90) and
onProgressReceiverError (L97).
iOS is not affected: Task.getTask() (index.ios.ts L277-287) creates a Task
when the native task isn't in the map, instead of returning undefined.
Impact
Without discardUncaughtJsExceptions this is a hard crash — the exception
escapes a native callback. With the flag it is "only" a discarded exception, but
since progress fires repeatedly, a single orphaned upload produces a burst of
them (in our case, one error report per broadcast).
Steps to reproduce
- Start a multipartUpload large enough to take several seconds.
- Kill the app process while it runs (adb shell am force-stop <pkg>, or let
the OS reclaim it) — the upload service survives.
- Reopen the app so a fresh runtime calls init() and registers the observer.
- The service reports the resumed/finished upload and the TypeError is thrown.
Suggested fix
Guard the lookup in the four handlers, e.g.:
function onProgressReceiverProgress(context: Context, uploadInfo: UploadInfo) {
const uploadId = uploadInfo.getUploadId();
const task = Task.fromId(uploadId);
+ if (!task) {
+ // Upload started by a previous process: there is no JS task to notify.
+ return;
+ }
Related (bigger, happy to file separately)
Even with the guard, the outcome of an upload started before the restart is
simply lost — the app can never learn whether it succeeded. Two API additions
would make it recoverable: letting the caller supply the upload id (it is
currently generated internally as session.id + '{n}'), and a way to re-attach
listeners to an in-flight or finished upload by id. Today the only workaround is
reconciling against the server after the fact.
Which package(s)
@nativescript/background-http@6.0.2 — Android only.
Environment
Issue
Task.cache lives only in the JS runtime, but the gotev upload service is a
foreground service that outlives it. If the OS kills the app process while an
upload is in flight, the service keeps going (or is restarted) and eventually
broadcasts progress/completion. The GlobalRequestObserver registered by
init() then dispatches into a fresh runtime whose Task.cache is empty,
Task.fromId() returns undefined, and the handlers dereference it right away.
Two reports from the same upload, in the order they arrived:
Note the upload itself succeeded — only the JS callback blew up.
Root cause
packages/background-http/index.android.ts on main:
Task.fromId (L228) is a plain Task.cache[id] lookup, and all four handlers
use the result unguarded: onProgressReceiverProgress (L42),
onProgressReceiverCompleted (L58), onProgressReceiverCancelled (L90) and
onProgressReceiverError (L97).
iOS is not affected: Task.getTask() (index.ios.ts L277-287) creates a Task
when the native task isn't in the map, instead of returning undefined.
Impact
Without discardUncaughtJsExceptions this is a hard crash — the exception
escapes a native callback. With the flag it is "only" a discarded exception, but
since progress fires repeatedly, a single orphaned upload produces a burst of
them (in our case, one error report per broadcast).
Steps to reproduce
the OS reclaim it) — the upload service survives.
Suggested fix
Guard the lookup in the four handlers, e.g.:
function onProgressReceiverProgress(context: Context, uploadInfo: UploadInfo) { const uploadId = uploadInfo.getUploadId(); const task = Task.fromId(uploadId); + if (!task) { + // Upload started by a previous process: there is no JS task to notify. + return; + }Related (bigger, happy to file separately)
Even with the guard, the outcome of an upload started before the restart is
simply lost — the app can never learn whether it succeeded. Two API additions
would make it recoverable: letting the caller supply the upload id (it is
currently generated internally as session.id + '{n}'), and a way to re-attach
listeners to an in-flight or finished upload by id. Today the only workaround is
reconciling against the server after the fact.