forked from zcash/orchard
-
Notifications
You must be signed in to change notification settings - Fork 0
Upstream PR 471 (OrchardZSA) – internal review and fixes #201
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.
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
Closed
Closed
Changes from all commits
Commits
Show all changes
153 commits
Select commit
Hold shift + click to select a range
7f8a848
Circleci project setup (#1)
PaulLaux 1328c62
issuer keys implementation (#5)
daniben31 e6b7762
Added NoteType to Notes (#2)
PaulLaux 088abc6
ZSA note encryption in Orchard crate (#3)
naure 1420f84
Zsa builder (#4)
PaulLaux 0b2988a
Issuance (#12)
PaulLaux 9b43497
E2E tests for issuance (#20)
alexeykoren 985d0d2
disabled split notes (#22)
PaulLaux f3ebe7a
Review fixes (#23)
PaulLaux 355b569
Review fixes2 (#24)
PaulLaux 9405f80
verify_issue_bundle() cleanup (#25)
PaulLaux d8f3563
ZSA burn functionality (#35)
alexeykoren 5a50fb8
Create dummy spend in empty builder by default (#36)
alexeykoren efbfc19
AssetID test vectors (#34)
alexeykoren babf1da
CI-Badge (#37)
PaulLaux 4681cc2
Use different random nullifiers in dummy notes (#39)
alexeykoren cec48d7
V3 encryption (#38)
PaulLaux 4e47677
Minimal API changes for Issuance in Client (#43)
alexeykoren cbf0a3a
Circuit: Split_flag handling (#42)
ConstanceBeguier 43d5e77
AssetBase spec update (#44)
PaulLaux 527e29a
Upgrade pprof version with a limited inferno version (#48)
ConstanceBeguier f0b7948
Making changes to the asset base derivation from the asset identifier…
vivek-arte 563b4e5
Circuit: Update value_commit_orchard to take into account asset (#50)
ConstanceBeguier 7d3b6df
Constant-time note commitment for ZEC and ZSA (#54)
ConstanceBeguier ea0fd59
Add tracking for supply info inside verify_issue_bundle (#55)
dmidem 9a35108
Add getter method for Bundle.burn field (#58)
dmidem 4e1c616
Merge branch 'main' into zsa1
ConstanceBeguier c77d96c
Merge branch main into zsa1 (#59)
ConstanceBeguier aeb9934
Circuit: Update note_commit to take into account asset (#56)
ConstanceBeguier b4f6281
Circuit: Use nf_old_pub to evaluate new note commitment (with rho_new…
ConstanceBeguier 95fcf88
Circuit: Check pk_d_old = derived_pk_d_old only when split_flag = 0 (…
ConstanceBeguier 8e71fff
Protect bundle burn from adding assets with zero amount (#60)
dmidem bedc732
Circuit: Add tests for orchard circuit (#63)
ConstanceBeguier 32eee6e
Do not create split notes with native asset (#65)
ConstanceBeguier 02fa582
Global padding for bundle (#67)
ConstanceBeguier 9965a6d
Add serialization of finalize flag (#68)
ConstanceBeguier 7ad2bac
Fix IssueBundle and IssueAction structures (#70)
ConstanceBeguier 950b806
Updates to TXID Digest and Authorizing Data Commitment (#66)
vivek-arte 8bc18f7
Merge branch main into zsa1
ConstanceBeguier 5b003f8
Merge branch main into zsa1 (#72)
ConstanceBeguier f49be89
Update code to refer to the upgraded version of librustzcash and refl…
dmidem aa1d895
Fix issuance key derivation (#74)
ConstanceBeguier daf6269
Preventing Asset Base from being the identity point on the Pallas cur…
vivek-arte 477f949
Update random nullifier for split notes and circuit (#76)
ConstanceBeguier d4ff716
Merge branch 'zsa1' into upgrade_librustzcash_for_orchard_v05_new
dmidem 21d7273
Use tag instead of branch for zcash_note_encryption (librustzcash) re…
dmidem 62d4ae7
Change tag name for zcash_note_encryption in root Cargo.toml
dmidem 1a00c4a
Upgrade orchard to v0.5 and integrate corresponding librustzcash upgr…
dmidem ff2ac96
Circuit: Add constraints (#77)
ConstanceBeguier 081513b
Circuit: Fix balance violation (#78)
ConstanceBeguier 139ecca
Circuit: Add enable_zsa flag (#79)
ConstanceBeguier 7937e5b
Enhance and cleanup ivk-to-bytes-visibility-downgrade branch (#81)
dmidem 8b0560d
Circuit: optimized short range check on 4 and 5 bits (#86)
ConstanceBeguier 2810365
Circuit: remove mutiplexer chip (#90)
ConstanceBeguier a680f41
Circuit: optimize ZEC/ZSA hash computations in note commitment (#87)
ConstanceBeguier 7b943e1
Adopt Orchard ZSA for Zebra (introduce zcash_note_encryption_zsa alia…
dmidem f38d6b9
Rename `sk_iss` to `isk`, the `IssuanceKey` struct to `IssuanceAuthor…
vivek-arte 0ee75f5
Circuit: move mux functionality into CondSwap chip (#94)
ConstanceBeguier 344b647
Update zcash_note_encryption dependency reference (#95)
dmidem 1a1f3e7
Changing the Issuance Authorization Signature to the BIP 340 Schnorr …
vivek-arte c9a8f49
Merge branch 'zcash-0.8.0' into zsa1-with-zcash-0.8.0
dmidem f3d9459
Fix compilation errors after the merge (there're still several compli…
dmidem 0f56f52
Fix compilation errors in tests
dmidem 149827d
Fix cargo doc issues
dmidem c8c84aa
Fix Builder::bundle function to pass unit tests from tests folder cor…
dmidem ac371f0
Fix several comments, rename some flag constants
dmidem 7969b9e
Fix flag names after renaming
dmidem 47f7aae
Disable rustfmt for one line in keys.rs module to prevent removing do…
dmidem bb9e03b
Fix typo in constant name
dmidem dd956b8
Fix comments, rename DEFAULT_WITH(OUT)_ZSA to DEVAULT_VANILLA(ZSA)
dmidem 123b609
1) Use Option instead of usize::MAX in builder::partition_by_asset fu…
dmidem ee89541
Refactor build function to remove `Copy + Into<i64>` constraints
dmidem 73218b0
Remove extra comment
dmidem 50c6310
Fix comments, indroduce and use MatadataIdx type in the builder
dmidem 78c8efc
Update to upstream (zcash/main v0.8.0) (#103)
dmidem 39b479e
OrchardZSA backward compatability 0.8.0 (#104)
dmidem 6e6112c
Synchronize Orchard with updates from zcash_note_encryption for zcash…
dmidem 07b3697
Update ShieldedOutput implementation to return reference from enc_cip…
dmidem 3ba9e5b
Remove redundancy in vanilla/zsa circuits
ConstanceBeguier e2fb49d
Updating test vectors (#114)
vivek-arte dd69425
Remove ZSA dummy notes
ConstanceBeguier 410037d
Changing the type of the asset description from String to Vec<u8> (#113)
vivek-arte a7c02d2
Provide compatibility with librustzcash/zcash_client_backend: derive …
dmidem 25020f8
Exclude non-native assets from value_balance (#120)
alexeykoren 8a2a8de
Simplify Unproven struct
ConstanceBeguier adc6995
Update SIGHASH according to OrchardDomain
ConstanceBeguier 97cf5a3
Add Clone trait to auth-related structures (#125)
alexeykoren 8b89888
Unify unique AssetBase concept in IssueAction
ConstanceBeguier fe15076
Add a reference note for each first issuance
ConstanceBeguier 9eb97f0
Simplify bundle build function return type (#129)
ConstanceBeguier 3d2515b
Fix some links and typo (#127)
ConstanceBeguier 0d2f439
Install libfontconfig1-dev to fix CI build on updated Ubuntu runner (…
dmidem 69f92a3
Update AssetSuply and SupplyInfo (new) (#133)
dmidem 3dbdbc5
Update to orchard_auth_digest computation (#134)
vivek-arte 70daf8b
[book] Update book with ZSA changes
ConstanceBeguier 40e7e10
Update derivation of the rho value in Issue Notes
ConstanceBeguier 01e85a5
Add NULLIFIER_L constant
ConstanceBeguier a95364c
Refactor AssetSupply to AssetRecords and update issuance verification…
dmidem b62f72a
Fix cargo clippy warnings in PR #138 (#140)
dmidem b8d4543
Merge PCZT into zsa1
ConstanceBeguier e88e261
Extend Orchard protocol flavor derives (#144)
dmidem aeec27d
Add ZSA feature into PCZT
ConstanceBeguier 123a2f4
Merge branch 'zsa1' into merge_pczt
ConstanceBeguier 190a50c
Move burn to ActionGroup in OrchardZSA (#145)
alexeykoren 831ca10
change asset_desc to asset_desc_hash (#156)
PaulLaux 75c55b2
resolved conflicts and updated orchard_digest for zsa
PaulLaux 6d557af
Merge branch 'zsa1' into merge_pczt_b and update hashes
PaulLaux 5c09d33
Merge pczt (#157)
PaulLaux c7d57b1
Reduced Circle CI and github CI runtime (#158)
PaulLaux 37101f4
Lock Sinsemilla version (#160)
ConstanceBeguier 57f39bf
Merge branch 'main' into merge_main_fcb14def
ConstanceBeguier d56c5d5
Fix tests comparing orchard_digest against a fixed value
ConstanceBeguier cb539d4
Update test_vectors with last version of zcash-test-vector
ConstanceBeguier 5f080b4
Fix clippy errors
ConstanceBeguier 8388060
Replace ZIP32_ORCHARD_PERSONALIZATION_FOR_ISSUANCE const by ZIP32_ORC…
ConstanceBeguier 572ba6b
Expose (Extracted)NoteCommitment in public API
ConstanceBeguier 502c410
Disable default-features for k256
ConstanceBeguier 8509081
Update Rust version in README
ConstanceBeguier cf31792
Merge main fcb14def into zsa1 (#152)
PaulLaux 0a61d36
Update with new version of Sinsemilla (#159)
ConstanceBeguier f2411a0
Use fixed size array for asset_id (#162)
PaulLaux 9a25bd1
Simplify compute_asset_desc_hash function (#161)
ConstanceBeguier 7e4f28c
Use the last version of zcash_note_encryption (#165)
ConstanceBeguier 6616893
Fix CI breakage by pinning proptest 1.2.0 and locking font-kit/half v…
dmidem cbcdb71
Pczt without domain (#168)
alexeykoren fdd69e2
Update zcash_note_encryption crate (#171)
ConstanceBeguier 1746a4b
Remove circuit flag when it is possible (#173)
ConstanceBeguier 9f44f55
Fix fake_compact_action visibility (#174)
ConstanceBeguier f7bcf01
Update circuit implementation (#167)
ConstanceBeguier c1f3dcc
Upgrate proptest dependency to 1.4.0 to be compatible with Zebra (#175)
dmidem 806878c
Adopt proptest version range (>=1.0,<1.7) and roll back Cargo.lock (#…
dmidem fab774d
Simplify Orchard Flavor (#176)
ConstanceBeguier 58b710b
Simplify ZSA implementation (#178)
ConstanceBeguier 0982ff6
Remove native_asset_base prop_compose function (#180)
ConstanceBeguier 826dc03
Add rho derivation check in verify_issue_bundle (#183)
ConstanceBeguier 76d9b7e
Updating the encoding of the Issuance Authorization keys and signatur…
vivek-arte 8784d39
Refactor IssueAuthSigScheme trait (#184)
ConstanceBeguier cc68804
Add versioned signatures (#181)
ConstanceBeguier 0e496fe
Merge branch 'main' into merge_main_9d89b50
ConstanceBeguier 532b0d2
Merge main (until commit 9d89b50) into zsa1 (#186)
PaulLaux b4517e3
Reduce diff with upstream (#187)
ConstanceBeguier d18f0d5
Update zcash_note_encryption (9f7e93) (#190)
ConstanceBeguier 962564b
Fix issuance_digest, issuance_auth_digest and orchard_digest (#189)
ConstanceBeguier 78b6f76
Updates to property testing (#191)
vivek-arte cf6d3b6
Create OrchardSighashVersion and IssueSighashVersion enum (#192)
ConstanceBeguier abd6ecc
Add spend_auth_sigs_auth_digest into orchard_auth_digest (#193)
ConstanceBeguier fc73b8e
Shuffle actions in a Bundle (#194)
ConstanceBeguier a7de16f
Fix split note in PCZT (#195)
ConstanceBeguier 2083efe
Remove PCZT v2 (#196)
ConstanceBeguier a02fdf1
Remove dev-only numeric-extension patterns mistakenly committed to .g…
dmidem c7b2d1e
CI: disable LTO in release tests; add apt update to Ubuntu steps
dmidem ccf4c19
Address upstream PR #417 review comments (stage 1 - see details for l…
dmidem cd41b4f
Address upstream PR #417 review comments (stage 2 - see details for l…
dmidem File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,51 @@ | ||
| # Use the latest 2.1 version of CircleCI pipeline process engine. | ||
| # See: https://circleci.com/docs/2.0/configuration-reference | ||
| version: 2.1 | ||
|
|
||
| orbs: | ||
| slack: circleci/slack@4.1 | ||
|
|
||
| # Define a job to be invoked later in a workflow. | ||
| # See: https://circleci.com/docs/2.0/configuration-reference/#jobs | ||
| jobs: | ||
| cargo-test: | ||
| # Specify the execution environment. You can specify an image from Dockerhub or use one of our Convenience Images from CircleCI's Developer Hub. | ||
| # See: https://circleci.com/docs/2.0/configuration-reference/#docker-machine-macos-windows-executor | ||
| docker: | ||
| - image: cimg/rust:1.71.0 | ||
| # Add steps to the job | ||
| # See: https://circleci.com/docs/2.0/configuration-reference/#steps | ||
| resource_class: large # specify a resource class | ||
| steps: | ||
| - checkout | ||
| - restore_cache: | ||
| keys: | ||
| - rust-cache-v1-{{ checksum "Cargo.lock" }} | ||
| - rust-cache-v1- | ||
| - run: | ||
| name: "cargo test" | ||
| command: | | ||
| sudo apt update && sudo apt-get install libfontconfig libfontconfig1-dev libfreetype6-dev; | ||
| cargo version; | ||
| cargo test --all --all-features --release; | ||
| - save_cache: | ||
| paths: | ||
| - target | ||
| - ~/.cargo/registry | ||
| - ~/.cargo/git | ||
| key: rust-cache-v1-{{ checksum "Cargo.lock" }} | ||
| - slack/notify: | ||
| event: fail | ||
| template: basic_fail_1 | ||
| - slack/notify: | ||
| event: pass | ||
| template: basic_success_1 | ||
|
|
||
|
|
||
| # Invoke jobs via workflows | ||
| # See: https://circleci.com/docs/2.0/configuration-reference/#workflows | ||
| workflows: | ||
| build-and-test: | ||
| jobs: | ||
| - cargo-test: | ||
| context: CI-Orchard-slack |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| /target | ||
| **/*.rs.bk | ||
| .vscode | ||
| .idea | ||
| action-circuit-layout.png |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From str4d (r2542357168)
I checked these revisions.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FIXED