From 276edfd562251db4b6c600a55210e7b950ccdfd3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C4=99drzej=20Stuczy=C5=84ski?= Date: Fri, 31 Mar 2023 15:56:21 +0100 Subject: [PATCH] clippy --- .../network-requester/src/allowed_hosts/filter.rs | 6 +++--- .../network-requester/src/allowed_hosts/hosts.rs | 6 ++++-- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/service-providers/network-requester/src/allowed_hosts/filter.rs b/service-providers/network-requester/src/allowed_hosts/filter.rs index 8404bd5fb3..77dbc6c94b 100644 --- a/service-providers/network-requester/src/allowed_hosts/filter.rs +++ b/service-providers/network-requester/src/allowed_hosts/filter.rs @@ -67,7 +67,7 @@ impl OutboundRequestFilter { async fn check_standard_list(&self, host: &RequestHost) -> bool { let guard = self.standard_list.get().await; - self.check_group(&*guard, host) + self.check_group(&guard, host) } fn check_group(&self, group: &HostsGroup, host: &RequestHost) -> bool { @@ -106,7 +106,7 @@ impl OutboundRequestFilter { async fn check_request_host(&mut self, request_host: &RequestHost) -> bool { // first check our own allow list - let local_allowed = self.check_allowed_hosts(&request_host).await; + let local_allowed = self.check_allowed_hosts(request_host).await; // if it's locally allowed, no point in checking the standard list if local_allowed { @@ -114,7 +114,7 @@ impl OutboundRequestFilter { } // if that failed, check the standard list - self.check_standard_list(&request_host).await + self.check_standard_list(request_host).await } /// Returns `true` if a host's root domain is in the `allowed_hosts` list. diff --git a/service-providers/network-requester/src/allowed_hosts/hosts.rs b/service-providers/network-requester/src/allowed_hosts/hosts.rs index cef9893a1e..ec0b9c9be6 100644 --- a/service-providers/network-requester/src/allowed_hosts/hosts.rs +++ b/service-providers/network-requester/src/allowed_hosts/hosts.rs @@ -60,6 +60,7 @@ impl HostsStore { self.data.contains_ip_network(network) } + #[allow(unused)] pub(super) fn add_host>(&mut self, host: H) { match host.into() { Host::Domain(domain) => self.add_domain(&domain), @@ -67,6 +68,7 @@ impl HostsStore { } } + #[allow(unused)] pub(super) fn add_ipnet(&mut self, network: IpNetwork) { if !self.contains_ipnetwork(network) { self.data.add_ipnet(network); @@ -111,9 +113,9 @@ impl HostsStore { let parent_dir = file .parent() .expect("parent dir does not exist for {file:?}"); - fs::create_dir_all(&parent_dir) + fs::create_dir_all(parent_dir) .unwrap_or_else(|_| panic!("could not create storage directory at {parent_dir:?}")); - File::create(&file).unwrap(); + File::create(file).unwrap(); } }