Skip to content

Commit

Permalink
zcash_note_encryption: Remove esk check requirement from `Domain::p…
Browse files Browse the repository at this point in the history
…arse_note_plaintext_without_memo_ovk`

This method is only called from `try_output_recovery_with_ock`, and we
can instead rely on the check performed in `check_note_validity`,
reducing the number of checks that `Domain` implementations need to
perform.

The `esk` and `ephemeral_key` parameters become unused, and will be
removed in a subsequent commit (as this change needs to be synchronized
with the `orchard` crate).
  • Loading branch information
str4d committed May 19, 2023
1 parent b341903 commit a115a8f
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions components/zcash_note_encryption/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -229,8 +229,6 @@ pub trait Domain {
/// which may be passed via `self`).
/// - The note plaintext contains valid encodings of its various fields.
/// - Any domain-specific requirements are satisfied.
/// - `ephemeral_key` can be derived from `esk` and the diversifier within the note
/// plaintext.
///
/// `&self` is passed here to enable the implementation to enforce contextual checks,
/// such as rules like [ZIP 212] that become active at a specific block height.
Expand Down

0 comments on commit a115a8f

Please sign in to comment.