Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

auth-server: Use admin auth to proxy relayer requests #52

Merged
merged 1 commit into from
Oct 23, 2024

auth-server: Use admin auth to proxy relayer requests

e1ace78
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

auth-server: Use admin auth to proxy relayer requests #52

auth-server: Use admin auth to proxy relayer requests
e1ace78
Select commit
Loading
Failed to load commit list.
GitHub Actions / clippy failed Oct 23, 2024 in 0s

clippy

2 errors

Details

Results

Message level Amount
Internal compiler error 0
Error 2
Warning 0
Note 0
Help 0

Versions

  • rustc 1.78.0-nightly (3cbb93223 2024-03-13)
  • cargo 1.78.0-nightly (7065f0ef4 2024-03-12)
  • clippy 0.1.78 (3cbb932 2024-03-13)

Annotations

Check failure on line 123 in price-reporter/src/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused `async` for function with no await statements

error: unused `async` for function with no await statements
   --> price-reporter/src/main.rs:94:1
    |
94  | / async fn init_default_price_streams(
95  | |     global_price_streams: &GlobalPriceStreams,
96  | |     config: ExchangeConnectionsConfig,
97  | | ) -> Result<(), ServerError> {
...   |
122 | |     Ok(())
123 | | }
    | |_^
    |
    = help: consider removing the `async` from this function
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_async
    = note: requested on the command line with `-D clippy::unused-async`

Check failure on line 87 in auth/auth-server/src/server/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

`std::result::Result<(), ApiError>` is not a future

error[E0277]: `std::result::Result<(), ApiError>` is not a future
  --> auth/auth-server/src/server/mod.rs:87:60
   |
87 |         self.admin_authenticate(path, &mut headers, &body).await?;
   |                                                           -^^^^^
   |                                                           ||
   |                                                           |`std::result::Result<(), ApiError>` is not a future
   |                                                           help: remove the `.await`
   |
   = help: the trait `warp::Future` is not implemented for `std::result::Result<(), ApiError>`, which is required by `std::result::Result<(), ApiError>: std::future::IntoFuture`
   = note: std::result::Result<(), ApiError> must be a future or must implement `IntoFuture` to be awaited
   = note: required for `std::result::Result<(), ApiError>` to implement `std::future::IntoFuture`