[Mixpanel sharedAPIWithToken:] raises exception. Causes app to crash on launch every time. about mixpanel-iphone

[alexkarpenko]

My current workaround is to add a try-catch to unarchiveEvents, and remove the event file if the unarchiver fails:

- (void)unarchiveEvents {
    @try {
        self.eventQueue = [NSKeyedUnarchiver unarchiveObjectWithFile:[self eventFilePath]];
    }
    @catch (NSException *exception) {
        [[NSFileManager defaultManager] removeItemAtPath:[self eventFilePath] error:nil];
        self.eventQueue = nil;
    }

    if (!self.eventQueue) {
        self.eventQueue = [NSMutableArray array];
    }
}

[kasrak]

Hey,

Thanks for reporting this. I've put the try-catches in the unarchive methods. That should prevent any more app crashes.

I'm wondering how this could've happened. Were you updating from a previous version of the library or was this a brand new implementation?

[alexkarpenko]

Hay kasrak,

A few of our users ran into this while using our live app store version. Our app store version has an older mixpanel SDK (preceding the people additions). I got a tester who experienced this issue to run a fresh build with the above SHA version, and got that crash report back.

Unfortunately all I have are guesses as to the cause, since I can't reproduce this myself. The tester reported running out of disk space while using our app, after which point the app would crash on launch. Perhaps the archive operation leaves the file in a corrupt state when there's not enough disk space. I also store some error data (error domain, code, description) in properties which might might have been malformed somehow. How does NSKeyedArchiver deal with objects that don't implement NSCoding?

[malectro]

Sounds like we're a go with the try/catches, and we haven't had any reported issues with the KeyedArchiver in the meantime. Please feel free to reopen if you're experiencing more issues with it.