LocalIP / NetIO: make defaultRouteOnly defaults to true

Author: CarterLi

CHANGELOG.md

@@ -13,6 +13,7 @@ Changes:
     * `--battery-use-setup-api`: control whether `SetupAPI` should be used on Windows to detect battery info, which supports multi batteries, but slower.
     * `--wm-detect-plugin`: control whether WM plugins should be detected. Note it's implemented with global processes enumeration and can report false results.
     * `--de-slow-version-detection`: control DE version should be detected with slow operations. It's usually not necessary and only provided as a backup.
+* `--localip-default-route-only` and `--netio-default-route-only` defaults to true to avoid large number of results
 
 Features:
 * Quirks for MIPS platforms (CPU, Linux)

doc/json_schema.json

@@ -1243,7 +1243,7 @@
                                     "defaultRouteOnly": {
                                         "description": "Show ips that are used for default routing only",
                                         "type": "boolean",
-                                        "default": false
+                                        "default": true
                                     },
                                     "key": {
                                         "$ref": "#/$defs/key"
@@ -1274,7 +1274,7 @@
                                     "defaultRouteOnly": {
                                         "description": "Show ips that are used for default routing only",
                                         "type": "boolean",
-                                        "default": false
+                                        "default": true
                                     },
                                     "key": {
                                         "$ref": "#/$defs/key"

src/data/help.txt

@@ -160,10 +160,10 @@ Module specific options:
     --localip-show-mac <?value>:             Show mac addresses in local ip module. Default is false
     --localip-show-loop <?value>:            Show loop back addresses (127.0.0.1) in local ip module. Default is false
     --localip-name-prefix <str>:             Show interfaces with given interface name prefix only. Default is empty
-    --localip-default-route-only <?value>:   Show the interface that is used for default routing only. Default is false
+    --localip-default-route-only <?value>:   Show the interface that is used for default routing only. Default is true
     --localip-compact <?value>:              Show all IPs in one line. Default is false
     --netio-name-prefix <str>:               Show interfaces with given name prefix only. Default is empty
-    --netio-default-route-only <?value>:     Show the interfac that is used for default routing only. Default is false
+    --netio-default-route-only <?value>:     Show the interfac that is used for default routing only. Default is true
     --publicip-timeout:                      Time in milliseconds to wait for the public ip server to respond. Default is disabled (0)
     --publicip-url:                          The URL of public IP detection server to be used.
     --weather-location:                      Set the location to be used. It must be URI encoded (eg a whitespace must be encoded as `+`).

src/modules/localip/localip.c

@@ -105,7 +105,7 @@ void ffPrintLocalIp(FFLocalIpOptions* options)
             if(options->moduleArgs.outputFormat.length == 0)
             {
                 ffPrintLogoAndKey(key.chars, 0, &options->moduleArgs, FF_PRINT_TYPE_NO_CUSTOM_KEY);
-                printIp(ip, !options->defaultRouteOnly);
+                printIp(ip, true);
                 putchar('\n');
             }
             else
@@ -370,7 +370,7 @@ void ffInitLocalIpOptions(FFLocalIpOptions* options)
 
     options->showType = FF_LOCALIP_TYPE_IPV4_BIT;
     ffStrbufInit(&options->namePrefix);
-    options->defaultRouteOnly = false;
+    options->defaultRouteOnly = true;
 }
 
 void ffDestroyLocalIpOptions(FFLocalIpOptions* options)

src/modules/netio/netio.c

@@ -230,7 +230,7 @@ void ffInitNetIOOptions(FFNetIOOptions* options)
     ffOptionInitModuleArg(&options->moduleArgs);
 
     ffStrbufInit(&options->namePrefix);
-    options->defaultRouteOnly = false;
+    options->defaultRouteOnly = true;
 }
 
 void ffDestroyNetIOOptions(FFNetIOOptions* options)