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.

Sign up for GitHub

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

Jump to bottom

Implement tari_crypto hashing API: DAN layer #4392

Closed
hansieodendaal opened this issue Aug 5, 2022 · 0 comments
Closed

Implement tari_crypto hashing API: DAN layer #4392

hansieodendaal opened this issue Aug 5, 2022 · 0 comments
Assignees
@jorgeantonio21
Labels
A-security Area - Security related

Comments

@hansieodendaal
Copy link
Contributor

Effective GitHub hash: d25d726
# Reference Function/Area Yes/No URL URL long
27 dan_layer/core/src/models/hot_stuff_message.rs:202 pub fn create_signature_challenge( Yes hot_stuff_message.rs:202 https://github.com/tari-project/tari/blob/development/dan_layer/core/src/models/hot_stuff_message.rs#L202
28 dan_layer/core/src/models/tari_dan_payload.rs:58 fn calculate_hash( Yes tari_dan_payload.rs:58 https://github.com/tari-project/tari/blob/development/dan_layer/core/src/models/tari_dan_payload.rs#L58
29 dan_layer/core/src/templates/tip004_template.rs:97 fn hash_of(s: &str) Yes tip004_template.rs:97 https://github.com/tari-project/tari/blob/development/dan_layer/core/src/templates/tip004_template.rs#L97
30 dan_layer/engine/src/instructions/instruction.rs:81 pub fn calculate_hash( Yes instruction.rs:81 https://github.com/tari-project/tari/blob/development/dan_layer/engine/src/instructions/instruction.rs#L81
@jorgeantonio21 jorgeantonio21 self-assigned this Aug 9, 2022
@jorgeantonio21 jorgeantonio21 mentioned this issue Aug 9, 2022
Merged
stringhandler pushed a commit that referenced this issue Aug 10, 2022
@jorgeantonio21
fix: implement hashing api for dan layer (see issue #4392) (#4427)
f7c5e77 
Description
--- Add tari hashing api for DAN layer.

Motivation and Context
--- Tackle issue #4392.

How Has This Been Tested?
--- Existing unit tests.
@hansieodendaal hansieodendaal added the A-security Area - Security related label Aug 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jorgeantonio21 jorgeantonio21

Labels
A-security Area - Security related
Projects
Tari Esme Testnet
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jorgeantonio21 @hansieodendaal