[PM-3807] feat: remove standalone Fido2Key

bitwarden · Sep 11, 2023 · cfbbe10 · cfbbe10
1 parent 8a36bb7
commit cfbbe10
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 68 deletions.
diff --git a/src/Api/Vault/Controllers/SyncController.cs b/src/Api/Vault/Controllers/SyncController.cs
@@ -1,6 +1,4 @@
 using Bit.Api.Vault.Models.Response;
-using Bit.Core;
-using Bit.Core.Context;
 using Bit.Core.Entities;
 using Bit.Core.Enums;
 using Bit.Core.Enums.Provider;
@@ -10,7 +8,6 @@
 using Bit.Core.Services;
 using Bit.Core.Settings;
 using Bit.Core.Tools.Repositories;
-using Bit.Core.Vault.Models.Data;
 using Bit.Core.Vault.Repositories;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
@@ -31,8 +28,6 @@ public class SyncController : Controller
     private readonly IPolicyRepository _policyRepository;
     private readonly ISendRepository _sendRepository;
     private readonly GlobalSettings _globalSettings;
-    private readonly ICurrentContext _currentContext;
-    private readonly Version _fido2KeyCipherMinimumVersion = new Version(Constants.Fido2KeyCipherMinimumVersion);
 
     public SyncController(
         IUserService userService,
@@ -44,8 +39,7 @@ public SyncController(
         IProviderUserRepository providerUserRepository,
         IPolicyRepository policyRepository,
         ISendRepository sendRepository,
-        GlobalSettings globalSettings,
-        ICurrentContext currentContext)
+        GlobalSettings globalSettings)
     {
         _userService = userService;
         _folderRepository = folderRepository;
@@ -57,7 +51,6 @@ public SyncController(
         _policyRepository = policyRepository;
         _sendRepository = sendRepository;
         _globalSettings = globalSettings;
-        _currentContext = currentContext;
     }
 
     [HttpGet("")]
@@ -79,7 +72,7 @@ await _providerUserRepository.GetManyOrganizationDetailsByUserAsync(user.Id,
         var hasEnabledOrgs = organizationUserDetails.Any(o => o.Enabled);
         var folders = await _folderRepository.GetManyByUserIdAsync(user.Id);
         var allCiphers = await _cipherRepository.GetManyByUserIdAsync(user.Id, hasEnabledOrgs);
-        var ciphers = FilterFidoKeys(allCiphers);
+        var ciphers = allCiphers;
         var sends = await _sendRepository.GetManyByUserIdAsync(user.Id);
 
         IEnumerable<CollectionDetails> collections = null;
@@ -100,13 +93,4 @@ await _providerUserRepository.GetManyOrganizationDetailsByUserAsync(user.Id,
             collectionCiphersGroupDict, excludeDomains, policies, sends);
         return response;
     }
-
-    private ICollection<CipherDetails> FilterFidoKeys(ICollection<CipherDetails> ciphers)
-    {
-        if (_currentContext.ClientVersion >= _fido2KeyCipherMinimumVersion)
-        {
-            return ciphers;
-        }
-        return ciphers.Where(c => c.Type != Core.Vault.Enums.CipherType.Fido2Key).ToList();
-    }
 }
diff --git a/src/Api/Vault/Models/Request/CipherRequestModel.cs b/src/Api/Vault/Models/Request/CipherRequestModel.cs
@@ -82,9 +82,6 @@ public Cipher ToCipher(Cipher existingCipher)
             case CipherType.SecureNote:
                 existingCipher.Data = JsonSerializer.Serialize(ToCipherSecureNoteData(), JsonHelpers.IgnoreWritingNull);
                 break;
-            case CipherType.Fido2Key:
-                existingCipher.Data = JsonSerializer.Serialize(ToCipherFido2KeyData(), JsonHelpers.IgnoreWritingNull);
-                break;
             default:
                 throw new ArgumentException("Unsupported type: " + nameof(Type) + ".");
         }
@@ -232,28 +229,6 @@ private CipherSecureNoteData ToCipherSecureNoteData()
             Type = SecureNote.Type,
         };
     }
-
-    private CipherFido2KeyData ToCipherFido2KeyData()
-    {
-        return new CipherFido2KeyData
-        {
-            Name = Name,
-            Notes = Notes,
-            Fields = Fields?.Select(f => f.ToCipherFieldData()),
-            PasswordHistory = PasswordHistory?.Select(ph => ph.ToCipherPasswordHistoryData()),
-
-            CredentialId = Fido2Key.CredentialId,
-            KeyAlgorithm = Fido2Key.KeyAlgorithm,
-            KeyType = Fido2Key.KeyType,
-            KeyCurve = Fido2Key.KeyCurve,
-            KeyValue = Fido2Key.KeyValue,
-            RpId = Fido2Key.RpId,
-            RpName = Fido2Key.RpName,
-            UserHandle = Fido2Key.UserHandle,
-            UserDisplayName = Fido2Key.UserDisplayName,
-            Counter = Fido2Key.Counter
-        };
-    }
 }
 
 public class CipherWithIdRequestModel : CipherRequestModel

diff --git a/src/Api/Vault/Models/Response/CipherResponseModel.cs b/src/Api/Vault/Models/Response/CipherResponseModel.cs
@@ -48,12 +48,6 @@ public CipherMiniResponseModel(Cipher cipher, IGlobalSettings globalSettings, bo
                 cipherData = identityData;
                 Identity = new CipherIdentityModel(identityData);
                 break;
-            case CipherType.Fido2Key:
-                var fido2KeyData = JsonSerializer.Deserialize<CipherFido2KeyData>(cipher.Data);
-                Data = fido2KeyData;
-                cipherData = fido2KeyData;
-                Fido2Key = new CipherFido2KeyModel(fido2KeyData);
-                break;
             default:
                 throw new ArgumentException("Unsupported " + nameof(Type) + ".");
         }

diff --git a/src/Core/Vault/Enums/CipherType.cs b/src/Core/Vault/Enums/CipherType.cs
@@ -8,5 +8,4 @@ public enum CipherType : byte
     SecureNote = 2,
     Card = 3,
     Identity = 4,
-    Fido2Key = 5,
 }
diff --git a/src/Core/Vault/Models/Data/CipherFido2KeyData.cs b/src/Core/Vault/Models/Data/CipherFido2KeyData.cs