Fixes in pinning configuration (#129)

Author: gthea

splitio/lib/splitio.dart

@@ -9,6 +9,7 @@ export 'package:splitio_platform_interface/split_impression.dart';
 export 'package:splitio_platform_interface/split_result.dart';
 export 'package:splitio_platform_interface/split_sync_config.dart';
 export 'package:splitio_platform_interface/split_view.dart';
+export 'package:splitio_platform_interface/split_certificate_pinning_configuration.dart';
 
 typedef ClientReadinessCallback = void Function(SplitClient splitClient);
 

splitio_platform_interface/lib/certificate_pinning_configuration.dart renamed to splitio_platform_interface/lib/split_certificate_pinning_configuration.dart

@@ -1,8 +1,8 @@
 class CertificatePinningConfiguration {
 
-  final Map<String, Set<String>> _pins = {};
+  final Map<String, List<String>> _pins = {};
 
-  Map<String, Set<String>> get pins => _pins;
+  Map<String, List<String>> get pins => _pins;
 
   CertificatePinningConfiguration addPin(String host, String pin) {
     pin = pin.trim();
@@ -11,7 +11,7 @@ class CertificatePinningConfiguration {
     }
 
     if (!_pins.containsKey(host)) {
-      _pins[host] = <String>{};
+      _pins[host] = [];
     }
 
     _pins[host]?.add(pin);

splitio_platform_interface/lib/split_configuration.dart

@@ -1,4 +1,4 @@
-import 'package:splitio_platform_interface/certificate_pinning_configuration.dart';
+import 'package:splitio_platform_interface/split_certificate_pinning_configuration.dart';
 import 'package:splitio_platform_interface/split_sync_config.dart';
 
 class SplitConfiguration {
@@ -46,7 +46,7 @@ class SplitConfiguration {
   ///
   /// [readyTimeout] Maximum amount of time in seconds to wait before firing the SDK_READY_TIMED_OUT event. Defaults to 10 seconds.
   ///
-  /// [certificatePinningConfig] Certificate pinning configuration. It is a map where the key is the host and the value is a set of base64 SHA-256 or base64 SHA-1 hashes of the SPKI.
+  /// [certificatePinningConfiguration] Certificate pinning configuration. It is a map where the key is the host and the value is a set of base64 SHA-256 or base64 SHA-1 hashes of the SPKI.
   SplitConfiguration({
     int? featuresRefreshRate,
     int? segmentsRefreshRate,
@@ -73,7 +73,7 @@ class SplitConfiguration {
     bool? encryptionEnabled,
     SplitLogLevel? logLevel,
     int? readyTimeout = 10,
-    CertificatePinningConfiguration? certificatePinningConfig,
+    CertificatePinningConfiguration? certificatePinningConfiguration,
   }) {
     if (featuresRefreshRate != null) {
       configurationMap['featuresRefreshRate'] = featuresRefreshRate;
@@ -180,10 +180,10 @@ class SplitConfiguration {
       configurationMap['readyTimeout'] = readyTimeout;
     }
 
-    if (certificatePinningConfig != null &&
-        certificatePinningConfig.pins.isNotEmpty) {
+    if (certificatePinningConfiguration != null &&
+        certificatePinningConfiguration.pins.isNotEmpty) {
       configurationMap['certificatePinningConfiguration'] = {
-        'pins': certificatePinningConfig.pins
+        'pins': certificatePinningConfiguration.pins
       };
     }
   }

splitio_platform_interface/test/certificate_pinning_configuration_test.dart

@@ -1,5 +1,5 @@
 import 'package:flutter_test/flutter_test.dart';
-import 'package:splitio_platform_interface/certificate_pinning_configuration.dart';
+import 'package:splitio_platform_interface/split_certificate_pinning_configuration.dart';
 
 void main() {
   test('single pin can be added', () {

splitio_platform_interface/test/splitio_configuration_test.dart

@@ -1,5 +1,5 @@
 import 'package:flutter_test/flutter_test.dart';
-import 'package:splitio_platform_interface/certificate_pinning_configuration.dart';
+import 'package:splitio_platform_interface/split_certificate_pinning_configuration.dart';
 import 'package:splitio_platform_interface/split_configuration.dart';
 import 'package:splitio_platform_interface/split_sync_config.dart';
 
@@ -31,7 +31,7 @@ void main() {
         encryptionEnabled: true,
         logLevel: SplitLogLevel.debug,
         readyTimeout: 1,
-        certificatePinningConfig: CertificatePinningConfiguration()
+        certificatePinningConfiguration: CertificatePinningConfiguration()
             .addPin('host1', 'pin1')
             .addPin('host2', 'pin3')
             .addPin('host1', 'pin2'));