summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorLinus Färnstrand <linus@mullvad.net>2018-07-30 13:23:17 +0200
committerLinus Färnstrand <linus@mullvad.net>2018-07-30 13:23:17 +0200
commit6b794ac1b3454d08a8efea71e912b4e7cbc6c9ee (patch)
treecbd0ff3968aad4d897e5daf2c10bd46c34d03d12
parenta9270436ab85bc01eb3313d72ce231008676d7f0 (diff)
downloadmullvadvpn-6b794ac1b3454d08a8efea71e912b4e7cbc6c9ee.tar.xz
mullvadvpn-6b794ac1b3454d08a8efea71e912b4e7cbc6c9ee.zip
Reformat with rustfmt 0.9.0
-rw-r--r--mullvad-cli/src/cmds/account.rs6
-rw-r--r--mullvad-cli/src/cmds/auto_connect.rs3
-rw-r--r--mullvad-cli/src/cmds/lan.rs3
-rw-r--r--mullvad-cli/src/cmds/relay.rs27
-rw-r--r--mullvad-cli/src/cmds/tunnel.rs9
-rw-r--r--mullvad-daemon/src/main.rs3
-rw-r--r--mullvad-daemon/src/relays.rs3
-rw-r--r--mullvad-daemon/src/rpc_address_file.rs6
-rw-r--r--mullvad-ipc-client/src/lib.rs4
-rw-r--r--mullvad-problem-report/src/main.rs15
-rw-r--r--mullvad-rpc/src/rest.rs3
-rw-r--r--mullvad-types/src/custom_tunnel.rs3
-rw-r--r--talpid-core/src/firewall/linux/mod.rs3
-rw-r--r--talpid-core/src/firewall/macos/dns.rs3
-rw-r--r--talpid-core/src/firewall/macos/mod.rs7
-rw-r--r--talpid-ipc/src/client.rs21
-rw-r--r--talpid-ipc/src/lib.rs3
-rw-r--r--talpid-openvpn-plugin/src/processing.rs4
18 files changed, 52 insertions, 74 deletions
diff --git a/mullvad-cli/src/cmds/account.rs b/mullvad-cli/src/cmds/account.rs
index 5ad7822222..5757aac42d 100644
--- a/mullvad-cli/src/cmds/account.rs
+++ b/mullvad-cli/src/cmds/account.rs
@@ -23,12 +23,10 @@ impl Command for Account {
.help("The Mullvad account token to configure the client with")
.required(true),
),
- )
- .subcommand(
+ ).subcommand(
clap::SubCommand::with_name("get")
.about("Display information about the currently configured account"),
- )
- .subcommand(
+ ).subcommand(
clap::SubCommand::with_name("unset")
.about("Removes the account number from the settings"),
)
diff --git a/mullvad-cli/src/cmds/auto_connect.rs b/mullvad-cli/src/cmds/auto_connect.rs
index 57c125c4b6..f471fbde56 100644
--- a/mullvad-cli/src/cmds/auto_connect.rs
+++ b/mullvad-cli/src/cmds/auto_connect.rs
@@ -22,8 +22,7 @@ impl Command for AutoConnect {
.required(true)
.possible_values(&["on", "off"]),
),
- )
- .subcommand(
+ ).subcommand(
clap::SubCommand::with_name("get")
.about("Display the current auto-connect setting"),
)
diff --git a/mullvad-cli/src/cmds/lan.rs b/mullvad-cli/src/cmds/lan.rs
index 7f6b3a0444..bd6516f6bf 100644
--- a/mullvad-cli/src/cmds/lan.rs
+++ b/mullvad-cli/src/cmds/lan.rs
@@ -22,8 +22,7 @@ impl Command for Lan {
.required(true)
.possible_values(&["allow", "block"]),
),
- )
- .subcommand(
+ ).subcommand(
clap::SubCommand::with_name("get")
.about("Display the current local network sharing setting"),
)
diff --git a/mullvad-cli/src/cmds/relay.rs b/mullvad-cli/src/cmds/relay.rs
index 61f2e8d77e..2bdf84f40a 100644
--- a/mullvad-cli/src/cmds/relay.rs
+++ b/mullvad-cli/src/cmds/relay.rs
@@ -34,50 +34,42 @@ impl Command for Relay {
.required(true)
.index(1)
.possible_values(&["openvpn", "wireguard"]),
- )
- .arg(
+ ).arg(
clap::Arg::with_name("host")
.help("Hostname or IP")
.required(true)
.index(2),
- )
- .arg(
+ ).arg(
clap::Arg::with_name("port")
.help("Remote network port")
.required(true)
.index(3),
- )
- .arg(
+ ).arg(
clap::Arg::with_name("protocol")
.help("Transport protocol. For Wireguard this is ignored.")
.index(4)
.default_value("udp")
.possible_values(&["udp", "tcp"]),
),
- )
- .subcommand(
+ ).subcommand(
clap::SubCommand::with_name("location")
.about(
"Set country or city to select relays from. Use the 'list' \
command to show available alternatives.",
- )
- .arg(
+ ).arg(
clap::Arg::with_name("country")
.help(
"The two letter country code, or 'any' for no preference.",
- )
- .required(true)
+ ).required(true)
.index(1)
.validator(country_code_validator),
- )
- .arg(
+ ).arg(
clap::Arg::with_name("city")
.help("The three letter city code")
.index(2)
.validator(city_code_validator),
),
- )
- .subcommand(
+ ).subcommand(
clap::SubCommand::with_name("tunnel")
.about("Set tunnel constraints")
.arg(clap::Arg::with_name("port").required(true).index(1))
@@ -88,8 +80,7 @@ impl Command for Relay {
.possible_values(&["any", "udp", "tcp"]),
),
),
- )
- .subcommand(clap::SubCommand::with_name("get"))
+ ).subcommand(clap::SubCommand::with_name("get"))
.subcommand(
clap::SubCommand::with_name("list")
.setting(clap::AppSettings::SubcommandRequired)
diff --git a/mullvad-cli/src/cmds/tunnel.rs b/mullvad-cli/src/cmds/tunnel.rs
index 28af8d38ea..bf7172c655 100644
--- a/mullvad-cli/src/cmds/tunnel.rs
+++ b/mullvad-cli/src/cmds/tunnel.rs
@@ -27,13 +27,10 @@ impl Command for Tunnel {
.help(
"Sets the optional mssfix parameter. \
Set an empty string to clear it.",
- )
- .required(true),
+ ).required(true),
),
- )
- .setting(clap::AppSettings::SubcommandRequired),
- )
- .subcommand(
+ ).setting(clap::AppSettings::SubcommandRequired),
+ ).subcommand(
clap::SubCommand::with_name("get")
.help("Retrieves the current setting for mssfix"),
),
diff --git a/mullvad-daemon/src/main.rs b/mullvad-daemon/src/main.rs
index d9df085654..db322d8155 100644
--- a/mullvad-daemon/src/main.rs
+++ b/mullvad-daemon/src/main.rs
@@ -484,8 +484,7 @@ impl Daemon {
.join(
self.version_proxy
.is_app_version_supported(&current_version),
- )
- .map(|(latest_versions, is_supported)| AppVersionInfo {
+ ).map(|(latest_versions, is_supported)| AppVersionInfo {
current_is_supported: is_supported,
latest: latest_versions,
});
diff --git a/mullvad-daemon/src/relays.rs b/mullvad-daemon/src/relays.rs
index 44a2e67084..ef81adc4fa 100644
--- a/mullvad-daemon/src/relays.rs
+++ b/mullvad-daemon/src/relays.rs
@@ -315,8 +315,7 @@ impl RelaySelector {
.find(|relay| {
i = i.saturating_sub(relay.weight);
i == 0
- })
- .unwrap(),
+ }).unwrap(),
)
}
}
diff --git a/mullvad-daemon/src/rpc_address_file.rs b/mullvad-daemon/src/rpc_address_file.rs
index b3d3656a04..d0df2c1919 100644
--- a/mullvad-daemon/src/rpc_address_file.rs
+++ b/mullvad-daemon/src/rpc_address_file.rs
@@ -35,7 +35,8 @@ pub fn write(rpc_address: &str, shared_secret: &str) -> Result<()> {
// Avoids opening an existing file owned by another user and writing sensitive data to it.
remove()?;
- let file_path = mullvad_paths::get_rpc_address_path().chain_err(|| ErrorKind::UnknownFilePath)?;
+ let file_path =
+ mullvad_paths::get_rpc_address_path().chain_err(|| ErrorKind::UnknownFilePath)?;
if let Some(parent_dir) = file_path.parent() {
fs::create_dir_all(parent_dir)
@@ -52,7 +53,8 @@ pub fn write(rpc_address: &str, shared_secret: &str) -> Result<()> {
/// Removes the RPC file, if it exists.
pub fn remove() -> Result<()> {
- let file_path = mullvad_paths::get_rpc_address_path().chain_err(|| ErrorKind::UnknownFilePath)?;
+ let file_path =
+ mullvad_paths::get_rpc_address_path().chain_err(|| ErrorKind::UnknownFilePath)?;
if let Err(error) = fs::remove_file(&file_path) {
if error.kind() == io::ErrorKind::NotFound {
diff --git a/mullvad-ipc-client/src/lib.rs b/mullvad-ipc-client/src/lib.rs
index 51cba68eab..c4b4ebdd3f 100644
--- a/mullvad-ipc-client/src/lib.rs
+++ b/mullvad-ipc-client/src/lib.rs
@@ -117,8 +117,8 @@ impl DaemonRpcClient {
P: AsRef<Path>,
{
let file_path = file_path.as_ref();
- let rpc_file =
- File::open(file_path).chain_err(|| ErrorKind::ReadRpcFileError(file_path.to_owned()))?;
+ let rpc_file = File::open(file_path)
+ .chain_err(|| ErrorKind::ReadRpcFileError(file_path.to_owned()))?;
let reader = BufReader::new(rpc_file);
let mut lines = reader.lines();
diff --git a/mullvad-problem-report/src/main.rs b/mullvad-problem-report/src/main.rs
index 63c4c5af20..1d0e9f92f8 100644
--- a/mullvad-problem-report/src/main.rs
+++ b/mullvad-problem-report/src/main.rs
@@ -100,16 +100,14 @@ fn run() -> Result<()> {
.value_name("PATH")
.takes_value(true)
.required(true),
- )
- .arg(
+ ).arg(
clap::Arg::with_name("extra_logs")
.help("Paths to additional log files to be included.")
.multiple(true)
.value_name("EXTRA LOGS")
.takes_value(true)
.required(false),
- )
- .arg(
+ ).arg(
clap::Arg::with_name("redact")
.help("List of words and expressions to remove from the report")
.long("redact")
@@ -117,8 +115,7 @@ fn run() -> Result<()> {
.multiple(true)
.takes_value(true),
),
- )
- .subcommand(
+ ).subcommand(
clap::SubCommand::with_name("send")
.about("Send collected problem report")
.arg(
@@ -128,16 +125,14 @@ fn run() -> Result<()> {
.help("The path to previously collected report file.")
.takes_value(true)
.required(true),
- )
- .arg(
+ ).arg(
clap::Arg::with_name("email")
.long("email")
.short("e")
.help("Reporter's email")
.takes_value(true)
.required(false),
- )
- .arg(
+ ).arg(
clap::Arg::with_name("message")
.long("message")
.short("m")
diff --git a/mullvad-rpc/src/rest.rs b/mullvad-rpc/src/rest.rs
index 988dfba7c0..00e2563915 100644
--- a/mullvad-rpc/src/rest.rs
+++ b/mullvad-rpc/src/rest.rs
@@ -55,8 +55,7 @@ fn create_request_processing_future<CC: hyper::client::Connect>(
} else {
future::err(ErrorKind::HttpError(response.status()).into())
}
- })
- .and_then(|response: hyper::Response| response.body().concat2().from_err())
+ }).and_then(|response: hyper::Response| response.body().concat2().from_err())
.map(|response_chunk| response_chunk.to_vec())
.then(move |response_result| {
if let Err(_) = response_tx.send(response_result) {
diff --git a/mullvad-types/src/custom_tunnel.rs b/mullvad-types/src/custom_tunnel.rs
index 5542ef9e71..7b2f17ae67 100644
--- a/mullvad-types/src/custom_tunnel.rs
+++ b/mullvad-types/src/custom_tunnel.rs
@@ -41,6 +41,5 @@ fn resolve_to_ip(host: &str) -> Result<IpAddr> {
.or_else(|| {
info!("No IPv4 for host {}", host);
ipv6.pop()
- })
- .ok_or_else(|| ErrorKind::InvalidHost(host.to_owned()).into())
+ }).ok_or_else(|| ErrorKind::InvalidHost(host.to_owned()).into())
}
diff --git a/talpid-core/src/firewall/linux/mod.rs b/talpid-core/src/firewall/linux/mod.rs
index 67005516a3..f15617aba4 100644
--- a/talpid-core/src/firewall/linux/mod.rs
+++ b/talpid-core/src/firewall/linux/mod.rs
@@ -373,7 +373,8 @@ fn check_iface(rule: &mut Rule, direction: Direction, iface: &str) -> Result<()>
}
fn iface_index(name: &str) -> Result<libc::c_uint> {
- let c_name = CString::new(name).chain_err(|| ErrorKind::InvalidInterfaceName(name.to_owned()))?;
+ let c_name =
+ CString::new(name).chain_err(|| ErrorKind::InvalidInterfaceName(name.to_owned()))?;
let index = unsafe { libc::if_nametoindex(c_name.as_ptr()) };
if index == 0 {
let error = io::Error::last_os_error();
diff --git a/talpid-core/src/firewall/macos/dns.rs b/talpid-core/src/firewall/macos/dns.rs
index 2cde3cf8ff..5cadf3f58d 100644
--- a/talpid-core/src/firewall/macos/dns.rs
+++ b/talpid-core/src/firewall/macos/dns.rs
@@ -281,8 +281,7 @@ fn read_dns(store: &SCDynamicStore, path: CFString) -> Option<Vec<DnsServer>> {
dictionary
.find2(&CFString::from_static_string("ServerAddresses"))
.map(|array_ptr| unsafe { CFType::wrap_under_get_rule(array_ptr) })
- })
- .and_then(|addresses| {
+ }).and_then(|addresses| {
if let Some(array) = addresses.downcast::<CFArray<CFType>>() {
parse_cf_array_to_strings(array)
} else {
diff --git a/talpid-core/src/firewall/macos/mod.rs b/talpid-core/src/firewall/macos/mod.rs
index c4cb25f15b..0e6301c618 100644
--- a/talpid-core/src/firewall/macos/mod.rs
+++ b/talpid-core/src/firewall/macos/mod.rs
@@ -57,8 +57,8 @@ impl Firewall for PacketFilter {
self.restore_state(),
self.restore_dns(),
].into_iter()
- .collect::<Result<Vec<_>>>()
- .map(|_| ())
+ .collect::<Result<Vec<_>>>()
+ .map(|_| ())
}
}
@@ -200,8 +200,7 @@ impl PacketFilter {
let allow_multicast = rule_builder
.to(pfctl::Ip::from(ipnetwork_compat(IpNetwork::V4(
*super::MULTICAST_NET,
- ))))
- .build()?;
+ )))).build()?;
rules.push(allow_net);
rules.push(allow_multicast);
}
diff --git a/talpid-ipc/src/client.rs b/talpid-ipc/src/client.rs
index 9c22c5e7fa..63c95cc74f 100644
--- a/talpid-ipc/src/client.rs
+++ b/talpid-ipc/src/client.rs
@@ -257,11 +257,12 @@ impl Handler {
) -> Result<(SubscriptionId, JsonValue)> {
match notification.remove("params") {
Some(JsonValue::Object(mut parameters)) => {
- let raw_id = parameters
- .remove("subscription")
- .ok_or_else(|| ErrorKind::InvalidSubscriptionEvent("Missing subscription ID"))?;
- let id = SubscriptionId::parse_value(&raw_id)
- .ok_or_else(|| ErrorKind::InvalidSubscriptionEvent("Invalid subscription ID"))?;
+ let raw_id = parameters.remove("subscription").ok_or_else(|| {
+ ErrorKind::InvalidSubscriptionEvent("Missing subscription ID")
+ })?;
+ let id = SubscriptionId::parse_value(&raw_id).ok_or_else(|| {
+ ErrorKind::InvalidSubscriptionEvent("Invalid subscription ID")
+ })?;
let event = parameters
.remove("result")
.ok_or_else(|| ErrorKind::InvalidSubscriptionEvent("Missing event data"))?;
@@ -385,8 +386,7 @@ impl WsIpcClient {
id,
handler: Box::new(handler),
unsubscribe_method,
- })
- .chain_err(|| ErrorKind::ConnectionHandlerStopped)
+ }).chain_err(|| ErrorKind::ConnectionHandlerStopped)
}
pub fn call<S, T, O>(&mut self, method: S, params: &T) -> Result<O>
@@ -408,9 +408,12 @@ impl WsIpcClient {
.send(command)
.chain_err(|| ErrorKind::ConnectionHandlerStopped)?;
- let json_result = response_rx.recv().chain_err(|| ErrorKind::MissingResponse)?;
+ let json_result = response_rx
+ .recv()
+ .chain_err(|| ErrorKind::MissingResponse)?;
- Ok(serde_json::from_value(json_result?).chain_err(|| ErrorKind::DeserializeResponseError)?)
+ Ok(serde_json::from_value(json_result?)
+ .chain_err(|| ErrorKind::DeserializeResponseError)?)
}
}
diff --git a/talpid-ipc/src/lib.rs b/talpid-ipc/src/lib.rs
index 6a8a523bea..4c6c737165 100644
--- a/talpid-ipc/src/lib.rs
+++ b/talpid-ipc/src/lib.rs
@@ -67,8 +67,7 @@ impl IpcServer {
.map(|server| IpcServer {
address: format!("ws://{}", server.addr()),
server: server,
- })
- .chain_err(|| ErrorKind::IpcServerError)
+ }).chain_err(|| ErrorKind::IpcServerError)
}
/// Returns the localhost address this `IpcServer` is listening on.
diff --git a/talpid-openvpn-plugin/src/processing.rs b/talpid-openvpn-plugin/src/processing.rs
index 0d041939c4..99533945ed 100644
--- a/talpid-openvpn-plugin/src/processing.rs
+++ b/talpid-openvpn-plugin/src/processing.rs
@@ -25,8 +25,8 @@ pub struct EventProcessor {
impl EventProcessor {
pub fn new(arguments: &Arguments) -> Result<EventProcessor> {
trace!("Creating EventProcessor");
- let mut ipc_client =
- WsIpcClient::connect(&arguments.server_id).chain_err(|| "Unable to create IPC client")?;
+ let mut ipc_client = WsIpcClient::connect(&arguments.server_id)
+ .chain_err(|| "Unable to create IPC client")?;
trace!("Authenticating EventProcessor");
match ipc_client.call("authenticate", &[&arguments.credentials]) {