-
Notifications
You must be signed in to change notification settings - Fork 747
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
Crash on jailbroken iPod Touch 6G #9
Comments
Guessing it can't find the framework's resource bundle. Are you on the latest version? There was a recent fix that might be related #2. Otherwise, could you provide some context around how you're building (e.g. Cocoa pods, Carthage, etc.)? I'd like to be able to reproduce the problem on my end. |
Hi @wtmoose! Thanks for the quick reply! I'm using So far this crash happened once with a user, can't reproduce using an iPhone or iPad on my end. |
Gotcha. Hmm. Thanks for reporting. I'll look into it, though I'm at the try! Swift NYC conference for a couple of days. How are you building the library (e.g. Cocoa pods, Carthage, etc.)? |
As a workaround, you could copy the image into your project and write your own config method that sets the image instead of having SwiftMessages set it. |
I built the library with CocoaPods, using Fastlane. The library works great, and the alerts show on the App Store version on my iPhone 6. Maybe it is isolated to iPod Touch? |
Are you on CocoaPods 1.0.1? |
Yep!
|
Any idea if the device in question was jailbroken? Just asking because #12 is reporting a similar-sounding issue and the device in question is jailbroken. |
Hi @wtmoose, Thanks for the follow up. Yes, 100% jailbreak according to Fabric 👍 |
@wtmoose Any hint on why it crashes on Jailbroken device? |
No. I don't have a jailbroken device. I've been waiting for a kind jailbreaker to drop in and help out. Got close with #12, but I never heard back. Do you have any data on percentage of users affected? |
@wtmoose Yes, 100% of my JB users are experiencing this crash. I don't have a JB device as well :( |
Do you have a percentage of all users affected? |
@wtmoose just JB users |
Minor update: we tried to pin down a jailbroken device last week, with no luck. None of our devices are on a version of 9.x or 10.x that can be jailbroken. We did try a "partially" jailbroken 9.4 device, but couldn't reproduce the crash. If and when the latest version of 10.x is jailbroken, we'll give that a try. Beyond that, I'm not sure what else can be done until someone with a jailbroken device lends a hand. You can probably work around the issue by copying the .nib file into your project and removing or changing the image referenced by the image view. |
Still jailbroken device iPhone 6s on 9.3.3 crash on your library.
any workaround yet? |
@kwstasna No. I can't get my hands on a jailbroken device. Can you help? |
@kwstasna Let me clarify that "no". I don't have a workaround I can implement in my library. There is a likely workaround you can implement in your project, as I mentioned in the comment directly above your's. |
this was from Fabric Crashlytics. Although i will try to find a jailbroken device to try your possible work around soon. |
One of my users told me that he made my app working by installing Don't know if that helps you! |
@kwstasna Good to know. Thanks. |
I copied the CardView.xib in my project, but I'm not sure I get the "change the image referenced by the image view" part. Do I have to copy all the xcassets in my project as well? EDIT: Do you mean to only clear the errorIcon image input field? |
My suggestion is don't use any resources in the framework bundle if you want to avoid crashes on jailbroken devices. The |
@wtmoose I see! Well, I made a release today with the CardView.xib in my project, but didn't copy any images. we'll see how it goes! Icons were showing up correctly, but I can't test it on a JB device.. |
Aw. Sorry it took me so long to reply |
@wtmoose No worries, open source does not pay the house :) You confirm I should be safe with the nib in my project (i cleared the UIImageView image) without the images? From what I see, icons are showing correctly and no crash. |
I think it will probably crash if you're still calling |
@wtmoose Just to confirm, you mean crash on a JB device, correct? Because it doesn't crash on a non JB device, with the xib in my projects and the image loaded from the framework bundle. Thanks! |
@allaire I haven't personally witnessed a crash on a non-jailbroken device, so yes. I'm interested in trying a potential workaround: move images out of asset catalog and switching from pdf to png. If I made that change on a branch, would you be willing to ship that version at some point? |
@wtmoose I only use pdfs as well. I'll report back if I see any crash with the nib in the project (but without the icons/pdfs). I only have 2-3 crashing users, all JB. |
same problem on Jailbreak device,how to resolve ? |
|
@crossle workarounds are discussed in this ticket. To summarize, if you're concerned about jailbroken phones, don't use any resources in the SwiftMessages framework. Copy anything you want to use into your project. |
I came across this tickets on the CocoaPods library, which seems potentially relevant: |
Hey I got the same issue after updating Google Maps Pod from version 2.2.0 to version 2.5.0. |
Here's the crash report: http://crashes.to/s/b56939d1ae3. |
@victor-sarda I can't see how a See the active discussion in #128 as we've identified a fix. |
Oh, cool I didn't see the last comments, good news so! |
Just got the same crash recently for a user on my app. But there is a quickfix to avoid the crash. Just remove the force unwrap
It's better to have no icon than a crash for jailbroken users |
@Mycose it is fixed on the head of the develop branch. Always get the icon. No crash. |
@wtmoose oh okay cool ! |
@wtmoose Should I close this then? I'll try to cut a release with the master branch and see how it goes 😄 |
@allaire No. I'll close it when I release 4.0.1, which should be in the next few days. |
Should be fixed in 4.1.0 (though I can't verify). Reopen if you continue to see an issue. |
I am still designing all my alerts using SwiftMessages, and almost all my alerts are custom xib files (made from CardView.xib), is this issue isolated to the default theme based alerts only? Just want to confirm? Or has this been resolved? |
@annjawn It's fixed. |
Got the following crash:
iPod Touch 6G on 9.3.3
The text was updated successfully, but these errors were encountered: