Cordova-plugin-background-geolocation: ios: debug mode produces endless loop in notification handling

[Ritzlgrmft]

Your Environment

• Plugin version: 3.1.0
• Platform: iOS
• OS version: 13.1
• Device manufacturer and model:
  
• Running in Simulator: Yes
• Cordova version (cordova -v): Capacitor 1.3.0
• Cordova platform version (cordova platform ls):
• Plugin configuration options: debug: true, desiredAccuracy: BackgroundGeolocation.HIGH_ACCURACY, saveBatteryOnBackground: true
• Link to your project:

Context

In CDVBackgroundGeolocation, the notification delegate gets set:

center.delegate = self;

Immediately afterwards, in debug mode, this value gets stored in prevNotificationDelegate:

prevNotificationDelegate = center.delegate;
center.delegate = self;

When handling the notification, it calls always the handler of prevNotificationDelegate (which is self). This leads to and endless loop.

Expected Behavior

Notification should be handled in debug mode as well.

Actual Behavior

App crashes.

Possible Fix

IMHO prevNotificationDelegate is completely useless. Probably you can just remove it. Or at least call it only, if it != self.

Steps to Reproduce

1.
2.
3.
4.

Context

Debug logs

[StevieWag]

any news on that?!

[jackie-d]

This still happens with @mauron85/cordova-plugin-background-geolocation@~3.1.0 + "@ionic-native/background-geolocation": "^5.22.0".

Also system log reports: Service exited due to SIGSEGV | sent by exc handler[27280](segmentation fault due to too much recursion)

The fix proposed by @Ritzlgrmft it's mandatory to keep the iOs version working, otherwise it will just crash at start.

Is there any way to disable the notification?

[jackie-d]

Related to:
https://github.com/mauron85/cordova-plugin-background-geolocation/issues/645
https://github.com/mauron85/cordova-plugin-background-geolocation/pull/633

The latest and this one explain the issue details really well and address the fix.
Any maintaner or iOs dev to open a PR?

Temporary solution: disable debug mode in config

this.backgroundGeolocation.configure({
  // ...
  debug: false
  // ...
});

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale[bot]]

This issue has been automatically closed, because it has not had recent activity. If you believe this issue shouldn't be closed, please reopen or write down a comment requesting issue reopening with explanation why you think it's important. Thank you for your contributions.

[jdnichollsc]

Oh why?