-
Notifications
You must be signed in to change notification settings - Fork 143
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
fix: iOS Crash at 0xfffffffffffffffe #1700
Comments
Likely the same as #1698. |
If anyone is experiencing this and is using a crash reporting service, we've added a new guide on how to get the corresponding .dsym files to upload them to your service for more detailed reports: |
|
Thank you. This looks like the same crash point as expected. We still don't know how to reproduce, but we're about to meet with another customer in hopes of reproducing now.
|
OK, we have a repro! https://github.com/felangel/fresh/tree/master/packages/fresh_dio crashes repeatably. We're investigating now. |
Not sure if it helps: We're not using fresh_dio but a similar solution for token refresh: ..interceptors.add(
InterceptorsWrapper(
onRequest: (RequestOptions options, handler) async {
try {
final token = await passLoginHandler.getToken();
options.headers.addAll(<String, String>{
'Authorization': 'Bearer $token',
'Accept': 'application/json',
});
return handler.next(options);
} catch (e) {
handler.reject(DioError(requestOptions: options, error: e.toString()));
}
},
onError: (DioError error, handler) {
//...
handler.reject(error);
},
), where the getToken function is calling the refreshCredentials() from the EDIT: when our app crashes I'm indeed sending a request |
Thank you! We were able to get a reproducible crash with package:dio and Flutter and are working on reducing it now. The crash ends up in Dio's async json parsing which uses a Dart isolate. So far it seems to only crash inside Flutter (not pure Dart). I'm sure it's a bug in our forked Dart, and hopefully we'll know more soon. 🤞 |
Thank you again! We found the bug and have a fix we will be releasing in the morning. See #1698 (comment) for more info. |
https://discord.com/channels/1030243211995791380/1125516353370075306/1202928574097657906
Manually symbolicated:
The text was updated successfully, but these errors were encountered: