Skip to content

Conversation

@David-Henner
Copy link
Contributor

@David-Henner David-Henner commented Dec 3, 2025

StoryWPB-7394 [iOS] remove cryptobox project from wire-ios

Issue

This PR is part of a larger effort to remove cryptobox and rely only on core crypto.

  1. chore(cryptobox): retire ProteusProvider - WPB-21821 #3903
  2. chore(cryptobox): remove proteusViaCoreCrypto flag - WPB-14604 #3954
  3. chore(cryptobox): remove keyStore - WPB-7393 #3955

Changes in this PR:

  • Removed wire-ios-cryptobox framework
  • Removed cryptobox dependency from mock transport
  • Removed the action to reset session on recoverable cryptobox errors
  • Removed developer setting to "share cryptobox"
  • Removed remaining cryptobox dependencies

Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

…wireapp/wire-ios into chore/retire-proteus-provider-WPB-21821
# Conflicts:
#	wire-ios-request-strategy/Tests/Sources/Notifications/PushNotifications/ZMLocalNotificationTests_Message.swift
# Conflicts:
#	wire-ios-data-model/Tests/Source/Model/UserClient/UserClientKeyStoreTests.swift
@github-actions
Copy link
Contributor

github-actions bot commented Dec 3, 2025

Test Results

    7 files    980 suites   21m 55s ⏱️
7 523 tests 7 492 ✅ 27 💤 4 ❌
7 524 runs  7 497 ✅ 27 💤 0 ❌

For more details on these failures, see this check.

Results for commit f79a37c.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@netbe netbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, although left a couple questions

Base automatically changed from chore/remove-keystore-WPB-7393 to develop December 12, 2025 11:06
# Conflicts:
#	WireMessaging/Sources/WireMessagingDomain/WireCells/Protocols/WireCellsEditingURLRepositoryProtocol.swift
#	WireMessaging/Sources/WireMessagingUI/WireCells/Components/Common/WireCellsNode+Convenience.swift
#	wire-ios-data-model/Source/ManagedObjectContext/NSManagedObjectContext+AccountDirectory.swift
#	wire-ios-data-model/Source/Proteus/CryptoboxMigrationManager.swift
#	wire-ios-data-model/Tests/Core Crypto/CoreCryptoConfigProviderTests.swift
#	wire-ios-data-model/Tests/Proteus/CryptoboxMigrationManagerTests.swift
#	wire-ios-request-strategy/Sources/Synchronization/Decoding/EventDecoder.swift
#	wire-ios-request-strategy/Sources/Synchronization/Decoding/EventDecoderTest.swift
#	wire-ios-request-strategy/Tests/Sources/Notifications/PushNotifications/ZMLocalNotificationTests_Event.swift
#	wire-ios-utilities/Source/DeveloperFlag.swift
@David-Henner David-Henner added this pull request to the merge queue Dec 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 12, 2025
@David-Henner David-Henner added this pull request to the merge queue Dec 14, 2025
Merged via the queue into develop with commit ab47168 Dec 14, 2025
16 checks passed
@David-Henner David-Henner deleted the chore/remove-cryptobox-framework-WPB-7394 branch December 14, 2025 19:27
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

Successfully merging this pull request may close these issues.

4 participants