Remove json schema when it's not used (#253)

We don't use json schema for the password manager sdk. Let's remove it
and if we ever need it we can add it back.

Author: Hinton

Cargo.lock

@@ -1,15 +1,14 @@
 use std::collections::HashMap;
 
 use bitwarden_api_api::models::PolicyResponseModel;
-use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 use serde_repr::{Deserialize_repr, Serialize_repr};
 use uuid::Uuid;
 
 use crate::{require, MissingFieldError};
 
 /// Represents a policy that can be applied to an organization.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 pub struct Policy {
     id: Uuid,
     organization_id: Uuid,
@@ -18,7 +17,7 @@ pub struct Policy {
     enabled: bool,
 }
 
-#[derive(Serialize_repr, Deserialize_repr, Debug, JsonSchema)]
+#[derive(Serialize_repr, Deserialize_repr, Debug)]
 #[repr(u8)]
 pub enum PolicyType {
     /// Requires users to have 2fa enabled

crates/bitwarden-core/src/admin_console/policy.rs

@@ -1,5 +1,3 @@
-use schemars::JsonSchema;
-
 /// Validate the provided password passes the provided Master Password Requirements Policy.
 pub(crate) fn satisfies_policy(
     password: String,
@@ -33,7 +31,7 @@ pub(crate) fn satisfies_policy(
     true
 }
 
-#[derive(Debug, JsonSchema)]
+#[derive(Debug)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 #[allow(dead_code)]
 pub struct MasterPasswordPolicyOptions {

crates/bitwarden-core/src/auth/password/policy.rs

@@ -5,13 +5,12 @@ use bitwarden_api_identity::{
 use bitwarden_crypto::{
     default_pbkdf2_iterations, CryptoError, HashPurpose, Kdf, MasterKey, RsaKeyPair,
 };
-use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 use thiserror::Error;
 
 use crate::{ApiError, Client};
 
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 pub struct RegisterRequest {
     pub email: String,

crates/bitwarden-core/src/auth/register.rs

@@ -11,7 +11,6 @@ use bitwarden_crypto::{
     AsymmetricCryptoKey, CryptoError, EncString, Kdf, KeyDecryptable, KeyEncryptable, MasterKey,
     SymmetricCryptoKey, UnsignedSharedKey, UserKey,
 };
-use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 #[cfg(feature = "wasm")]
 use {tsify_next::Tsify, wasm_bindgen::prelude::*};
@@ -35,7 +34,7 @@ pub enum MobileCryptoError {
 }
 
 /// State used for initializing the user cryptographic state.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 #[cfg_attr(feature = "wasm", derive(Tsify), tsify(into_wasm_abi, from_wasm_abi))]
@@ -51,7 +50,7 @@ pub struct InitUserCryptoRequest {
 }
 
 /// The crypto method used to initialize the user cryptographic state.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Enum))]
 #[cfg_attr(feature = "wasm", derive(Tsify), tsify(into_wasm_abi, from_wasm_abi))]
@@ -102,7 +101,7 @@ pub enum InitUserCryptoMethod {
 }
 
 /// Auth requests supports multiple initialization methods.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Enum))]
 #[cfg_attr(feature = "wasm", derive(Tsify), tsify(into_wasm_abi, from_wasm_abi))]
@@ -222,7 +221,7 @@ pub async fn initialize_user_crypto(
 }
 
 /// Represents the request to initialize the user's organizational cryptographic state.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 #[cfg_attr(feature = "wasm", derive(Tsify), tsify(into_wasm_abi, from_wasm_abi))]
@@ -252,7 +251,7 @@ pub(super) async fn get_user_encryption_key(client: &Client) -> Result<String, M
 }
 
 /// Response from the `update_password` function
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 pub struct UpdatePasswordResponse {
@@ -301,7 +300,7 @@ pub(super) fn update_password(
 }
 
 /// Request for deriving a pin protected user key
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 pub struct DerivePinKeyResponse {
@@ -438,7 +437,7 @@ pub(super) fn derive_key_connector(
 }
 
 /// Response from the `make_key_pair` function
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 #[cfg_attr(feature = "wasm", derive(Tsify), tsify(into_wasm_abi, from_wasm_abi))]
@@ -461,7 +460,7 @@ pub(super) fn make_key_pair(user_key: String) -> Result<MakeKeyPairResponse, Cry
 }
 
 /// Request for `verify_asymmetric_keys`.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 #[cfg_attr(feature = "wasm", derive(Tsify), tsify(into_wasm_abi, from_wasm_abi))]
@@ -475,7 +474,7 @@ pub struct VerifyAsymmetricKeysRequest {
 }
 
 /// Response for `verify_asymmetric_keys`.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 #[cfg_attr(feature = "wasm", derive(Tsify), tsify(into_wasm_abi, from_wasm_abi))]

crates/bitwarden-core/src/mobile/crypto.rs

@@ -4,14 +4,13 @@
 
 use base64::{engine::general_purpose::STANDARD, Engine};
 use bitwarden_crypto::fingerprint;
-use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 use thiserror::Error;
 
 use crate::{key_management::AsymmetricKeyId, MissingPrivateKeyError, VaultLockedError};
 
 /// Request to generate a fingerprint.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Record))]
 pub struct FingerprintRequest {
@@ -24,7 +23,7 @@ pub struct FingerprintRequest {
 /// Response containing a generated fingerprint.
 ///
 /// TODO: We should attempt to remove this and just return a string.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 pub struct FingerprintResponse {
     /// The generated fingerprint.

crates/bitwarden-core/src/platform/generate_fingerprint.rs

@@ -20,7 +20,6 @@ use bitwarden_api_api::{
 };
 use bitwarden_crypto::{HashPurpose, MasterKey};
 use log::{debug, info};
-use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 use thiserror::Error;
 
@@ -101,7 +100,7 @@ fn build_secret_verification_request(
 }
 
 /// The response from the server when requesting the user's API key.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 pub struct UserApiKeyResponse {
     /// The user's API key, which represents the client_secret portion of an oauth request.

crates/bitwarden-core/src/platform/get_user_api_key.rs

@@ -1,8 +1,7 @@
-use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 
 /// Request to verify a user's secret.
-#[derive(Serialize, Deserialize, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Debug)]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 pub struct SecretVerificationRequest {
     /// The user's master password to use for user verification. If supplied, this will be used for

crates/bitwarden-core/src/platform/secret_verification_request.rs

@@ -3,7 +3,6 @@ use std::pin::Pin;
 use base64::{engine::general_purpose::STANDARD, Engine};
 use generic_array::{typenum::U32, GenericArray};
 use rand::Rng;
-use schemars::JsonSchema;
 use zeroize::{Zeroize, Zeroizing};
 
 use super::{
@@ -15,7 +14,7 @@ use crate::{
     CryptoError, EncString, KeyDecryptable, Result, SymmetricCryptoKey, UserKey,
 };
 
-#[derive(Copy, Clone, JsonSchema)]
+#[derive(Copy, Clone)]
 #[cfg_attr(feature = "uniffi", derive(uniffi::Enum))]
 pub enum HashPurpose {
     ServerAuthorization = 1,

crates/bitwarden-crypto/src/keys/master_key.rs

@@ -30,7 +30,6 @@ chrono = { workspace = true, features = ["std"] }
 credential-exchange-format = ">=0.1, <0.2"
 csv = "1.3.0"
 num-traits = ">=0.2, <0.3"
-schemars = { workspace = true }
 serde = { workspace = true }
 serde_json = { workspace = true }
 thiserror = { workspace = true }