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

CIP-0040? | Collateral output #216

Merged
merged 18 commits into from
Jul 2, 2022
Merged

CIP-0040? | Collateral output #216

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
b4c7280
added work from PR#25 to support single stake pool links as well
rphair Jan 17, 2021
b24d0ef
for brevity
rphair Jan 17, 2021
235ad40
for clarity
rphair Jan 17, 2021
75faa8f
response to first round of editing by co-author
rphair Jan 18, 2021
0389fa1
second round of co-author edits
rphair Jan 18, 2021
adbee6f
reclassified wallet integration as Motivation rather than Rationale
rphair Jan 18, 2021
176b5c0
2 of 3 changes from @nicarq via PR 65
rphair Feb 20, 2021
20f35f6
undone most recent commit (removing PR65 suggestions)
rphair Feb 20, 2021
f43d28c
Merge branch 'master' of github.com:rphair/CIPs
rphair Feb 20, 2021
85c4b77
undone most recent commit (removing PR65 suggestions)
rphair Feb 20, 2021
db81cb6
Merge remote-tracking branch 'upstream/master'
rphair Apr 12, 2021
efd6491
Collateral output CIP
SebastienGllmt Feb 10, 2022
d3c48ce
rename file
SebastienGllmt Feb 10, 2022
07ff3b5
Include self-contained balancing requirement
SebastienGllmt Feb 14, 2022
138565e
Add Andre to the CIP
SebastienGllmt Feb 15, 2022
b136b3b
Merge branch 'master' of github.com:cardano-foundation/CIPs
rphair Jul 2, 2022
89239b2
Merge branch 'output-collateral' of github.com:cardano-foundation/CIP…
rphair Jul 2, 2022
c50eff3
pre merge changes, attempt as 3rd party
rphair Jul 2, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion CollateralOutput/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
CIP: ?
Title: Explicit Collateral Output
Authors: Sebastien Guillemot <[email protected]>
Authors: Sebastien Guillemot <[email protected]>, Jared Corduan <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Authors: Sebastien Guillemot <[email protected]>, Jared Corduan <jared.corduan@iohk.io>
Authors: Sebastien Guillemot <[email protected]>, Andre Knispel <andre.knispel@iohk.io>

Status: Draft
Type: Standards
Created: 2022-02-10
Expand Down Expand Up @@ -55,6 +55,10 @@ However, if collateral output is specified, then
2. Collateral output needs to be balanced according to `sum(collateral_input) = sum(collateral_output) + collateral_consumed`
Where `collateral_consumed` is equal to the old formula (`quot (txfee txb * (collateralPercent pp)) 100`). Note that when collateral is consumed, any certificate, etc. in the transaction is ignored so they have no impact on the change calculation.

## Self-contained balancing

Some use-cases like hardware wallets, who do not have access to the content of the collateral inputs, cannot easily check if the collateral is balanced. Similar to how we specify an explicit fee as part of the transaction body to tackle this problem, the transaction body also needs a new field that explicitly specified how much collateral will be consumed in the case of phase-2 validation failure.

## Copyright

This CIP is licensed under [CC-BY-4.0](https://creativecommons.org/licenses/by/4.0/legalcode)