Skip to content

fix: call e2eRotate on existing conversations [WPB-24528]#20923

Merged
e-maad merged 1 commit intodevfrom
fix/handle-e2e-rotate-for-existing-convs-WPB-24528
Apr 6, 2026
Merged

fix: call e2eRotate on existing conversations [WPB-24528]#20923
e-maad merged 1 commit intodevfrom
fix/handle-e2e-rotate-for-existing-convs-WPB-24528

Conversation

@e-maad
Copy link
Copy Markdown
Contributor

@e-maad e-maad commented Apr 2, 2026

BugWPB-24528 [Web] User not able to send messages after e2ei was enbaled while user is logged in

Summary

When we enable e2eI for existing MLS client, all the conversations must me called with e2eiRotate.

Security Checklist (required)

  • External inputs are validated & sanitized on client and/or server where applicable.
  • API responses are validated; unexpected shapes are handled safely (fallbacks or errors).
  • No unsafe HTML is rendered; if unavoidable, sanitization is applied and documented where it happens.
  • Injection risks (XSS/SQL/command) are prevented via safe APIs and/or escaping.

Accessibility (required)

Standards Acknowledgement (required)


Screenshots or demo (if the user interface changed)

Notes for reviewers

  • Trade-offs:
  • Follow-ups (linked issues):
  • Linked PRs (e.g. web-packages):

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 2, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 14
  • Failed: 0
  • Skipped: 0
  • 🔁 Flaky: 0
  • 📊 Total: 14
  • Total Runtime: 521.8s (~ 8 min 42 sec)

@e-maad e-maad enabled auto-merge April 6, 2026 18:25
@e-maad e-maad added this pull request to the merge queue Apr 6, 2026
Merged via the queue into dev with commit 89ea5a7 Apr 6, 2026
85 checks passed
@e-maad e-maad deleted the fix/handle-e2e-rotate-for-existing-convs-WPB-24528 branch April 6, 2026 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants