From 813d275d1262a0ef74313beac66f4b0d78f3dc26 Mon Sep 17 00:00:00 2001
From: NaiJi <naijiworld@protonmail.com>
Date: Mon, 19 Dec 2022 22:47:35 +0400
Subject: [PATCH 1/2] feat: Implement logging for GraphQL API map

Log to application console all requests and all responses
---
 lib/logic/api_maps/graphql_maps/api_map.dart | 58 ++++++++++++++++++--
 1 file changed, 52 insertions(+), 6 deletions(-)

diff --git a/lib/logic/api_maps/graphql_maps/api_map.dart b/lib/logic/api_maps/graphql_maps/api_map.dart
index 185a5e54..0f786417 100644
--- a/lib/logic/api_maps/graphql_maps/api_map.dart
+++ b/lib/logic/api_maps/graphql_maps/api_map.dart
@@ -1,9 +1,48 @@
+// ignore_for_file: prefer_foreach
+
+import 'dart:async';
 import 'dart:io';
 
 import 'package:graphql_flutter/graphql_flutter.dart';
 import 'package:http/io_client.dart';
 import 'package:selfprivacy/config/get_it_config.dart';
 import 'package:selfprivacy/logic/api_maps/staging_options.dart';
+import 'package:selfprivacy/logic/models/message.dart';
+
+void _logToAppConsole<T>(final T objectToLog) {
+  getIt.get<ConsoleModel>().addMessage(
+        Message(
+          text: objectToLog.toString(),
+        ),
+      );
+}
+
+class LoggingLink extends Link {
+  @override
+  Stream<Response> request(
+    final Request request, [
+    final NextLink? forward,
+  ]) async* {
+    _logToAppConsole(request);
+    yield* forward!(request);
+  }
+}
+
+class LoggingResponseParser extends ResponseParser {
+  @override
+  Response parseResponse(final Map<String, dynamic> body) {
+    final response = super.parseResponse(body);
+    _logToAppConsole(response);
+    return response;
+  }
+
+  @override
+  GraphQLError parseError(final Map<String, dynamic> error) {
+    final graphQlError = super.parseError(error);
+    _logToAppConsole(graphQlError);
+    return graphQlError;
+  }
+}
 
 abstract class ApiMap {
   Future<GraphQLClient> getClient() async {
@@ -22,16 +61,23 @@ abstract class ApiMap {
     final httpLink = HttpLink(
       'https://api.$rootAddress/graphql',
       httpClient: ioClient,
+      parser: LoggingResponseParser(),
     );
 
     final String token = _getApiToken();
 
-    final Link graphQLLink = isWithToken
-        ? AuthLink(
-            getToken: () async =>
-                customToken == '' ? 'Bearer $token' : customToken,
-          ).concat(httpLink)
-        : httpLink;
+    final Link graphQLLink = LoggingLink().concat(
+      isWithToken
+          ? AuthLink(
+              getToken: () async =>
+                  customToken == '' ? 'Bearer $token' : customToken,
+            ).concat(httpLink)
+          : httpLink,
+    );
+
+    // Every request goes through either chain:
+    // 1. AuthLink -> HttpLink -> LoggingLink
+    // 2.             HttpLink -> LoggingLink
 
     return GraphQLClient(
       cache: GraphQLCache(),

From 8dffcab30d28caab2b679ef5c5c99c2be3d3ea41 Mon Sep 17 00:00:00 2001
From: NaiJi <naijiworld@protonmail.com>
Date: Mon, 19 Dec 2022 22:53:07 +0400
Subject: [PATCH 2/2] chore: Rename and unify GraphQL logging classes

---
 lib/logic/api_maps/graphql_maps/api_map.dart | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/lib/logic/api_maps/graphql_maps/api_map.dart b/lib/logic/api_maps/graphql_maps/api_map.dart
index 0f786417..02b9bc04 100644
--- a/lib/logic/api_maps/graphql_maps/api_map.dart
+++ b/lib/logic/api_maps/graphql_maps/api_map.dart
@@ -1,6 +1,3 @@
-// ignore_for_file: prefer_foreach
-
-import 'dart:async';
 import 'dart:io';
 
 import 'package:graphql_flutter/graphql_flutter.dart';
@@ -17,7 +14,7 @@ void _logToAppConsole<T>(final T objectToLog) {
       );
 }
 
-class LoggingLink extends Link {
+class RequestLoggingLink extends Link {
   @override
   Stream<Response> request(
     final Request request, [
@@ -28,7 +25,7 @@ class LoggingLink extends Link {
   }
 }
 
-class LoggingResponseParser extends ResponseParser {
+class ResponseLoggingParser extends ResponseParser {
   @override
   Response parseResponse(final Map<String, dynamic> body) {
     final response = super.parseResponse(body);
@@ -61,12 +58,12 @@ abstract class ApiMap {
     final httpLink = HttpLink(
       'https://api.$rootAddress/graphql',
       httpClient: ioClient,
-      parser: LoggingResponseParser(),
+      parser: ResponseLoggingParser(),
     );
 
     final String token = _getApiToken();
 
-    final Link graphQLLink = LoggingLink().concat(
+    final Link graphQLLink = RequestLoggingLink().concat(
       isWithToken
           ? AuthLink(
               getToken: () async =>
@@ -76,8 +73,8 @@ abstract class ApiMap {
     );
 
     // Every request goes through either chain:
-    // 1. AuthLink -> HttpLink -> LoggingLink
-    // 2.             HttpLink -> LoggingLink
+    // 1. RequestLoggingLink -> AuthLink -> HttpLink
+    // 2. RequestLoggingLink -> HttpLink
 
     return GraphQLClient(
       cache: GraphQLCache(),