Skip to content

Add an optional validation of attachment checksum#12

Merged
martincivan merged 2 commits intoQualityUnit:masterfrom
ProtonMail:feat/validate_checksums
May 15, 2023
Merged

Add an optional validation of attachment checksum#12
martincivan merged 2 commits intoQualityUnit:masterfrom
ProtonMail:feat/validate_checksums

Conversation

@donataso
Copy link

@donataso donataso commented May 5, 2023

No description provided.

@martincivan
Copy link
Contributor

@donataso thank you for your contribution!

Could you please add unit test case testing this added logic?
Both validating the checksum is correct and throwing the exception when it doesn't match...

@donataso
Copy link
Author

donataso commented May 8, 2023

Thanks for the response! Added a test.

@martincivan martincivan merged commit a3a50e8 into QualityUnit:master May 15, 2023
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.

2 participants