fix: Dio error holds a reference to null values (#825)

Author: Nidal-Bakir

packages/dart/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [3.1.14](https://github.com/parse-community/Parse-SDK-Flutter/compare/dart-3.1.13...dart-3.1.14) (2023-02-26)
+
+### Bug Fixes
+
+* Dio error object holds a reference to null values ([#774](https://github.com/parse-community/Parse-SDK-Flutter/issues/774))
+
 ## [3.1.13](https://github.com/parse-community/Parse-SDK-Flutter/compare/dart-3.1.12...dart-3.1.13) (2023-02-15)
 
 ### Bug Fixes
@@ -161,7 +167,7 @@ Bug fixes
 ## 1.0.22
 
 Added dirty children
-Added option of sembast or share_preferences 
+Added option of sembast or share_preferences
 
 ## 1.0.21
 
@@ -183,7 +189,7 @@ Bug fix
 
 ## 1.0.17
 
-LiveQuery fix 
+LiveQuery fix
 Bug fixes
 
 ## 1.0.16

packages/dart/lib/src/base/parse_constants.dart

@@ -1,7 +1,7 @@
 part of flutter_parse_sdk;
 
 // Library
-const String keySdkVersion = '3.1.13';
+const String keySdkVersion = '3.1.14';
 const String keyLibraryName = 'Flutter Parse SDK';
 
 // End Points

packages/dart/lib/src/network/parse_dio_client.dart

@@ -27,11 +27,16 @@ class ParseDioClient extends ParseClient {
         path,
         options: _Options(headers: options?.headers),
       );
+
       return ParseNetworkResponse(
-          data: dioResponse.data!, statusCode: dioResponse.statusCode!);
+        data: dioResponse.data!,
+        statusCode: dioResponse.statusCode!,
+      );
     } on dio.DioError catch (error) {
       return ParseNetworkResponse(
-          data: error.response?.data, statusCode: error.response!.statusCode!);
+        data: error.response?.data ?? _fallbackErrorData,
+        statusCode: error.response?.statusCode ?? ParseError.otherCause,
+      );
     }
   }
 
@@ -51,12 +56,15 @@ class ParseDioClient extends ParseClient {
             headers: options?.headers, responseType: dio.ResponseType.bytes),
       );
       return ParseNetworkByteResponse(
-          bytes: dioResponse.data, statusCode: dioResponse.statusCode!);
+        bytes: dioResponse.data,
+        statusCode: dioResponse.statusCode!,
+      );
     } on dio.DioError catch (error) {
       if (error.response != null) {
         return ParseNetworkByteResponse(
-            data: error.response?.data,
-            statusCode: error.response!.statusCode!);
+          data: error.response?.data ?? _fallbackErrorData,
+          statusCode: error.response?.statusCode ?? ParseError.otherCause,
+        );
       } else {
         return _getOtherCaseErrorForParseNetworkResponse(error.error);
       }
@@ -72,11 +80,16 @@ class ParseDioClient extends ParseClient {
         data: data,
         options: _Options(headers: options?.headers),
       );
+
       return ParseNetworkResponse(
-          data: dioResponse.data!, statusCode: dioResponse.statusCode!);
+        data: dioResponse.data!,
+        statusCode: dioResponse.statusCode!,
+      );
     } on dio.DioError catch (error) {
       return ParseNetworkResponse(
-          data: error.response?.data, statusCode: error.response!.statusCode!);
+        data: error.response?.data ?? _fallbackErrorData,
+        statusCode: error.response?.statusCode ?? ParseError.otherCause,
+      );
     }
   }
 
@@ -89,11 +102,16 @@ class ParseDioClient extends ParseClient {
         data: data,
         options: _Options(headers: options?.headers),
       );
+
       return ParseNetworkResponse(
-          data: dioResponse.data!, statusCode: dioResponse.statusCode!);
+        data: dioResponse.data!,
+        statusCode: dioResponse.statusCode!,
+      );
     } on dio.DioError catch (error) {
       return ParseNetworkResponse(
-          data: error.response?.data, statusCode: error.response!.statusCode!);
+        data: error.response?.data ?? _fallbackErrorData,
+        statusCode: error.response?.statusCode ?? ParseError.otherCause,
+      );
     }
   }
 
@@ -111,13 +129,17 @@ class ParseDioClient extends ParseClient {
         options: _Options(headers: options?.headers),
         onSendProgress: onSendProgress,
       );
+
       return ParseNetworkResponse(
-          data: dioResponse.data!, statusCode: dioResponse.statusCode!);
+        data: dioResponse.data!,
+        statusCode: dioResponse.statusCode!,
+      );
     } on dio.DioError catch (error) {
       if (error.response != null) {
         return ParseNetworkResponse(
-            data: error.response?.data,
-            statusCode: error.response!.statusCode!);
+          data: error.response?.data ?? _fallbackErrorData,
+          statusCode: error.response?.statusCode ?? ParseError.otherCause,
+        );
       } else {
         return _getOtherCaseErrorForParseNetworkResponse(error.error);
       }
@@ -138,13 +160,20 @@ class ParseDioClient extends ParseClient {
         path,
         options: _Options(headers: options?.headers),
       );
+
       return ParseNetworkResponse(
-          data: dioResponse.data!, statusCode: dioResponse.statusCode!);
+        data: dioResponse.data!,
+        statusCode: dioResponse.statusCode!,
+      );
     } on dio.DioError catch (error) {
       return ParseNetworkResponse(
-          data: error.response?.data, statusCode: error.response!.statusCode!);
+        data: error.response?.data ?? _fallbackErrorData,
+        statusCode: error.response?.statusCode ?? ParseError.otherCause,
+      );
     }
   }
+
+  String get _fallbackErrorData => '{"$keyError":"NetworkError"}';
 }
 
 /// Creates a custom version of HTTP Client that has Parse Data Preset

packages/dart/pubspec.yaml

@@ -1,6 +1,6 @@
 name: parse_server_sdk
 description: Dart plugin for Parse Server, (https://parseplatform.org), (https://back4app.com)
-version: 3.1.13
+version: 3.1.14
 homepage: https://github.com/parse-community/Parse-SDK-Flutter
 
 environment: