fix: use http package to handle zip file upload

- Add `http` package dependency, and use it instead of manually making the upload post request.

Author: MoKamall

bin/commands/upload_so_files.dart

@@ -96,7 +96,8 @@ class UploadSoFilesCommand {
 
       // Validate architecture
       if (!validArchs.contains(options.arch)) {
-        print('[Instabug-CLI] Error: Invalid architecture: ${options.arch}. Valid options: ${validArchs.join(', ')}');
+        print(
+            '[Instabug-CLI] Error: Invalid architecture: ${options.arch}. Valid options: ${validArchs.join(', ')}');
         throw Exception(
             'Invalid architecture: ${options.arch}. Valid options: ${validArchs.join(', ')}');
       }
@@ -106,31 +107,44 @@ class UploadSoFilesCommand {
       print('File: ${options.file}');
       print('App Version: ${options.name}');
 
-      // TODO: Implement the actual upload logic here
-      // This would typically involve:
-      // 1. Reading the zip file
-      // 2. Making an HTTP request to the upload endpoint
-      // 3. Handling the response
-
-      // Make an HTTP request to the upload endpoint
-      final body = {
-        'arch': options.arch,
-        'api_key': options.apiKey,
-        'application_token': options.token,
-        'so_file': options.file,
-        'app_version': options.name,
-      };
-
       const endPoint = 'https://api.instabug.com/api/web/public/so_files';
 
-      final response = await makeHttpPostRequest(
-        url: endPoint,
-        body: body,
+      // Create multipart request
+      final request = http.MultipartRequest('POST', Uri.parse(endPoint));
+
+      // Add form fields
+      request.fields['arch'] = options.arch;
+      request.fields['api_key'] = options.apiKey;
+      request.fields['application_token'] = options.token;
+      request.fields['app_version'] = options.name;
+
+      // Add the zip file
+      final fileStream = http.ByteStream(file.openRead());
+      final fileLength = await file.length();
+      final multipartFile = http.MultipartFile(
+        'so_file',
+        fileStream,
+        fileLength,
+        filename: file.path.split('/').last,
       );
+      request.files.add(multipartFile);
+
+      final response = await request.send();
+
+      if (response.statusCode < 200 || response.statusCode >= 300) {
+        final responseBody = await response.stream.bytesToString();
+        print('[Instabug-CLI] Error: Failed to upload .so files');
+        print('Status Code: ${response.statusCode}');
+        print('Response: $responseBody');
+        exit(1);
+      }
 
-      print('Successfully uploaded .so files for version: ${options.name} with arch ${options.arch}');
+      print(
+          'Successfully uploaded .so files for version: ${options.name} with arch ${options.arch}');
+      exit(0);
     } catch (e) {
-      print('[Instabug-CLI] Error: Error uploading .so files: $e');
+      print('[Instabug-CLI] Error uploading .so files, $e');
+      print('[Instabug-CLI] Error Stack Trace: ${StackTrace.current}');
       exit(1);
     }
   }

bin/instabug.dart

@@ -1,13 +1,14 @@
 #!/usr/bin/env dart
 
-import 'dart:convert';
 import 'dart:io';
 
 import 'package:args/args.dart';
+import 'package:http/http.dart' as http;
 
 part 'commands/upload_so_files.dart';
 
 // Command registry for easy management
+// ignore: avoid_classes_with_only_static_members
 class CommandRegistry {
   static final Map<String, CommandHandler> _commands = {
     'upload-so-files': CommandHandler(
@@ -27,35 +28,30 @@ class CommandHandler {
   CommandHandler({required this.parser, required this.execute});
 }
 
-Future<bool> makeHttpPostRequest({
-  required String url,
-  required Map<String, String> body,
-  Map<String, String>? headers,
-}) async {
-  try {
-  final client = HttpClient();
-
-  final request = await client.postUrl(Uri.parse(url));
-
-  request.headers.contentType = ContentType.json;
-
-  request.write(jsonEncode(body));
-
-    final response = await request.close();
-
-    if (response.statusCode >= 200 && response.statusCode < 300) {
-      final responseBody = await response.transform(utf8.decoder).join();
-      return true;
-    } else {
-      print('Error: ${response.statusCode}');
-      return false;
-    }
-
-  } catch (e) {
-    print('[Instabug-CLI] Error while making HTTP POST request: $e');
-    exit(1);
-  }
-}
+// Future<bool> makeHttpPostRequest({
+//   required String url,
+//   required Map<String, String> body,
+//   Map<String, String>? headers,
+// }) async {
+//   try {
+//     final client = HttpClient();
+//     final request = await client.postUrl(Uri.parse(url));
+//     request.headers.contentType = ContentType.json;
+//     request.write(jsonEncode(body));
+//     final response = await request.close();
+//     if (response.statusCode >= 200 && response.statusCode < 300) {
+//       final responseBody = await response.transform(utf8.decoder).join();
+//       return true;
+//     } else {
+//       print('Error Code: ${response.statusCode}');
+//       print('Error Body: ${await response.transform(utf8.decoder).join()}');
+//       return false;
+//     }
+//   } catch (e) {
+//     print('[Instabug-CLI] Error while making HTTP POST request: $e');
+//     exit(1);
+//   }
+// }
 
 void main(List<String> args) async {
   final parser = ArgParser()..addFlag('help', abbr: 'h');

pubspec.yaml

@@ -11,6 +11,7 @@ dependencies:
   args: ^2.3.0
   flutter:
     sdk: flutter
+  http: ^0.13.3
   meta: ^1.3.0
   stack_trace: ^1.10.0