Skip to content

update for checkMessageProcessingErrors#1250

Merged
sosnovsky merged 2 commits intomasterfrom
tests/issue-1234
Dec 24, 2021
Merged

update for checkMessageProcessingErrors#1250
sosnovsky merged 2 commits intomasterfrom
tests/issue-1234

Conversation

@fcvakintos
Copy link
Contributor

Added new verification for "key mismatch unexpectedly produces a modal", updated "not integrity protected - should show a warning and not decrypt automatically" message check

This PR contains new verification for "key mismatch unexpectedly produces a modal", update for "not integrity protected - should show a warning and not decrypt automatically" message

close #1234


Tests

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

Added new verification for "key mismatch unexpectedly produces a modal", updated "not integrity protected - should show a warning and not decrypt automatically" message check
@fcvakintos fcvakintos requested a review from sosnovsky December 24, 2021 08:16
Copy link
Collaborator

@sosnovsky sosnovsky left a comment

Choose a reason for hiding this comment

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

Well done 👍

@sosnovsky sosnovsky merged commit b1f602a into master Dec 24, 2021
@sosnovsky sosnovsky deleted the tests/issue-1234 branch December 24, 2021 09:49
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.

update decrypt error tests

3 participants