P2P replace DNS panic with map_err (#3383)

Replace panic on DNS resolution error with map_err to bubble up error to the caller Co-authored-by: Nym Seddon <[email protected]>
mimblewimble · Jul 17, 2020 · d78980c · d78980c
1 parent 824c51a
commit d78980c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/p2p/src/types.rs b/p2p/src/types.rs
@@ -181,9 +181,9 @@ impl<'de> Visitor<'de> for PeerAddrs {
 				Ok(ip) => peers.push(PeerAddr(ip)),
 				// If that fails it's probably a DNS record
 				Err(_) => {
-					let socket_addrs = entry
-						.to_socket_addrs()
-						.unwrap_or_else(|_| panic!("Unable to resolve DNS: {}", entry));
+					let socket_addrs = entry.to_socket_addrs().map_err(|_| {
+						serde::de::Error::custom(format!("Unable to resolve DNS: {}", entry))
+					})?;
 					peers.append(&mut socket_addrs.map(PeerAddr).collect());
 				}
 			}