5/n Check protected mode is applied before filtering the sensitive params

Summary:
**Context**
Protected Mode is introduced for finance / health related apps integrity purpose last year. It will drop ALL the non-standard parameters of the event. Sensitive parameters filtering will drop the particular parameters for each event based on the config fetched from server for all apps (eventually). Given that we never need to filter the standard parameters, we check if the protected mode is applied before processing the sensitive param filter.

**Change in this diff**
- Add a check of protected mode is applied

Reviewed By: KylinChang

Differential Revision: D53818572

fbshipit-source-id: 360d2b173347568c44faf2d43683a05be00e727a

Author: Xinzhu Cai

facebook-core/src/main/java/com/facebook/appevents/AppEvent.kt

@@ -15,6 +15,7 @@ import com.facebook.FacebookException
 import com.facebook.LoggingBehavior
 import com.facebook.appevents.eventdeactivation.EventDeactivationManager.processDeprecatedParameters
 import com.facebook.appevents.integrity.IntegrityManager
+import com.facebook.appevents.integrity.ProtectedModeManager.protectedModeIsApplied
 import com.facebook.appevents.integrity.RedactedEventsManager
 import com.facebook.appevents.integrity.SensitiveParamsManager.processFilterSensitiveParams
 import com.facebook.appevents.internal.AppEventUtility.bytesToHex
@@ -43,27 +44,27 @@ class AppEvent : Serializable {
 
   @Throws(JSONException::class, FacebookException::class)
   constructor(
-      contextName: String,
-      eventName: String,
-      valueToSum: Double?,
-      parameters: Bundle?,
-      isImplicitlyLogged: Boolean,
-      isInBackground: Boolean,
-      currentSessionId: UUID?
+          contextName: String,
+          eventName: String,
+          valueToSum: Double?,
+          parameters: Bundle?,
+          isImplicitlyLogged: Boolean,
+          isInBackground: Boolean,
+          currentSessionId: UUID?
   ) {
     isImplicit = isImplicitlyLogged
     inBackground = isInBackground
     name = eventName
     jsonObject =
-        getJSONObjectForAppEvent(contextName, eventName, valueToSum, parameters, currentSessionId)
+            getJSONObjectForAppEvent(contextName, eventName, valueToSum, parameters, currentSessionId)
     checksum = calculateChecksum()
   }
 
   private constructor(
-      jsonString: String,
-      isImplicit: Boolean,
-      inBackground: Boolean,
-      checksum: String?
+          jsonString: String,
+          isImplicit: Boolean,
+          inBackground: Boolean,
+          checksum: String?
   ) {
     jsonObject = JSONObject(jsonString)
     this.isImplicit = isImplicit
@@ -79,10 +80,10 @@ class AppEvent : Serializable {
   // for old events we don't have a checksum
   val isChecksumValid: Boolean
     get() =
-        if (checksum == null) {
-          // for old events we don't have a checksum
-          true
-        } else calculateChecksum() == checksum
+      if (checksum == null) {
+        // for old events we don't have a checksum
+        true
+      } else calculateChecksum() == checksum
 
   private fun getJSONObjectForAppEvent(
           contextName: String,
@@ -134,26 +135,28 @@ class AppEvent : Serializable {
       val value = parameters[key]
       if (value !is String && value !is Number) {
         throw FacebookException(
-            String.format(
-                "Parameter value '%s' for key '%s' should be a string" + " or a numeric type.",
-                value,
-                key))
+                String.format(
+                        "Parameter value '%s' for key '%s' should be a string" + " or a numeric type.",
+                        value,
+                        key))
       }
       paramMap[key] = value.toString()
     }
-    processFilterSensitiveParams(paramMap as MutableMap<String, String?>, name)
+    if (!protectedModeIsApplied(parameters)) {
+      processFilterSensitiveParams(paramMap as MutableMap<String, String?>, name)
+    }
     IntegrityManager.processParameters(paramMap)
     processParameters(paramMap as MutableMap<String, String?>, name)
     processDeprecatedParameters(paramMap as MutableMap<String, String?>, name)
     return paramMap
   }
-  
+
   internal class SerializationProxyV2
   constructor(
-      private val jsonString: String,
-      private val isImplicit: Boolean,
-      private val inBackground: Boolean,
-      private val checksum: String?
+          private val jsonString: String,
+          private val isImplicit: Boolean,
+          private val inBackground: Boolean,
+          private val checksum: String?
   ) : Serializable {
     @Throws(JSONException::class, ObjectStreamException::class)
     private fun readResolve(): Any {
@@ -172,10 +175,10 @@ class AppEvent : Serializable {
 
   override fun toString(): String {
     return String.format(
-        "\"%s\", implicit: %b, json: %s",
-        jsonObject.optString("_eventName"),
-        isImplicit,
-        jsonObject.toString())
+            "\"%s\", implicit: %b, json: %s",
+            jsonObject.optString("_eventName"),
+            isImplicit,
+            jsonObject.toString())
   }
 
   private fun calculateChecksum(): String {
@@ -215,11 +218,11 @@ class AppEvent : Serializable {
           identifier = "<None Provided>"
         }
         throw FacebookException(
-            String.format(
-                Locale.ROOT,
-                "Identifier '%s' must be less than %d characters",
-                identifier,
-                MAX_IDENTIFIER_LENGTH))
+                String.format(
+                        Locale.ROOT,
+                        "Identifier '%s' must be less than %d characters",
+                        identifier,
+                        MAX_IDENTIFIER_LENGTH))
       }
       var alreadyValidated: Boolean
       synchronized(validatedIdentifiers) {
@@ -230,11 +233,11 @@ class AppEvent : Serializable {
           synchronized(validatedIdentifiers) { validatedIdentifiers.add(identifier) }
         } else {
           throw FacebookException(
-              String.format(
-                  "Skipping event named '%s' due to illegal name - must be " +
-                      "under 40 chars and alphanumeric, _, - or space, and " +
-                      "not start with a space or hyphen.",
-                  identifier))
+                  String.format(
+                          "Skipping event named '%s' due to illegal name - must be " +
+                                  "under 40 chars and alphanumeric, _, - or space, and " +
+                                  "not start with a space or hyphen.",
+                          identifier))
         }
       }
     }