Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crashes lately #131

Closed
crspybits opened this issue Oct 2, 2018 · 8 comments
Closed

Crashes lately #131

crspybits opened this issue Oct 2, 2018 · 8 comments

Comments

@crspybits
Copy link
Owner

  1. Rod-- when he was zooming in on an image
  2. Dany-- when he was downloading images when the app was installed on a new device
  3. Natasha-- IIRC, when she was looking at the discussion for an image.

Need to upload the dSyms's for the app to Crashlytics and get the crash reports.

@danligas
Copy link

danligas commented Oct 2, 2018 via email

@crspybits
Copy link
Owner Author

On Dany's issue-- looking at the logs, I found crspybits/SyncServerII#89. Dany suggests "Seems to me that the si crash was some aggregated by the slow download. Possibly some sort of unhandled timeout?".

@crspybits
Copy link
Owner Author

Possibly, the app went into the background on Dany's downloads because the screen locked? I have to check, but I think apps do go into the background when the screen locks. SharedImages really needs to be handling background downloads better. See https://stackoverflow.com/questions/29029632/is-an-ios-app-active-when-the-device-is-locked and https://developer.apple.com/library/archive/documentation/iPhone/Conceptual/iPhoneOSProgrammingGuide/StrategiesforHandlingAppStateTransitions/StrategiesforHandlingAppStateTransitions.html

@crspybits
Copy link
Owner Author

On Dany's issue -- first time download:

There are a few issues with first time download. I think the main one is that the app shouldn't crash. I've got a ticket for that.
And it would be for the best if the download restarts nicely when the app returns to the foreground.

@crspybits
Copy link
Owner Author

Dany: Here are my concerns over that issue : your server should not be absorbing all that bandwidth. You will incur heavy usage carges everytime a customer initializes a new device.

@danligas
Copy link

danligas commented Oct 17, 2018 via email

@crspybits
Copy link
Owner Author

The crashes look like they should be from v0.19.0 of the app. However, when I upload the dSyms for this, I get no crashes reported with Crashlytics. Yarg.

@crspybits
Copy link
Owner Author

Don't have enough info to take this further.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants