Merge pull request #1027 from appwrite/fix-pong-null-safety

fix: added null checks for message.data & pong type checking

Author: christyjacob4

templates/android/library/src/main/java/io/package/models/RealtimeModels.kt.twig

@@ -17,7 +17,7 @@ data class RealtimeCallback(
 
 open class RealtimeResponse(
     val type: String,
-    val data: Any
+    val data: Any?
 )
 
 data class RealtimeResponseEvent<T>(

templates/android/library/src/main/java/io/package/services/Realtime.kt.twig

@@ -175,16 +175,17 @@ class Realtime(client: Client) : Service(client), CoroutineScope {
                 when (message.type) {
                     TYPE_ERROR -> handleResponseError(message)
                     TYPE_EVENT -> handleResponseEvent(message)
+                    TYPE_PONG -> {}
                 }
             }
         }
 
         private fun handleResponseError(message: RealtimeResponse) {
-            throw message.data.jsonCast<{{ spec.title | caseUcfirst }}Exception>()
+            throw message.data?.jsonCast<{{ spec.title | caseUcfirst }}Exception>() ?: RuntimeException("Data is not present")
         }
 
         private suspend fun handleResponseEvent(message: RealtimeResponse) {
-            val event = message.data.jsonCast<RealtimeResponseEvent<Any>>()
+            val event = message.data?.jsonCast<RealtimeResponseEvent<Any>>() ?: return
             if (event.channels.isEmpty()) {
                 return
             }

templates/flutter/lib/src/realtime_response.dart.twig

@@ -30,7 +30,7 @@ class RealtimeResponse {
   factory RealtimeResponse.fromMap(Map<String, dynamic> map) {
     return RealtimeResponse(
       type: map['type'],
-      data: Map<String, dynamic>.from(map['data']),
+      data: Map<String, dynamic>.from(map['data'] ?? {}),
     );
   }
 

templates/react-native/src/client.ts.twig

@@ -11,8 +11,8 @@ type Headers = {
 }
 
 type RealtimeResponse = {
-    type: 'error' | 'event' | 'connected' | 'response';
-    data: RealtimeResponseAuthenticated | RealtimeResponseConnected | RealtimeResponseError | RealtimeResponseEvent<unknown>;
+    type: 'error' | 'event' | 'connected' | 'response' | 'pong';
+    data: RealtimeResponseAuthenticated | RealtimeResponseConnected | RealtimeResponseError | RealtimeResponseEvent<unknown> | undefined;
 }
 
 type RealtimeRequest = {
@@ -301,6 +301,8 @@ class Client {
                             })
                         }
                         break;
+                    case 'pong':
+                        break; // Handle pong response if needed
                     case 'error':
                         throw message.data;
                     default:

templates/web/src/client.ts.twig

@@ -19,14 +19,14 @@ type Headers = {
  */
 type RealtimeResponse = {
     /**
-     * Type of the response: 'error', 'event', 'connected', 'pong', or 'response'.
+     * Type of the response: 'error', 'event', 'connected', 'response' or 'pong'.
      */
     type: 'error' | 'event' | 'connected' | 'response' | 'pong';
 
     /**
      * Data associated with the response based on the response type.
      */
-    data: RealtimeResponseAuthenticated | RealtimeResponseConnected | RealtimeResponseError | RealtimeResponseEvent<unknown>;
+    data: RealtimeResponseAuthenticated | RealtimeResponseConnected | RealtimeResponseError | RealtimeResponseEvent<unknown> | undefined;
 }
 
 /**
@@ -496,6 +496,8 @@ class Client {
                             })
                         }
                         break;
+                    case 'pong':
+                        break; // Handle pong response if needed
                     case 'error':
                         throw message.data;
                     default: