From 4b30a34bb8c34c4c861aca780b73b07d1a02d84f Mon Sep 17 00:00:00 2001 From: Colton Hurst Date: Thu, 26 Sep 2024 12:42:15 -0400 Subject: [PATCH] Revert "[PM-12431] Temporarily un-export bitwarden_fido (#1072)" This reverts commit 73d3ce83cebb68e156955fe6eee8e98b13a5a0bd. --- .github/workflows/publish-rust-crates.yml | 2 +- Cargo.lock | 1 + crates/bitwarden/Cargo.toml | 6 ++---- crates/bitwarden/src/error.rs | 4 +--- crates/bitwarden/src/lib.rs | 16 ++++------------ 5 files changed, 9 insertions(+), 20 deletions(-) diff --git a/.github/workflows/publish-rust-crates.yml b/.github/workflows/publish-rust-crates.yml index ec9848127..bdb39d9b7 100644 --- a/.github/workflows/publish-rust-crates.yml +++ b/.github/workflows/publish-rust-crates.yml @@ -72,7 +72,7 @@ jobs: env: PUBLISH_GRACE_SLEEP: 10 CARGO_REGISTRY_TOKEN: ${{ steps.retrieve-secrets.outputs.cratesio-api-token }} - run: cargo-release release publish --exclude bw --exclude bws --exclude bitwarden-fido --execute --no-confirm + run: cargo-release release publish --exclude bw --exclude bws --execute --no-confirm - name: Update deployment status to Success if: ${{ inputs.release_type != 'Dry Run' && success() }} diff --git a/Cargo.lock b/Cargo.lock index 9bfdbe70c..a1a9bcd23 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -367,6 +367,7 @@ dependencies = [ "bitwarden-core", "bitwarden-crypto", "bitwarden-exporters", + "bitwarden-fido", "bitwarden-generators", "bitwarden-send", "bitwarden-sm", diff --git a/crates/bitwarden/Cargo.toml b/crates/bitwarden/Cargo.toml index 31d84b877..f927564cf 100644 --- a/crates/bitwarden/Cargo.toml +++ b/crates/bitwarden/Cargo.toml @@ -30,8 +30,7 @@ uniffi = [ "bitwarden-core/uniffi", "bitwarden-crypto/uniffi", "bitwarden-exporters/uniffi", - # PR #1072 - # "bitwarden-fido/uniffi", + "bitwarden-fido/uniffi", "bitwarden-generators/uniffi", "bitwarden-send/uniffi", "bitwarden-vault/uniffi", @@ -44,8 +43,7 @@ bitwarden-api-identity = { workspace = true } bitwarden-core = { workspace = true } bitwarden-crypto = { workspace = true } bitwarden-exporters = { workspace = true, optional = true } -# PR #1072 -# bitwarden-fido = { workspace = true, optional = true } +bitwarden-fido = { workspace = true, optional = true } bitwarden-generators = { workspace = true, optional = true } bitwarden-send = { workspace = true, optional = true } bitwarden-sm = { workspace = true, optional = true } diff --git a/crates/bitwarden/src/error.rs b/crates/bitwarden/src/error.rs index a6f6c09de..06ef09648 100644 --- a/crates/bitwarden/src/error.rs +++ b/crates/bitwarden/src/error.rs @@ -37,8 +37,6 @@ pub enum Error { ExportError(#[from] ExportError), // Fido - /* - # PR #1072 #[cfg(all(feature = "uniffi", feature = "internal"))] #[error(transparent)] MakeCredential(#[from] bitwarden_fido::MakeCredentialError), @@ -59,7 +57,7 @@ pub enum Error { #[cfg(all(feature = "uniffi", feature = "internal"))] #[error(transparent)] Fido2Client(#[from] bitwarden_fido::Fido2ClientError), - */ + #[error("Internal error: {0}")] Internal(Cow<'static, str>), } diff --git a/crates/bitwarden/src/lib.rs b/crates/bitwarden/src/lib.rs index 5751c313b..c87693ce7 100644 --- a/crates/bitwarden/src/lib.rs +++ b/crates/bitwarden/src/lib.rs @@ -79,18 +79,10 @@ pub mod internal { pub use bitwarden_vault::*; } - /* - PM-12431 - # PR #1072 - - Temporarily un-export bitwarden_fido, as bitwarden_fido - is not ready for release yet. This will unblock the - 1.0 releases. - */ - // #[cfg(feature = "uniffi")] - // pub mod fido { - // pub use bitwarden_fido::*; - // } + #[cfg(feature = "uniffi")] + pub mod fido { + pub use bitwarden_fido::*; + } } #[cfg(feature = "internal")] pub use internal::*;