Skip to content

Comments

Fixed displaying decrypted message for NO_MDC errors#1494

Merged
IvanPizhenko merged 4 commits intomasterfrom
issue_1363_decrypt_no_mdc
Oct 11, 2021
Merged

Fixed displaying decrypted message for NO_MDC errors#1494
IvanPizhenko merged 4 commits intomasterfrom
issue_1363_decrypt_no_mdc

Conversation

@DenBond7
Copy link
Collaborator

@DenBond7 DenBond7 commented Oct 5, 2021

This PR fixed displaying decrypted message for NO_MDC errors

close #1363


Tests (delete all except exactly one):

  • 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

…inles 0.2.15 or high). Update tests. Updated image: flowcrypt/flowcrypt-email-server:0.0.9| #1363
DenBond7 and others added 3 commits October 10, 2021 21:47
# Conflicts:
#	FlowCrypt/src/main/java/com/flowcrypt/email/security/pgp/PgpMsg.kt
#	docker-mailserver/docker-compose.yml
#	docker-mailserver/maildata_source/flowcrypt.test/default/mailboxes/INBOX/dbox-Mails/dovecot.index.cache
#	docker-mailserver/maildata_source/flowcrypt.test/default/mailboxes/INBOX/dbox-Mails/dovecot.index.log
#	docker-mailserver/maildata_source/flowcrypt.test/denbond7/dovecot.list.index.log
#	docker-mailserver/maildata_source/flowcrypt.test/denbond7/mailboxes/INBOX/dbox-Mails/dovecot.index.cache
#	docker-mailserver/maildata_source/flowcrypt.test/denbond7/mailboxes/Sent/dbox-Mails/dovecot.index.cache
#	docker-mailserver/maildata_source/flowcrypt.test/denbond7/mailboxes/Sent/dbox-Mails/dovecot.index.log
@DenBond7 DenBond7 marked this pull request as ready for review October 11, 2021 10:32
@IvanPizhenko IvanPizhenko merged commit ca1dd2a into master Oct 11, 2021
@IvanPizhenko IvanPizhenko deleted the issue_1363_decrypt_no_mdc branch October 11, 2021 20:23
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.

Kotlin decrypt: rendering of MDC error

2 participants