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

Ktra openid doesn't install via cargo #58

Open
davidv1992 opened this issue Jan 27, 2023 · 2 comments
Open

Ktra openid doesn't install via cargo #58

davidv1992 opened this issue Jan 27, 2023 · 2 comments

Comments

@davidv1992
Copy link

When installing Ktra 0.7.0 with openid-connect support via cargo install ktra --no-default-features --features=openid, compilation fails due to several missing crates:

error[E0432]: unresolved import `argon2`
 --> /home/david/.cargo/registry/src/github.com-1ecc6299db9ec823/ktra-0.7.0/src/db_manager/sled_db_manager.rs:6:14
  |
6 | use argon2::{self, hash_encoded, verify_encoded};
  |              ^^^^ no external crate `argon2`

error[E0433]: failed to resolve: use of undeclared crate or module `rand`
 --> /home/david/.cargo/registry/src/github.com-1ecc6299db9ec823/ktra-0.7.0/src/utils.rs:7:5
  |
7 | use rand::prelude::*;
  |     ^^^^ use of undeclared crate or module `rand`

error[E0433]: failed to resolve: use of undeclared crate or module `rand`
 --> /home/david/.cargo/registry/src/github.com-1ecc6299db9ec823/ktra-0.7.0/src/utils.rs:6:5
  |
6 | use rand::distributions::Alphanumeric;
  |     ^^^^ use of undeclared crate or module `rand`

error[E0432]: unresolved imports `argon2`, `rand::distributions::Alphanumeric`
 --> /home/david/.cargo/registry/src/github.com-1ecc6299db9ec823/ktra-0.7.0/src/db_manager/utils.rs:3:14
  |
3 | use argon2::{self, ThreadMode, Variant};
  |              ^^^^ no external crate `argon2`
  |
 ::: /home/david/.cargo/registry/src/github.com-1ecc6299db9ec823/ktra-0.7.0/src/utils.rs:6:5
  |
6 | use rand::distributions::Alphanumeric;
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0433]: failed to resolve: use of undeclared crate or module `argon2`
  --> /home/david/.cargo/registry/src/github.com-1ecc6299db9ec823/ktra-0.7.0/src/error.rs:41:12
   |
41 |     Argon2(argon2::Error),
   |            ^^^^^^ use of undeclared crate or module `argon2`

error[E0433]: failed to resolve: use of undeclared crate or module `rand`
  --> /home/david/.cargo/registry/src/github.com-1ecc6299db9ec823/ktra-0.7.0/src/utils.rs:31:9
   |
31 |         rand::thread_rng()
   |         ^^^^ use of undeclared crate or module `rand`
@davidv1992
Copy link
Author

The current state of #57 already fixes this. Perhaps consider merging this early and doing the final database refactors separately?

@gagbo
Copy link
Collaborator

gagbo commented Jan 27, 2023

yeah, the feature set is in a weird state. You'll be able to get it working by adding secure-auth with openid normally, as that's what our CI does (and add crates-io-mirroring if that still fails but I don't think this one is mandatory)

I can't really give an ETA on #57 landing, so the workaround of having an extra feature is probably the quickest workaround now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants