logos-messaging/logos-messaging-nim
1
0
Fork 0
mirror of https://github.com/logos-messaging/logos-messaging-nim.git synced 2026-01-04 06:53:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

Adding more detail in an error produced by 'parsePeerInfo' (#1687)

Browse Source 
* Adding more detail in an error produced by 'parsePeerInfo'

* utils/peers.nim: casting to 'string' in a more idiomatic/convenient way
This commit is contained in:
Ivan Folgueira Bande 2023-04-19 14:02:54 +02:00 committed by GitHub
parent e2dc11912c
commit 4655a9880f
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
waku/v2/utils/peers.nim
View File

@ -119,7 +119,7 @@ proc parsePeerInfo*(peer: RemotePeerInfo|string):
if peer is RemotePeerInfo:
return ok(cast[RemotePeerInfo](peer))
let multiAddr = ? MultiAddress.init(cast[string](peer))
let multiAddr = ? MultiAddress.init($peer)
.mapErr(proc(err: string):
string = "MultiAddress.init [" & err & "]")
@ -136,12 +136,12 @@ proc parsePeerInfo*(peer: RemotePeerInfo|string):
let p2pPartStr = p2pPart.toString()[]
if not p2pPartStr.contains("/"):
let msg = "Error in parsePeerInfo: p2p part should contain / [" &
p2pPartStr & "]"
let msg = "Error in parsePeerInfo: p2p part should contain / [p2pPartStr:" &
p2pPartStr & "] [peer:" & $peer & "]"
return err(msg)
let peerId = ? PeerID.init(p2pPartStr.split("/")[^1])
.mapErr(proc (e:cstring):string = cast[string](e))
.mapErr(proc (e:cstring):string = $e)
if not wireAddr.validWireAddr():
return err("Error in parsePeerInfo: Invalid node multiaddress")