diff options
| author | Sven-Hendrik Haase <svenstaro@gmail.com> | 2019-05-02 23:04:07 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-05-02 23:04:07 +0000 |
| commit | c0ef2fbf569047185ea472702523d7c1b54c030a (patch) | |
| tree | 478ff421f74e21d26f8e18ec8f030f84ce57a183 /src | |
| parent | Add note about empty arrays (diff) | |
| parent | Merge branch 'new-clippy-experiment' into new-clippy (diff) | |
| download | miniserve-c0ef2fbf569047185ea472702523d7c1b54c030a.tar.gz miniserve-c0ef2fbf569047185ea472702523d7c1b54c030a.zip | |
Merge pull request #99 from KSXGitHub/new-clippy
A better Clippy integration
Diffstat (limited to 'src')
| -rw-r--r-- | src/auth.rs | 4 | ||||
| -rw-r--r-- | src/listing.rs | 1 | ||||
| -rw-r--r-- | src/main.rs | 4 | ||||
| -rw-r--r-- | src/renderer.rs | 1 |
4 files changed, 6 insertions, 4 deletions
diff --git a/src/auth.rs b/src/auth.rs index 8e6532b..e526923 100644 --- a/src/auth.rs +++ b/src/auth.rs @@ -75,8 +75,8 @@ pub fn match_auth(basic_auth: BasicAuthParams, required_auth: &RequiredAuth) -> } /// Return `true` if hashing of `password` by `T` algorithm equals to `hash` -pub fn compare_hash<T: Digest>(password: String, hash: &Vec<u8>) -> bool { - get_hash::<T>(password) == *hash +pub fn compare_hash<T: Digest>(password: String, hash: &[u8]) -> bool { + get_hash::<T>(password) == hash } /// Get hash of a `text` diff --git a/src/listing.rs b/src/listing.rs index a030feb..87fd8a8 100644 --- a/src/listing.rs +++ b/src/listing.rs @@ -123,6 +123,7 @@ pub fn file_handler(req: &HttpRequest<crate::MiniserveConfig>) -> Result<fs::Nam /// List a directory and renders a HTML file accordingly /// Adapted from https://docs.rs/actix-web/0.7.13/src/actix_web/fs.rs.html#564 +#[allow(clippy::identity_conversion)] pub fn directory_listing<S>( dir: &fs::Directory, req: &HttpRequest<S>, diff --git a/src/main.rs b/src/main.rs index c1bb0aa..ea58fc6 100644 --- a/src/main.rs +++ b/src/main.rs @@ -83,12 +83,12 @@ fn run() -> Result<(), ContextualError> { && miniserve_config .path .symlink_metadata() - .map_err(|e| { + .map_err(|e| ContextualError::IOError( "Failed to retrieve symlink's metadata".to_string(), e, ) - })? + )? .file_type() .is_symlink() { diff --git a/src/renderer.rs b/src/renderer.rs index b292e70..69224cf 100644 --- a/src/renderer.rs +++ b/src/renderer.rs @@ -9,6 +9,7 @@ use crate::listing::{Entry, SortingMethod, SortingOrder}; use crate::themes::ColorScheme; /// Renders the file listing +#[allow(clippy::too_many_arguments)] pub fn page( serve_path: &str, entries: Vec<Entry>, |