Merge pull request #24 from hotwired/strada-logging

Move the `debugLoggingEnabled` flag to the Strada.config namespace

strada/src/main/kotlin/dev/hotwire/strada/StradaConfig.kt

@@ -1,5 +1,17 @@
 package dev.hotwire.strada
 
 class StradaConfig internal constructor() {
+    /**
+     * Set a custom JSON converter to easily decode Message.dataJson to a data
+     * object in received messages and to encode a data object back to json to
+     * reply with a custom message back to the web.
+     */
     var jsonConverter: StradaJsonConverter? = null
+
+    /**
+     * Enable debug logging to see message communication from/to the WebView.
+     *
+     * NOTE: You should not enable debug logging in production builds.
+     */
+    var debugLoggingEnabled = false
 }

strada/src/main/kotlin/dev/hotwire/strada/StradaLog.kt

@@ -3,21 +3,16 @@ package dev.hotwire.strada
 import android.util.Log
 
 @Suppress("unused")
-object StradaLog {
+internal object StradaLog {
     private const val DEFAULT_TAG = "StradaLog"
 
-    /**
-     * Enable debug logging to see message communication from/to the WebView.
-     */
-    var debugLoggingEnabled = false
-
     internal fun d(msg: String) = log(Log.DEBUG, DEFAULT_TAG, msg)
 
     internal fun e(msg: String) = log(Log.ERROR, DEFAULT_TAG, msg)
 
     private fun log(logLevel: Int, tag: String, msg: String) {
         when (logLevel) {
-            Log.DEBUG -> if (debugLoggingEnabled) Log.d(tag, msg)
+            Log.DEBUG -> if (Strada.config.debugLoggingEnabled) Log.d(tag, msg)
             Log.ERROR -> Log.e(tag, msg)
         }
     }