[INFO] fetching crate docker-registry2 0.0.2...
[INFO] checking docker-registry2-0.0.2 against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] extracting crate docker-registry2 0.0.2 into /workspace/builds/worker-5-tc2/source
[INFO] started tweaking crates.io crate docker-registry2 0.0.2
[INFO] finished tweaking crates.io crate docker-registry2 0.0.2
[INFO] tweaked toml for crates.io crate docker-registry2 0.0.2 written to /workspace/builds/worker-5-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate docker-registry2 0.0.2 on toolchain a5689b034ad287112f709693384a37f1377eb9cc
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate crates.io crate docker-registry2 0.0.2 already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:77db811e55d90add9212f6832c23229e3de5f1c1f9905e8cb07319ea15df73ac" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01
[INFO] running `Command { std: "docker" "start" "8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-w" "/opt/rustwide/workdir" "--user" "0:0" "8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling cc v1.2.57
[INFO] [stderr]    Compiling zerofrom-derive v0.1.6
[INFO] [stderr]    Compiling yoke-derive v0.8.1
[INFO] [stderr]    Compiling zerovec-derive v0.11.2
[INFO] [stderr]     Checking parking_lot_core v0.9.12
[INFO] [stderr]     Checking socket2 v0.6.3
[INFO] [stderr]     Checking mio v1.1.1
[INFO] [stderr]    Compiling tokio-macros v2.6.1
[INFO] [stderr]     Checking parking_lot v0.12.5
[INFO] [stderr]    Compiling openssl v0.10.76
[INFO] [stderr]     Checking futures-util v0.3.32
[INFO] [stderr]    Compiling num-traits v0.2.19
[INFO] [stderr]     Checking tracing v0.1.44
[INFO] [stderr]     Checking getrandom v0.2.17
[INFO] [stderr]    Compiling openssl-macros v0.1.1
[INFO] [stderr]     Checking tokio v1.50.0
[INFO] [stderr]     Checking bitflags v2.11.0
[INFO] [stderr]     Checking serde v1.0.228
[INFO] [stderr]    Compiling rust_decimal v1.40.0
[INFO] [stderr]    Compiling rustls v0.23.37
[INFO] [stderr]     Checking serde_json v1.0.149
[INFO] [stderr]     Checking arrayvec v0.7.6
[INFO] [stderr]     Checking zerofrom v0.1.6
[INFO] [stderr]     Checking winnow v0.7.15
[INFO] [stderr]     Checking yoke v0.8.1
[INFO] [stderr]    Compiling openssl-sys v0.9.112
[INFO] [stderr]    Compiling ring v0.17.14
[INFO] [stderr]     Checking zerovec v0.11.5
[INFO] [stderr]     Checking zerotrie v0.2.3
[INFO] [stderr]    Compiling native-tls v0.2.18
[INFO] [stderr]     Checking http-body-util v0.1.3
[INFO] [stderr]     Checking toml_datetime v1.0.0+spec-1.1.0
[INFO] [stderr]     Checking serde_spanned v1.0.4
[INFO] [stderr]     Checking tinystr v0.8.2
[INFO] [stderr]     Checking potential_utf v0.1.4
[INFO] [stderr]     Checking icu_collections v2.1.1
[INFO] [stderr]    Compiling rust_decimal_macros v1.40.0
[INFO] [stderr]     Checking icu_locale_core v2.1.1
[INFO] [stderr]     Checking toml_writer v1.0.6+spec-1.1.0
[INFO] [stderr]     Checking wae-types v0.0.2
[INFO] [stderr]     Checking getrandom v0.4.2
[INFO] [stderr]     Checking uuid v1.22.0
[INFO] [stderr]     Checking num_cpus v1.17.0
[INFO] [stderr]     Checking icu_provider v2.1.1
[INFO] [stderr]     Checking icu_properties v2.1.2
[INFO] [stderr]     Checking icu_normalizer v2.1.1
[INFO] [stderr]     Checking toml_parser v1.0.9+spec-1.1.0
[INFO] [stderr]     Checking futures-executor v0.3.32
[INFO] [stderr]     Checking toml v1.0.6+spec-1.1.0
[INFO] [stderr]     Checking futures v0.3.32
[INFO] [stderr]     Checking idna_adapter v1.2.1
[INFO] [stderr]     Checking idna v1.1.0
[INFO] [stderr]     Checking docker-types v0.0.2
[INFO] [stderr]     Checking url v2.5.8
[INFO] [stderr]     Checking tokio-util v0.7.18
[INFO] [stderr]     Checking rustls-webpki v0.103.9
[INFO] [stderr]     Checking tokio-native-tls v0.3.1
[INFO] [stderr]     Checking h2 v0.4.13
[INFO] [stderr]     Checking tokio-rustls v0.26.4
[INFO] [stderr]     Checking hyper v1.8.1
[INFO] [stderr]     Checking hyper-util v0.1.20
[INFO] [stderr]     Checking hyper-tls v0.6.0
[INFO] [stderr]     Checking wae-request v0.0.2
[INFO] [stderr]     Checking docker-registry2 v0.0.2 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `std::sync::Arc`
[INFO] [stdout]  --> src/client.rs:7:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | use std::sync::Arc;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `RequestBuilder`
[INFO] [stdout]   --> src/client.rs:10:63
[INFO] [stdout]    |
[INFO] [stdout] 10 | use wae_request::{HttpClient, HttpClientConfig, HttpResponse, RequestBuilder};
[INFO] [stdout]    |                                                               ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `self`
[INFO] [stdout]  --> src/downloader.rs:8:15
[INFO] [stdout]   |
[INFO] [stdout] 8 | use std::io::{self, Write};
[INFO] [stdout]   |               ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::time::Duration`
[INFO] [stdout]   --> src/downloader.rs:11:5
[INFO] [stdout]    |
[INFO] [stdout] 11 | use std::time::Duration;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Mutex`
[INFO] [stdout]   --> src/downloader.rs:14:19
[INFO] [stdout]    |
[INFO] [stdout] 14 | use tokio::sync::{Mutex, Semaphore};
[INFO] [stdout]    |                   ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `ImageManifest`
[INFO] [stdout]   --> src/downloader.rs:18:38
[INFO] [stdout]    |
[INFO] [stdout] 18 | use crate::types::{DownloadProgress, ImageManifest};
[INFO] [stdout]    |                                      ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::sync::Arc`
[INFO] [stdout]  --> src/client.rs:7:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | use std::sync::Arc;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `RequestBuilder`
[INFO] [stdout]   --> src/client.rs:10:63
[INFO] [stdout]    |
[INFO] [stdout] 10 | use wae_request::{HttpClient, HttpClientConfig, HttpResponse, RequestBuilder};
[INFO] [stdout]    |                                                               ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `self`
[INFO] [stdout]  --> src/downloader.rs:8:15
[INFO] [stdout]   |
[INFO] [stdout] 8 | use std::io::{self, Write};
[INFO] [stdout]   |               ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::time::Duration`
[INFO] [stdout]   --> src/downloader.rs:11:5
[INFO] [stdout]    |
[INFO] [stdout] 11 | use std::time::Duration;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Mutex`
[INFO] [stdout]   --> src/downloader.rs:14:19
[INFO] [stdout]    |
[INFO] [stdout] 14 | use tokio::sync::{Mutex, Semaphore};
[INFO] [stdout]    |                   ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `ImageManifest`
[INFO] [stdout]   --> src/downloader.rs:18:38
[INFO] [stdout]    |
[INFO] [stdout] 18 | use crate::types::{DownloadProgress, ImageManifest};
[INFO] [stdout]    |                                      ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `futures::StreamExt`
[INFO] [stdout]   --> src/downloader.rs:13:5
[INFO] [stdout]    |
[INFO] [stdout] 13 | use futures::StreamExt;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]   --> src/downloader.rs:53:27
[INFO] [stdout]    |
[INFO] [stdout] 53 |                 .map_err(|e| DockerError::storage_write_failed(&dir_path))?;
[INFO] [stdout]    |                           ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]   --> src/downloader.rs:46:23
[INFO] [stdout]    |
[INFO] [stdout] 46 |             .map_err(|e| DockerError::storage_write_failed(&storage_root))?;
[INFO] [stdout]    |                       ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:160:23
[INFO] [stdout]     |
[INFO] [stdout] 160 |             .map_err(|e| DockerError::storage_write_failed(&layer_file))?;
[INFO] [stdout]     |                       ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:152:27
[INFO] [stdout]     |
[INFO] [stdout] 152 |                 .map_err(|e| DockerError::storage_write_failed(&layer_file))?
[INFO] [stdout]     |                           ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:154:47
[INFO] [stdout]     |
[INFO] [stdout] 154 |             File::create(layer_path).map_err(|e| DockerError::storage_write_failed(&layer_file))?
[INFO] [stdout]     |                                               ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:129:27
[INFO] [stdout]     |
[INFO] [stdout] 129 |                 .map_err(|e| DockerError::storage_read_failed(&layer_file))?;
[INFO] [stdout]     |                           ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:197:31
[INFO] [stdout]     |
[INFO] [stdout] 197 |                     .map_err(|e| DockerError::storage_read_failed(&layer_file))?;
[INFO] [stdout]     |                               ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `futures::StreamExt`
[INFO] [stdout]   --> src/downloader.rs:13:5
[INFO] [stdout]    |
[INFO] [stdout] 13 | use futures::StreamExt;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]   --> src/downloader.rs:53:27
[INFO] [stdout]    |
[INFO] [stdout] 53 |                 .map_err(|e| DockerError::storage_write_failed(&dir_path))?;
[INFO] [stdout]    |                           ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]   --> src/downloader.rs:46:23
[INFO] [stdout]    |
[INFO] [stdout] 46 |             .map_err(|e| DockerError::storage_write_failed(&storage_root))?;
[INFO] [stdout]    |                       ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `endpoint`
[INFO] [stdout]   --> src/lib.rs:61:45
[INFO] [stdout]    |
[INFO] [stdout] 61 |     pub fn new(registry_type: RegistryType, endpoint: Option<&str>) -> Result<Self> {
[INFO] [stdout]    |                                             ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_endpoint`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:160:23
[INFO] [stdout]     |
[INFO] [stdout] 160 |             .map_err(|e| DockerError::storage_write_failed(&layer_file))?;
[INFO] [stdout]     |                       ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:152:27
[INFO] [stdout]     |
[INFO] [stdout] 152 |                 .map_err(|e| DockerError::storage_write_failed(&layer_file))?
[INFO] [stdout]     |                           ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:154:47
[INFO] [stdout]     |
[INFO] [stdout] 154 |             File::create(layer_path).map_err(|e| DockerError::storage_write_failed(&layer_file))?
[INFO] [stdout]     |                                               ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:129:27
[INFO] [stdout]     |
[INFO] [stdout] 129 |                 .map_err(|e| DockerError::storage_read_failed(&layer_file))?;
[INFO] [stdout]     |                           ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `e`
[INFO] [stdout]    --> src/downloader.rs:197:31
[INFO] [stdout]     |
[INFO] [stdout] 197 |                     .map_err(|e| DockerError::storage_read_failed(&layer_file))?;
[INFO] [stdout]     |                               ^ help: if this is intentional, prefix it with an underscore: `_e`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:123:32
[INFO] [stdout]     |
[INFO] [stdout] 123 |                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 123 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 123 +                     return Err(DockerError::request_error(&url, e.to_string()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 123 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 123 +                     return Err(<DockerError as Into<_>>::into(DockerError::request_error(&url, e.to_string())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:158:28
[INFO] [stdout]     |
[INFO] [stdout] 158 |                   return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ____________________________^
[INFO] [stdout] 159 | |                     "Failed to get manifest: {}",
[INFO] [stdout] 160 | |                     response.status
[INFO] [stdout] 161 | |                 ))
[INFO] [stdout] 162 | |                 .into());
[INFO] [stdout]     | |_______________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 161 -                 ))
[INFO] [stdout] 162 -                 .into());
[INFO] [stdout] 161 +                 )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 158 ~                 return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 159 |                     "Failed to get manifest: {}",
[INFO] [stdout] 160 |                     response.status
[INFO] [stdout] 161 ~                 ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:171:13
[INFO] [stdout]     |
[INFO] [stdout] 171 |           Err(DockerError::registry_error(
[INFO] [stdout]     |  _____________^
[INFO] [stdout] 172 | |             "Failed to get manifest after multiple attempts".to_string(),
[INFO] [stdout] 173 | |         )
[INFO] [stdout] 174 | |         .into())
[INFO] [stdout]     | |_______________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 173 -         )
[INFO] [stdout] 174 -         .into())
[INFO] [stdout] 173 +         ))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 171 ~         Err(<DockerError as Into<_>>::into(DockerError::registry_error(
[INFO] [stdout] 172 |             "Failed to get manifest after multiple attempts".to_string(),
[INFO] [stdout] 173 ~         )))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:190:17
[INFO] [stdout]     |
[INFO] [stdout] 190 |                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 190 -                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout] 190 +                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 190 -                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout] 190 +                 <DockerError as Into<_>>::into(DockerError::registry_error("Invalid WWW-Authenticate header".to_string())),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:215:17
[INFO] [stdout]     |
[INFO] [stdout] 215 |                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 215 -                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout] 215 +                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 215 -                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout] 215 +                 <DockerError as Into<_>>::into(DockerError::registry_error("Invalid WWW-Authenticate header".to_string())),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:234:24
[INFO] [stdout]     |
[INFO] [stdout] 234 |               return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 235 | |                 "Failed to get auth token: {}",
[INFO] [stdout] 236 | |                 response.status
[INFO] [stdout] 237 | |             ))
[INFO] [stdout] 238 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 237 -             ))
[INFO] [stdout] 238 -             .into());
[INFO] [stdout] 237 +             )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 234 ~             return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 235 |                 "Failed to get auth token: {}",
[INFO] [stdout] 236 |                 response.status
[INFO] [stdout] 237 ~             ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:267:32
[INFO] [stdout]     |
[INFO] [stdout] 267 |                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 267 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 267 +                     return Err(DockerError::request_error(&url, e.to_string()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 267 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 267 +                     return Err(<DockerError as Into<_>>::into(DockerError::request_error(&url, e.to_string())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:291:28
[INFO] [stdout]     |
[INFO] [stdout] 291 |                   return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ____________________________^
[INFO] [stdout] 292 | |                     "Failed to download layer: {}",
[INFO] [stdout] 293 | |                     response.status
[INFO] [stdout] 294 | |                 ))
[INFO] [stdout] 295 | |                 .into());
[INFO] [stdout]     | |_______________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 294 -                 ))
[INFO] [stdout] 295 -                 .into());
[INFO] [stdout] 294 +                 )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 291 ~                 return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 292 |                     "Failed to download layer: {}",
[INFO] [stdout] 293 |                     response.status
[INFO] [stdout] 294 ~                 ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:301:13
[INFO] [stdout]     |
[INFO] [stdout] 301 |           Err(DockerError::registry_error(
[INFO] [stdout]     |  _____________^
[INFO] [stdout] 302 | |             "Failed to download layer after multiple attempts".to_string(),
[INFO] [stdout] 303 | |         )
[INFO] [stdout] 304 | |         .into())
[INFO] [stdout]     | |_______________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 303 -         )
[INFO] [stdout] 304 -         .into())
[INFO] [stdout] 303 +         ))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 301 ~         Err(<DockerError as Into<_>>::into(DockerError::registry_error(
[INFO] [stdout] 302 |             "Failed to download layer after multiple attempts".to_string(),
[INFO] [stdout] 303 ~         )))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:340:32
[INFO] [stdout]     |
[INFO] [stdout] 340 |                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 340 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 340 +                     return Err(DockerError::request_error(&url, e.to_string()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 340 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 340 +                     return Err(<DockerError as Into<_>>::into(DockerError::request_error(&url, e.to_string())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:368:28
[INFO] [stdout]     |
[INFO] [stdout] 368 |                   return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ____________________________^
[INFO] [stdout] 369 | |                     "Failed to download layer: {}",
[INFO] [stdout] 370 | |                     response.status
[INFO] [stdout] 371 | |                 ))
[INFO] [stdout] 372 | |                 .into());
[INFO] [stdout]     | |_______________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 371 -                 ))
[INFO] [stdout] 372 -                 .into());
[INFO] [stdout] 371 +                 )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 368 ~                 return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 369 |                     "Failed to download layer: {}",
[INFO] [stdout] 370 |                     response.status
[INFO] [stdout] 371 ~                 ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:378:13
[INFO] [stdout]     |
[INFO] [stdout] 378 |           Err(DockerError::registry_error(
[INFO] [stdout]     |  _____________^
[INFO] [stdout] 379 | |             "Failed to download layer after multiple attempts".to_string(),
[INFO] [stdout] 380 | |         )
[INFO] [stdout] 381 | |         .into())
[INFO] [stdout]     | |_______________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 380 -         )
[INFO] [stdout] 381 -         .into())
[INFO] [stdout] 380 +         ))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 378 ~         Err(<DockerError as Into<_>>::into(DockerError::registry_error(
[INFO] [stdout] 379 |             "Failed to download layer after multiple attempts".to_string(),
[INFO] [stdout] 380 ~         )))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/downloader.rs:165:24
[INFO] [stdout]     |
[INFO] [stdout] 165 |               return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 166 | |                 "Download size mismatch: expected {}, got {}",
[INFO] [stdout] 167 | |                 size, downloaded
[INFO] [stdout] 168 | |             ))
[INFO] [stdout] 169 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 168 -             ))
[INFO] [stdout] 169 -             .into());
[INFO] [stdout] 168 +             )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 165 ~             return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 166 |                 "Download size mismatch: expected {}, got {}",
[INFO] [stdout] 167 |                 size, downloaded
[INFO] [stdout] 168 ~             ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/client.rs:27:5
[INFO] [stdout]    |
[INFO] [stdout] 27 |     async fn get_manifest(&self, image: &str, tag: &str) -> Result<ImageManifest>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout]    = note: `#[warn(async_fn_in_trait)]` on by default
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 27 -     async fn get_manifest(&self, image: &str, tag: &str) -> Result<ImageManifest>;
[INFO] [stdout] 27 +     fn get_manifest(&self, image: &str, tag: &str) -> impl std::future::Future<Output = Result<ImageManifest>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/client.rs:37:5
[INFO] [stdout]    |
[INFO] [stdout] 37 |     async fn download_layer(&self, image: &str, digest: &str) -> Result<HttpResponse>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 37 -     async fn download_layer(&self, image: &str, digest: &str) -> Result<HttpResponse>;
[INFO] [stdout] 37 +     fn download_layer(&self, image: &str, digest: &str) -> impl std::future::Future<Output = Result<HttpResponse>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/client.rs:48:5
[INFO] [stdout]    |
[INFO] [stdout] 48 |     async fn download_layer_with_range(
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 48 ~     fn download_layer_with_range(
[INFO] [stdout] 49 |         &self,
[INFO] [stdout] ...
[INFO] [stdout] 52 |         start: u64,
[INFO] [stdout] 53 ~     ) -> impl std::future::Future<Output = Result<HttpResponse>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `Result` that must be used
[INFO] [stdout]   --> src/downloader.rs:94:13
[INFO] [stdout]    |
[INFO] [stdout] 94 |             result.map_err(|e| DockerError::registry_error(e.to_string()))?;
[INFO] [stdout]    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]    = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]    |
[INFO] [stdout] 94 |             let _ = result.map_err(|e| DockerError::registry_error(e.to_string()))?;
[INFO] [stdout]    |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `docker-registry2` (lib test) due to 13 previous errors; 19 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: unused variable: `endpoint`
[INFO] [stdout]   --> src/lib.rs:61:45
[INFO] [stdout]    |
[INFO] [stdout] 61 |     pub fn new(registry_type: RegistryType, endpoint: Option<&str>) -> Result<Self> {
[INFO] [stdout]    |                                             ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_endpoint`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:123:32
[INFO] [stdout]     |
[INFO] [stdout] 123 |                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 123 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 123 +                     return Err(DockerError::request_error(&url, e.to_string()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 123 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 123 +                     return Err(<DockerError as Into<_>>::into(DockerError::request_error(&url, e.to_string())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:158:28
[INFO] [stdout]     |
[INFO] [stdout] 158 |                   return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ____________________________^
[INFO] [stdout] 159 | |                     "Failed to get manifest: {}",
[INFO] [stdout] 160 | |                     response.status
[INFO] [stdout] 161 | |                 ))
[INFO] [stdout] 162 | |                 .into());
[INFO] [stdout]     | |_______________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 161 -                 ))
[INFO] [stdout] 162 -                 .into());
[INFO] [stdout] 161 +                 )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 158 ~                 return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 159 |                     "Failed to get manifest: {}",
[INFO] [stdout] 160 |                     response.status
[INFO] [stdout] 161 ~                 ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:171:13
[INFO] [stdout]     |
[INFO] [stdout] 171 |           Err(DockerError::registry_error(
[INFO] [stdout]     |  _____________^
[INFO] [stdout] 172 | |             "Failed to get manifest after multiple attempts".to_string(),
[INFO] [stdout] 173 | |         )
[INFO] [stdout] 174 | |         .into())
[INFO] [stdout]     | |_______________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 173 -         )
[INFO] [stdout] 174 -         .into())
[INFO] [stdout] 173 +         ))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 171 ~         Err(<DockerError as Into<_>>::into(DockerError::registry_error(
[INFO] [stdout] 172 |             "Failed to get manifest after multiple attempts".to_string(),
[INFO] [stdout] 173 ~         )))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:190:17
[INFO] [stdout]     |
[INFO] [stdout] 190 |                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 190 -                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout] 190 +                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 190 -                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout] 190 +                 <DockerError as Into<_>>::into(DockerError::registry_error("Invalid WWW-Authenticate header".to_string())),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:215:17
[INFO] [stdout]     |
[INFO] [stdout] 215 |                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 215 -                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout] 215 +                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 215 -                 DockerError::registry_error("Invalid WWW-Authenticate header".to_string()).into(),
[INFO] [stdout] 215 +                 <DockerError as Into<_>>::into(DockerError::registry_error("Invalid WWW-Authenticate header".to_string())),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:234:24
[INFO] [stdout]     |
[INFO] [stdout] 234 |               return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 235 | |                 "Failed to get auth token: {}",
[INFO] [stdout] 236 | |                 response.status
[INFO] [stdout] 237 | |             ))
[INFO] [stdout] 238 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 237 -             ))
[INFO] [stdout] 238 -             .into());
[INFO] [stdout] 237 +             )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 234 ~             return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 235 |                 "Failed to get auth token: {}",
[INFO] [stdout] 236 |                 response.status
[INFO] [stdout] 237 ~             ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:267:32
[INFO] [stdout]     |
[INFO] [stdout] 267 |                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 267 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 267 +                     return Err(DockerError::request_error(&url, e.to_string()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 267 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 267 +                     return Err(<DockerError as Into<_>>::into(DockerError::request_error(&url, e.to_string())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:291:28
[INFO] [stdout]     |
[INFO] [stdout] 291 |                   return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ____________________________^
[INFO] [stdout] 292 | |                     "Failed to download layer: {}",
[INFO] [stdout] 293 | |                     response.status
[INFO] [stdout] 294 | |                 ))
[INFO] [stdout] 295 | |                 .into());
[INFO] [stdout]     | |_______________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 294 -                 ))
[INFO] [stdout] 295 -                 .into());
[INFO] [stdout] 294 +                 )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 291 ~                 return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 292 |                     "Failed to download layer: {}",
[INFO] [stdout] 293 |                     response.status
[INFO] [stdout] 294 ~                 ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:301:13
[INFO] [stdout]     |
[INFO] [stdout] 301 |           Err(DockerError::registry_error(
[INFO] [stdout]     |  _____________^
[INFO] [stdout] 302 | |             "Failed to download layer after multiple attempts".to_string(),
[INFO] [stdout] 303 | |         )
[INFO] [stdout] 304 | |         .into())
[INFO] [stdout]     | |_______________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 303 -         )
[INFO] [stdout] 304 -         .into())
[INFO] [stdout] 303 +         ))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 301 ~         Err(<DockerError as Into<_>>::into(DockerError::registry_error(
[INFO] [stdout] 302 |             "Failed to download layer after multiple attempts".to_string(),
[INFO] [stdout] 303 ~         )))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:340:32
[INFO] [stdout]     |
[INFO] [stdout] 340 |                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 340 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 340 +                     return Err(DockerError::request_error(&url, e.to_string()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 340 -                     return Err(DockerError::request_error(&url, e.to_string()).into());
[INFO] [stdout] 340 +                     return Err(<DockerError as Into<_>>::into(DockerError::request_error(&url, e.to_string())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:368:28
[INFO] [stdout]     |
[INFO] [stdout] 368 |                   return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ____________________________^
[INFO] [stdout] 369 | |                     "Failed to download layer: {}",
[INFO] [stdout] 370 | |                     response.status
[INFO] [stdout] 371 | |                 ))
[INFO] [stdout] 372 | |                 .into());
[INFO] [stdout]     | |_______________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 371 -                 ))
[INFO] [stdout] 372 -                 .into());
[INFO] [stdout] 371 +                 )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 368 ~                 return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 369 |                     "Failed to download layer: {}",
[INFO] [stdout] 370 |                     response.status
[INFO] [stdout] 371 ~                 ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/client.rs:378:13
[INFO] [stdout]     |
[INFO] [stdout] 378 |           Err(DockerError::registry_error(
[INFO] [stdout]     |  _____________^
[INFO] [stdout] 379 | |             "Failed to download layer after multiple attempts".to_string(),
[INFO] [stdout] 380 | |         )
[INFO] [stdout] 381 | |         .into())
[INFO] [stdout]     | |_______________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 380 -         )
[INFO] [stdout] 381 -         .into())
[INFO] [stdout] 380 +         ))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 378 ~         Err(<DockerError as Into<_>>::into(DockerError::registry_error(
[INFO] [stdout] 379 |             "Failed to download layer after multiple attempts".to_string(),
[INFO] [stdout] 380 ~         )))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `DockerError`
[INFO] [stdout]    --> src/downloader.rs:165:24
[INFO] [stdout]     |
[INFO] [stdout] 165 |               return Err(DockerError::registry_error(format!(
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 166 | |                 "Download size mismatch: expected {}, got {}",
[INFO] [stdout] 167 | |                 size, downloaded
[INFO] [stdout] 168 | |             ))
[INFO] [stdout] 169 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 168 -             ))
[INFO] [stdout] 169 -             .into());
[INFO] [stdout] 168 +             )));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 165 ~             return Err(<DockerError as Into<_>>::into(DockerError::registry_error(format!(
[INFO] [stdout] 166 |                 "Download size mismatch: expected {}, got {}",
[INFO] [stdout] 167 |                 size, downloaded
[INFO] [stdout] 168 ~             ))));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/client.rs:27:5
[INFO] [stdout]    |
[INFO] [stdout] 27 |     async fn get_manifest(&self, image: &str, tag: &str) -> Result<ImageManifest>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout]    = note: `#[warn(async_fn_in_trait)]` on by default
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 27 -     async fn get_manifest(&self, image: &str, tag: &str) -> Result<ImageManifest>;
[INFO] [stdout] 27 +     fn get_manifest(&self, image: &str, tag: &str) -> impl std::future::Future<Output = Result<ImageManifest>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/client.rs:37:5
[INFO] [stdout]    |
[INFO] [stdout] 37 |     async fn download_layer(&self, image: &str, digest: &str) -> Result<HttpResponse>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 37 -     async fn download_layer(&self, image: &str, digest: &str) -> Result<HttpResponse>;
[INFO] [stdout] 37 +     fn download_layer(&self, image: &str, digest: &str) -> impl std::future::Future<Output = Result<HttpResponse>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/client.rs:48:5
[INFO] [stdout]    |
[INFO] [stdout] 48 |     async fn download_layer_with_range(
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 48 ~     fn download_layer_with_range(
[INFO] [stdout] 49 |         &self,
[INFO] [stdout] ...
[INFO] [stdout] 52 |         start: u64,
[INFO] [stdout] 53 ~     ) -> impl std::future::Future<Output = Result<HttpResponse>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `Result` that must be used
[INFO] [stdout]   --> src/downloader.rs:94:13
[INFO] [stdout]    |
[INFO] [stdout] 94 |             result.map_err(|e| DockerError::registry_error(e.to_string()))?;
[INFO] [stdout]    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]    = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]    |
[INFO] [stdout] 94 |             let _ = result.map_err(|e| DockerError::registry_error(e.to_string()))?;
[INFO] [stdout]    |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `docker-registry2` (lib) due to 13 previous errors; 19 warnings emitted
[INFO] [stderr] warning: unused dependency `sha2`
[INFO] [stderr]   --> Cargo.toml:17:15
[INFO] [stderr]    |
[INFO] [stderr] 17 | [dependencies.sha2]
[INFO] [stderr]    |               ^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::unused_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the dependency on `sha2`
[INFO] [stderr] warning: `docker-registry2` (manifest) generated 1 warning
[INFO] running `Command { std: "docker" "inspect" "8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01", kill_on_drop: false }`
[INFO] [stdout] 8c14adef2007712b7bd48784af923943e5af69bab1f0f52de6d1c4a7cd930c01
