update cli flags

codex/conf.nim

@@ -37,9 +37,10 @@ import ./logutils
 import ./stores
 import ./units
 import ./utils
+import ./nat
 from ./validationconfig import MaxSlots, ValidationGroups
 
-export units, net, codextypes, logutils
+export units, net, codextypes, logutils,completeCmdArg,parseCmdArg,NatConfig
 export ValidationGroups, MaxSlots
 
 export
@@ -139,19 +140,13 @@ type
       name: "listen-addrs" }: seq[MultiAddress]
 
     # TODO: change this once we integrate nat support
+  
     nat* {.
-      desc: "IP Addresses to announce behind a NAT"
+      desc: "Specify method to use for determining public address. " &
-      defaultValue: ValidIpAddress.init("127.0.0.1")
+            "Must be one of: any, none, upnp, pmp, extip:<IP>"
-      defaultValueDesc: "127.0.0.1"
+      defaultValue: NatConfig(hasExtIp: false, nat: NatAny)
-      abbr: "a"
+      defaultValueDesc: "any"
-      name: "nat" }: ValidIpAddress
+      name: "nat" }: NatConfig
-
-    discoveryIp* {.
-      desc: "Discovery listen address"
-      defaultValue: ValidIpAddress.init(IPv4_any())
-      defaultValueDesc: "0.0.0.0"
-      abbr: "e"
-      name: "disc-ip" }: ValidIpAddress
 
     discoveryPort* {.
       desc: "Discovery (UDP) port"
@@ -527,6 +522,12 @@ proc readValue*(r: var TomlReader, val: var Duration)
     quit QuitFailure
   val = dur
 
+proc readValue*(r: var TomlReader, val: var NatConfig)
+               {.raises: [SerializationError].} =
+  val = try: parseCmdArg(NatConfig, r.readValue(string))
+        except CatchableError as err:
+          raise newException(SerializationError, err.msg)
+
 # no idea why confutils needs this:
 proc completeCmdArg*(T: type EthAddress; val: string): seq[string] =
   discard