[#114] Config.extraInfo now optional

CHANGELOG.md

@@ -6,7 +6,9 @@ The format is based on the Steamclock [Release Management Guide](https://github.
 ## Unreleased 
 - Allow more than one log to be attached to a Sentry report (#115)
 - Fixed issue with log auto rotation (#115)
+- Config.extraInfo now optional (#114)
 
+---
 
 ## Jitpack v2.3 : Jun 19, 2023
 - Added UserReports (#110)

app/src/main/java/com/steamclock/steamclogsample/App.kt

@@ -15,20 +15,7 @@ class App : Application() {
             fileWritePath = externalCacheDir,
             autoRotateConfig = AutoRotateConfig(10L), // Short rotate so we can more easily test
             filtering = appFiltering,
-            detailedLogsOnUserReports = true,
-            extraInfo = { purpose ->
-                when (purpose) {
-                    ExtraInfoPurpose.Error -> {
-                        mapOf("ExtraInfoPurpose" to "Error")
-                    }
-                    ExtraInfoPurpose.Fatal -> {
-                        mapOf("ExtraInfoPurpose" to "Fatal")
-                    }
-                    ExtraInfoPurpose.UserReport -> {
-                        mapOf("ExtraInfoPurpose" to "UserReport")
-                    }
-                }
-            }
+            detailedLogsOnUserReports = true
         ))
     }
 

app/src/main/java/com/steamclock/steamclogsample/MainActivity.kt

@@ -21,6 +21,20 @@ class MainActivity : AppCompatActivity() {
 
     private lateinit var binding: ActivityMainBinding
 
+    private fun getExtraInfo(purpose: ExtraInfoPurpose): Map<String, Any> {
+        return when (purpose) {
+            ExtraInfoPurpose.Error -> {
+                mapOf("ExtraInfoPurpose" to "Error")
+            }
+            ExtraInfoPurpose.Fatal -> {
+                mapOf("ExtraInfoPurpose" to "Fatal")
+            }
+            ExtraInfoPurpose.UserReport -> {
+                mapOf("ExtraInfoPurpose" to "UserReport")
+            }
+        }
+    }
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         binding = ActivityMainBinding.inflate(layoutInflater)
@@ -46,6 +60,9 @@ class MainActivity : AppCompatActivity() {
         binding.enableUserReportLogs.setOnCheckedChangeListener { _, checked ->
             clog.config.detailedLogsOnUserReports = checked
         }
+        binding.enableExtraConfigInfo.setOnCheckedChangeListener { _, checked ->
+            clog.config.extraInfo = if (checked) { this::getExtraInfo } else null
+        }
 
         binding.addUserId.setOnClickListener { clog.setUserId("1234") }
 

app/src/main/res/layout/activity_main.xml

@@ -55,6 +55,12 @@
             android:layout_width="match_parent"
             android:layout_height="wrap_content" />
 
+        <CheckBox
+            android:text="Add extra config info"
+            android:id="@+id/enable_extra_config_info"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content" />
+
         <View
             android:layout_width="match_parent"
             android:layout_height="20dp" />

steamclog/src/main/java/com/steamclock/steamclog/Config.kt

@@ -63,7 +63,7 @@ data class Config(
      * extra info passed into individual logging functions, this info is not redacted in any
      * way even if requireRedacted is set, the callback must handle and privacy preservation or redaction
      */
-    var extraInfo: (ExtraInfoPurpose) -> Map<String, Any>
+    var extraInfo: ((ExtraInfoPurpose) -> Map<String, Any>)? = null
 ) {
     override fun toString(): String {
         return "Config(" +

steamclog/src/main/java/com/steamclock/steamclog/Steamclog.kt

@@ -101,7 +101,7 @@ object SteamcLog {
         obj: Any?,
         purpose: ExtraInfoPurpose?
     ) {
-        val extraInfo = purpose?.let { config.extraInfo(purpose) }
+        val extraInfo = purpose?.let { config.extraInfo?.invoke(purpose) }
         val redactableObjectData =  obj?.getRedactedDescription()
         val attachLogFiles = purpose == ExtraInfoPurpose.UserReport && config.detailedLogsOnUserReports