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] Always parse encrypted messages on client #27092

Merged
merged 5 commits into from
Nov 8, 2022

Conversation

yash-rajpal
Copy link
Member

@yash-rajpal yash-rajpal commented Oct 18, 2022

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

We should ignore the cached md field for encrypted messages and should always parse the messages.
TC-10

@yash-rajpal yash-rajpal requested a review from a team as a code owner October 18, 2022 15:13
@codecov
Copy link

codecov bot commented Oct 18, 2022

Codecov Report

Merging #27092 (1eaa754) into develop (1a4d9ba) will increase coverage by 0.70%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #27092      +/-   ##
===========================================
+ Coverage    41.00%   41.71%   +0.70%     
===========================================
  Files          851      826      -25     
  Lines        18679    18233     -446     
  Branches      2049     1988      -61     
===========================================
- Hits          7660     7605      -55     
+ Misses       10730    10342     -388     
+ Partials       289      286       -3     
Flag Coverage Δ
e2e 41.71% <50.00%> (+0.70%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@yash-rajpal yash-rajpal added this to the 5.4.0 milestone Nov 7, 2022
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Nov 8, 2022
@kodiakhq kodiakhq bot merged commit e2e465c into develop Nov 8, 2022
@kodiakhq kodiakhq bot deleted the parse-encrypted-msgs-on-client branch November 8, 2022 12:57
bkrith pushed a commit to bkrith/Rocket.Chat that referenced this pull request Nov 12, 2022
@ggazzo ggazzo mentioned this pull request Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
squad: team-collab stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants