feat: Unify logging behavior across environments

android/capacitor/src/main/java/com/getcapacitor/Bridge.java

@@ -702,7 +702,7 @@ protected void savePermissionCall(PluginCall call) {
      */
     private JSInjector getJSInjector() {
         try {
-            String globalJS = JSExport.getGlobalJS(context, isDevMode());
+            String globalJS = JSExport.getGlobalJS(context, config.isLoggingEnabled(), isDevMode());
             String pluginJS = JSExport.getPluginJS(plugins.values());
             String cordovaJS = JSExport.getCordovaJS(context);
             String cordovaPluginsJS = JSExport.getCordovaPluginJS(context);

android/capacitor/src/main/java/com/getcapacitor/CapConfig.java

@@ -11,6 +11,7 @@
 import java.io.IOException;
 import java.util.HashMap;
 import java.util.Iterator;
+import java.util.Locale;
 import java.util.Map;
 import org.json.JSONException;
 import org.json.JSONObject;
@@ -20,6 +21,10 @@
  */
 public class CapConfig {
 
+    private static final String LOG_BEHAVIOR_NONE = "none";
+    private static final String LOG_BEHAVIOR_DEBUG = "debug";
+    private static final String LOG_BEHAVIOR_PRODUCTION = "production";
+
     // Server Config
     private boolean html5mode = true;
     private String serverUrl;
@@ -34,7 +39,7 @@ public class CapConfig {
     private boolean allowMixedContent = false;
     private boolean captureInput = false;
     private boolean webContentsDebuggingEnabled = false;
-    private boolean hideLogs = false;
+    private boolean loggingEnabled = true;
 
     // Embedded
     private String startPath;
@@ -110,7 +115,7 @@ private CapConfig(Builder builder) {
         this.allowMixedContent = builder.allowMixedContent;
         this.captureInput = builder.captureInput;
         this.webContentsDebuggingEnabled = builder.webContentsDebuggingEnabled;
-        this.hideLogs = builder.hideLogs;
+        this.loggingEnabled = builder.loggingEnabled;
 
         // Embedded
         this.startPath = builder.startPath;
@@ -137,6 +142,8 @@ private void loadConfig(AssetManager assetManager) {
      * Deserializes the config from JSON into a Capacitor Configuration object.
      */
     private void deserializeConfig(@Nullable Context context) {
+        boolean isDebug = context != null && (context.getApplicationInfo().flags & ApplicationInfo.FLAG_DEBUGGABLE) != 0;
+
         // Server
         html5mode = JSONUtils.getBoolean(configJSON, "server.html5mode", html5mode);
         serverUrl = JSONUtils.getString(configJSON, "server.url", null);
@@ -158,9 +165,27 @@ private void deserializeConfig(@Nullable Context context) {
                 JSONUtils.getBoolean(configJSON, "allowMixedContent", allowMixedContent)
             );
         captureInput = JSONUtils.getBoolean(configJSON, "android.captureInput", captureInput);
-        hideLogs = JSONUtils.getBoolean(configJSON, "android.hideLogs", JSONUtils.getBoolean(configJSON, "hideLogs", hideLogs));
-        webContentsDebuggingEnabled = context != null && (context.getApplicationInfo().flags & ApplicationInfo.FLAG_DEBUGGABLE) != 0;
-        webContentsDebuggingEnabled = JSONUtils.getBoolean(configJSON, "android.webContentsDebuggingEnabled", webContentsDebuggingEnabled);
+        webContentsDebuggingEnabled = JSONUtils.getBoolean(configJSON, "android.webContentsDebuggingEnabled", isDebug);
+
+        String logBehavior = JSONUtils.getString(
+            configJSON,
+            "android.loggingBehavior",
+            JSONUtils.getString(configJSON, "loggingBehavior", null)
+        );
+        if (logBehavior == null) {
+            boolean hideLogs = JSONUtils.getBoolean(configJSON, "android.hideLogs", JSONUtils.getBoolean(configJSON, "hideLogs", false));
+            logBehavior = hideLogs ? LOG_BEHAVIOR_NONE : LOG_BEHAVIOR_DEBUG;
+        }
+        switch (logBehavior.toLowerCase(Locale.ROOT)) {
+            case LOG_BEHAVIOR_PRODUCTION:
+                loggingEnabled = true;
+                break;
+            case LOG_BEHAVIOR_NONE:
+                loggingEnabled = false;
+                break;
+            default: // LOG_BEHAVIOR_DEBUG
+                loggingEnabled = isDebug;
+        }
 
         // Plugins
         pluginsConfiguration = deserializePluginsConfig(JSONUtils.getObject(configJSON, "plugins"));
@@ -214,8 +239,8 @@ public boolean isWebContentsDebuggingEnabled() {
         return webContentsDebuggingEnabled;
     }
 
-    public boolean isLogsHidden() {
-        return hideLogs;
+    public boolean isLoggingEnabled() {
+        return loggingEnabled;
     }
 
     public PluginConfig getPluginConfiguration(String pluginId) {
@@ -372,7 +397,7 @@ public static class Builder {
         private boolean allowMixedContent = false;
         private boolean captureInput = false;
         private Boolean webContentsDebuggingEnabled = null;
-        private boolean hideLogs = false;
+        private boolean loggingEnabled = true;
 
         // Embedded
         private String startPath = null;
@@ -467,8 +492,8 @@ public Builder setWebContentsDebuggingEnabled(boolean webContentsDebuggingEnable
             return this;
         }
 
-        public Builder setLogsHidden(boolean hideLogs) {
-            this.hideLogs = hideLogs;
+        public Builder setLoggingEnabled(boolean enabled) {
+            this.loggingEnabled = enabled;
             return this;
         }
     }

android/capacitor/src/main/java/com/getcapacitor/JSExport.java

@@ -11,8 +11,8 @@
 
 public class JSExport {
 
-    public static String getGlobalJS(Context context, boolean isDebug) {
-        return "window.Capacitor = { DEBUG: " + isDebug + ", Plugins: {} };";
+    public static String getGlobalJS(Context context, boolean loggingEnabled, boolean isDebug) {
+        return "window.Capacitor = { DEBUG: " + isDebug + ", isLoggingEnabled: " + loggingEnabled + ", Plugins: {} };";
     }
 
     public static String getCordovaJS(Context context) {

android/capacitor/src/main/java/com/getcapacitor/Logger.java

@@ -98,6 +98,6 @@ public static void error(String tag, String message, Throwable e) {
     }
 
     protected static boolean shouldLog() {
-        return config == null || !config.isLogsHidden();
+        return config == null || config.isLoggingEnabled();
     }
 }

android/capacitor/src/test/java/com/getcapacitor/ConfigBuildingTest.java

@@ -46,7 +46,7 @@ public void setup() {
                     .setAllowNavigation(new String[] { "http://www.google.com" })
                     .setAndroidScheme("test")
                     .setCaptureInput(true)
-                    .setLogsHidden(true)
+                    .setLoggingEnabled(true)
                     .setHTML5mode(false)
                     .setOverriddenUserAgentString("test-user-agent")
                     .setAppendedUserAgentString("test-append")
@@ -66,7 +66,7 @@ public void getCoreConfigValues() {
         assertArrayEquals(new String[] { "http://www.google.com" }, config.getAllowNavigation());
         assertEquals("test", config.getAndroidScheme());
         assertTrue(config.isInputCaptured());
-        assertTrue(config.isLogsHidden());
+        assertTrue(config.isLoggingEnabled());
         assertFalse(config.isHTML5Mode());
         assertEquals("test-user-agent", config.getOverriddenUserAgentString());
         assertEquals("test-append", config.getAppendedUserAgentString());

android/capacitor/src/test/java/com/getcapacitor/ConfigReadingTest.java

@@ -47,7 +47,7 @@ public void bad() {
             CapConfig config = CapConfig.loadDefault(context);
             assertEquals("not a real domain", config.getServerUrl());
             assertNull(config.getBackgroundColor());
-            assertFalse(config.isLogsHidden());
+            assertFalse(config.isLoggingEnabled());
         } catch (IOException e) {
             fail();
         }
@@ -62,7 +62,7 @@ public void flat() {
             assertEquals("level 1 override", config.getOverriddenUserAgentString());
             assertEquals("level 1 append", config.getAppendedUserAgentString());
             assertEquals("#ffffff", config.getBackgroundColor());
-            assertTrue(config.isLogsHidden());
+            assertFalse(config.isLoggingEnabled());
             assertEquals(1, config.getPluginConfiguration("SplashScreen").getInt("launchShowDuration", 0));
         } catch (IOException e) {
             fail();
@@ -78,7 +78,7 @@ public void hierarchy() {
             assertEquals("level 2 override", config.getOverriddenUserAgentString());
             assertEquals("level 2 append", config.getAppendedUserAgentString());
             assertEquals("#000000", config.getBackgroundColor());
-            assertFalse(config.isLogsHidden());
+            assertFalse(config.isLoggingEnabled());
         } catch (IOException e) {
             fail();
         }

cli/src/config.ts

@@ -62,6 +62,8 @@ export async function loadConfig(): Promise<Config> {
     },
   };
 
+  checkExternalConfig(conf);
+
   debug('config: %O', config);
 
   return config;
@@ -435,3 +437,16 @@ const config: CapacitorConfig = ${formatJSObject(extConfig)};
 
 export default config;\n`;
 }
+
+function checkExternalConfig(config: ExtConfigPairs): void {
+  if (
+    typeof config.extConfig.hideLogs !== 'undefined' ||
+    typeof config.extConfig.android?.hideLogs !== 'undefined' ||
+    typeof config.extConfig.ios?.hideLogs !== 'undefined'
+  ) {
+    logger.warn(
+      `The ${c.strong('hideLogs')} configuration option has been deprecated. ` +
+        `Please update to use ${c.strong('loggingBehavior')} instead.`,
+    );
+  }
+}

cli/src/declarations.ts

@@ -45,10 +45,20 @@ export interface CapacitorConfig {
    * Hide or show the native logs for iOS and Android.
    *
    * @since 2.1.0
+   * @deprecated 3.0.0
    * @default false
    */
   hideLogs?: boolean;
 
+  /**
+   * The build configuration under which Capacitor will generate logs.
+   * 'production' also implies logging in debug.
+   *
+   * @since 3.0.0
+   * @default debug
+   */
+  loggingBehavior?: 'none' | 'debug' | 'production';
+
   /**
    * User agent of Capacitor Web View.
    *
@@ -152,10 +162,21 @@ export interface CapacitorConfig {
      * Overrides global `hideLogs` option.
      *
      * @since 2.1.0
+     * @deprecated 3.0.0
      * @default false
      */
     hideLogs?: boolean;
 
+    /**
+     * The build configuration under which Capacitor will generate logs.
+     *
+     * Overrides global `loggingBehavior` option.
+     *
+     * @since 3.0.0
+     * @default debug
+     */
+    loggingBehavior?: 'none' | 'debug' | 'production';
+
     /**
      * Allowlist of plugins to include during `npx cap sync` for Android.
      *
@@ -268,11 +289,22 @@ export interface CapacitorConfig {
      *
      * Overrides global `hideLogs` option.
      *
-     * @since 1.1.0
+     * @since 2.1.0
+     * @deprecated 3.0.0
      * @default false
      */
     hideLogs?: boolean;
 
+    /**
+     * The build configuration under which Capacitor will generate logs.
+     *
+     * Overrides global `loggingBehavior` option.
+     *
+     * @since 3.0.0
+     * @default debug
+     */
+    loggingBehavior?: 'none' | 'debug' | 'production';
+
     /**
      * Allowlist of plugins to include during `npx cap sync` for iOS.
      *

core/src/bridge.ts

@@ -110,7 +110,7 @@ export const initBridge = (
           options: options || {},
         };
 
-        if (cap.DEBUG && pluginName !== 'Console') {
+        if (cap.isLoggingEnabled && pluginName !== 'Console') {
           cap.logToNative(callData);
         }
 
@@ -132,7 +132,7 @@ export const initBridge = (
    * Process a response from the native layer.
    */
   cap.fromNative = (result: PluginResult) => {
-    if (cap.DEBUG && result.pluginId !== 'Console') {
+    if (cap.isLoggingEnabled && result.pluginId !== 'Console') {
       cap.logFromNative(result);
     }
 

core/src/definitions.ts

@@ -80,6 +80,7 @@ export interface CapacitorGlobal {
   ) => void;
 
   DEBUG?: boolean;
+  isLoggingEnabled?: boolean;
 
   // Deprecated in v3, will be removed from v4
 

core/src/runtime.ts

@@ -251,6 +251,7 @@ export const createCapacitor = (win: WindowCapacitor): CapacitorInstance => {
   cap.registerPlugin = registerPlugin;
   cap.Exception = CapacitorException;
   cap.DEBUG = !!cap.DEBUG;
+  cap.isLoggingEnabled = !!cap.isLoggingEnabled;
 
   initBridge(win, cap);
   initEvents(win, cap);

core/src/tests/runtime.spec.ts

@@ -42,6 +42,19 @@ describe('runtime', () => {
     expect(cap.DEBUG).toBe(true);
   });
 
+  it('isLoggingEnabled false default', () => {
+    cap = createCapacitor(win);
+    expect(cap.isLoggingEnabled).toBe(false);
+  });
+
+  it('isLoggingEnabled set from window.Capacitor.isLoggingEnabled', () => {
+    (win as any).Capacitor = {
+      isLoggingEnabled: true,
+    };
+    cap = createCapacitor(win);
+    expect(cap.isLoggingEnabled).toBe(true);
+  });
+
   it('cannot reset server url after initializing capacitor', () => {
     win.WEBVIEW_SERVER_URL = 'whatever://home';
     cap = createCapacitor(win);

ios/Capacitor/Capacitor.xcodeproj/project.pbxproj

@@ -29,6 +29,7 @@
 		623D691E254C7462002D01D1 /* CAPInstanceConfiguration.m in Sources */ = {isa = PBXBuildFile; fileRef = 623D691C254C7462002D01D1 /* CAPInstanceConfiguration.m */; };
 		625AF1ED258963C700869675 /* WebViewAssetHandler.swift in Sources */ = {isa = PBXBuildFile; fileRef = 625AF1EC258963C700869675 /* WebViewAssetHandler.swift */; };
 		6263686025F6EC0100576C1C /* PluginCallAccessorTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 6263685F25F6EC0100576C1C /* PluginCallAccessorTests.m */; };
+		626D2D992613B61E0046CE81 /* hidinglogs.json in CopyFiles */ = {isa = PBXBuildFile; fileRef = 626D2D902613B4BB0046CE81 /* hidinglogs.json */; };
 		62959B162524DA7800A3D7F1 /* CAPPluginCall.h in Headers */ = {isa = PBXBuildFile; fileRef = 62959AE22524DA7700A3D7F1 /* CAPPluginCall.h */; settings = {ATTRIBUTES = (Public, ); }; };
 		62959B172524DA7800A3D7F1 /* JSExport.swift in Sources */ = {isa = PBXBuildFile; fileRef = 62959AE32524DA7700A3D7F1 /* JSExport.swift */; };
 		62959B192524DA7800A3D7F1 /* CAPBridgedPlugin.h in Headers */ = {isa = PBXBuildFile; fileRef = 62959AE52524DA7700A3D7F1 /* CAPBridgedPlugin.h */; settings = {ATTRIBUTES = (Public, ); }; };
@@ -108,6 +109,7 @@
 			dstPath = configurations;
 			dstSubfolderSpec = 1;
 			files = (
+				626D2D992613B61E0046CE81 /* hidinglogs.json in CopyFiles */,
 				62A91C3F2553710E00861508 /* nonjson.json in CopyFiles */,
 				62E0736125535E8700BAAADB /* server.json in CopyFiles */,
 				62E0736225535E8700BAAADB /* bad.json in CopyFiles */,
@@ -155,6 +157,7 @@
 		623D691C254C7462002D01D1 /* CAPInstanceConfiguration.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = CAPInstanceConfiguration.m; sourceTree = "<group>"; };
 		625AF1EC258963C700869675 /* WebViewAssetHandler.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = WebViewAssetHandler.swift; sourceTree = "<group>"; };
 		6263685F25F6EC0100576C1C /* PluginCallAccessorTests.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = PluginCallAccessorTests.m; sourceTree = "<group>"; };
+		626D2D902613B4BB0046CE81 /* hidinglogs.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = hidinglogs.json; sourceTree = "<group>"; };
 		62959AE22524DA7700A3D7F1 /* CAPPluginCall.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = CAPPluginCall.h; sourceTree = "<group>"; };
 		62959AE32524DA7700A3D7F1 /* JSExport.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = JSExport.swift; sourceTree = "<group>"; };
 		62959AE52524DA7700A3D7F1 /* CAPBridgedPlugin.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = CAPBridgedPlugin.h; sourceTree = "<group>"; };
@@ -376,6 +379,7 @@
 				62E0735225535E6500BAAADB /* server.json */,
 				62E0735325535E6500BAAADB /* bad.json */,
 				62E0735425535E6500BAAADB /* flat.json */,
+				626D2D902613B4BB0046CE81 /* hidinglogs.json */,
 				62E0735525535E6500BAAADB /* hierarchy.json */,
 				62A91C392553710300861508 /* nonjson.json */,
 			);

ios/Capacitor/Capacitor/CAPBridgeViewController.swift

@@ -29,8 +29,8 @@ import Cordova
     override public final func loadView() {
         // load the configuration and set the logging flag
         let configDescriptor = instanceDescriptor()
-        let configuration = InstanceConfiguration(with: configDescriptor)
-        CAPLog.enableLogging = configuration.enableLogging
+        let configuration = InstanceConfiguration(with: configDescriptor, isDebug: CapacitorBridge.isDevEnvironment)
+        CAPLog.enableLogging = configuration.loggingEnabled
         logWarnings(for: configDescriptor)
 
         if configDescriptor.instanceType == .fixed {
@@ -276,7 +276,7 @@ extension CAPBridgeViewController {
         aWebView.scrollView.contentInsetAdjustmentBehavior = configuration.contentInsetAdjustmentBehavior
         aWebView.allowsLinkPreview = configuration.allowLinkPreviews
         aWebView.configuration.preferences.setValue(true, forKey: "allowFileAccessFromFileURLs")
-        aWebView.scrollView.isScrollEnabled = configuration.enableScrolling
+        aWebView.scrollView.isScrollEnabled = configuration.scrollingEnabled
         if let overrideUserAgent = configuration.overridenUserAgentString {
             aWebView.customUserAgent = overrideUserAgent
         }

ios/Capacitor/Capacitor/CAPInstanceConfiguration.h

@@ -14,8 +14,8 @@ NS_SWIFT_NAME(InstanceConfiguration)
 @property (nonatomic, readonly, nonnull) NSURL *localURL;
 @property (nonatomic, readonly, nonnull) NSURL *serverURL;
 @property (nonatomic, readonly, nonnull) NSDictionary *pluginConfigurations;
-@property (nonatomic, readonly) BOOL enableLogging;
-@property (nonatomic, readonly) BOOL enableScrolling;
+@property (nonatomic, readonly) BOOL loggingEnabled;
+@property (nonatomic, readonly) BOOL scrollingEnabled;
 @property (nonatomic, readonly) BOOL allowLinkPreviews;
 @property (nonatomic, readonly) BOOL handleApplicationNotifications;
 @property (nonatomic, readonly) BOOL cordovaDeployDisabled;
@@ -25,7 +25,7 @@ NS_SWIFT_NAME(InstanceConfiguration)
 
 @property (nonatomic, readonly, nonnull) NSDictionary *legacyConfig DEPRECATED_MSG_ATTRIBUTE("Use direct properties instead");
 
-- (instancetype _Nonnull)initWithDescriptor:(CAPInstanceDescriptor* _Nonnull)descriptor NS_SWIFT_NAME(init(with:));
+- (instancetype _Nonnull)initWithDescriptor:(CAPInstanceDescriptor* _Nonnull)descriptor isDebug:(BOOL)debug NS_SWIFT_NAME(init(with:isDebug:));
 - (instancetype _Nonnull)updatingAppLocation:(NSURL* _Nonnull)location NS_SWIFT_NAME(updatingAppLocation(_:));
 @end