status-im/nimbus-eth1
2
0
Fork 0
mirror of https://github.com/status-im/nimbus-eth1.git synced 2025-02-04 00:05:22 +00:00
Code Issues Projects Releases Wiki Activity

Bump confutils: no more exception leak

Browse Source
This commit is contained in:
jangko 2024-02-12 13:01:05 +07:00
parent cc4d13835f
commit d8ec962b6a
No known key found for this signature in database
GPG Key ID: 31702AE10541E6B9
2 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
nimbus/config.nim
View File

@ -643,7 +643,7 @@ proc getProtocolFlags*(conf: NimbusConf): set[ProtocolFlag] =
proc getRpcFlags(api: openArray[string]): set[RpcFlag] =
if api.len == 0:
return {RpcFlag.Eth}
for item in repeatingList(api):
case item.toLowerAscii()
of "eth": result.incl RpcFlag.Eth
@ -747,7 +747,7 @@ func shareServerWithEngineApi*(conf: NimbusConf): bool =
func httpServerEnabled*(conf: NimbusConf): bool =
conf.graphqlEnabled or
conf.wsEnabled or
conf.rpcEnabled
conf.rpcEnabled
# KLUDGE: The `load()` template does currently not work within any exception
# annotated environment.
@ -768,9 +768,6 @@ proc makeConfig*(cmdLine = commandLineParams()): NimbusConf
{.pop.}
except CatchableError as e:
raise e
except Exception as e:
{.warning: "Kludge(BareExcept): `load()` template in vendor package needs to be updated"}
raiseAssert "Ooops makeConfig(): name=" & $e.name & " msg=" & e.msg
var networkId = result.getNetworkId()

2
vendor/nim-confutils vendored

@ -1 +1 @@
Subproject commit 7568f1b7c3142d8e87c1f3dd42924238926affbe
Subproject commit 57ff0b8555f8c56425e87ca5de1a96a209631b2e