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

fix: Proteus messages not being sent anymore - WPB-16067 #2526

Merged
merged 4 commits into from
Feb 13, 2025

Conversation

jullianm
Copy link
Contributor

@jullianm jullianm commented Feb 12, 2025

BugWPB-16067 [iOS] Unable to send any Proteus messages with 3.118

Issue

Context: Proteus messages are not sent anymore on 3.118

Cause: we use the legacy Cryptobox instead of CC because of proteusViaCoreCrypto returning false: plist we use to retrieve the flag value (set to true) was removed in previous PR.

Solution: add the config value back to plist

Testing

Proteus messages sending should behave normally

Checklist

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

@jullianm jullianm requested review from a team, netbe, samwyndham and KaterinaWire and removed request for a team February 12, 2025 13:50
Copy link
Contributor

github-actions bot commented Feb 12, 2025

Test Results

281 tests   281 ✅  7s ⏱️
  1 suites    0 💤
  1 files      0 ❌

Results for commit c84d7d6.

♻️ 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, I understood that Xcode creates the info.plist for additional keys

@netbe netbe enabled auto-merge (squash) February 13, 2025 10:30
@netbe netbe merged commit 207fd6d into release/cycle-3.118 Feb 13, 2025
8 of 9 checks passed
@netbe netbe deleted the fix/unsent-proteus-messages branch February 13, 2025 13:13
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.

3 participants