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

ADR 012: Handlers validation and execution separation #2631

Closed
wants to merge 25 commits into from
Closed

ADR 012: Handlers validation and execution separation #2631

Show file tree
Hide file tree
Changes from 20 commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
c0494b0
init adr 12
plafer Sep 9, 2022
05356c6
Host based API
hu55a1n1 Sep 11, 2022
916c5c3
Add appendices
hu55a1n1 Sep 11, 2022
2b78427
Add section on event logging
hu55a1n1 Sep 12, 2022
4ed1a01
Add .changelog entry
hu55a1n1 Sep 12, 2022
446c89e
Fix TypedStore associated type (syntax only)
plafer Sep 19, 2022
d70bd08
Update docs/architecture/adr-012-handlers-redesign.md
hu55a1n1 Sep 19, 2022
f1c0354
validation vs execution clarifications
plafer Sep 19, 2022
4df0522
Move ValidationContext and ExecutionContext methods to appendix
plafer Sep 19, 2022
82b85f1
Context clarification
plafer Sep 19, 2022
f6fdb03
Rework Host-based API section
plafer Sep 19, 2022
92508ad
ValidationContext and ExecutionContext clarification
plafer Sep 19, 2022
92f2ddb
Move Event logging section + misc
plafer Sep 19, 2022
2d72a2a
typo
plafer Sep 19, 2022
9c71b46
Remove statement. Was mentioned earlier already.
plafer Sep 19, 2022
666db87
IbcStore blanket implementation
plafer Sep 19, 2022
d0dc499
IbcValueForPath rework
plafer Sep 19, 2022
0ed15b2
Remove IbcSerde from main ADR
plafer Sep 19, 2022
382462b
TypedStore -> SubStore
plafer Sep 19, 2022
5aeb855
diagram
plafer Sep 19, 2022
c27fe03
Update docs/architecture/adr-012-handlers-redesign.md
plafer Sep 23, 2022
9ee1c62
Update docs/architecture/adr-012-handlers-redesign.md
plafer Sep 23, 2022
d3c0b03
Update docs/architecture/adr-012-handlers-redesign.md
plafer Sep 23, 2022
d2d67e4
add context for why the validation/execution split is necessary
plafer Sep 23, 2022
d3e0c38
add trait bounds
plafer Sep 23, 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
2 changes: 2 additions & 0 deletions .changelog/unreleased/improvements/ibc/2582-adr012-handler-validation-execution.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- Propose ADR012 for handlers validation and execution separation.
([#2582](https://github.com/informalsystems/ibc-rs/pull/2582)).
1 change: 1 addition & 0 deletions docs/architecture/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,3 +38,4 @@ To suggest an ADR, please make use of the [ADR template](./adr-template.md) prov
| [009](./adr-009-chain-endpoint-handle-standardization.md) | ChainEndpoint and ChainHandle methods standardization | Accepted |
| [010](./adr-010-unified-cli-arguments-hermes.md) | Unified approach for CLI arguments for Hermes v1.0.0 | Proposed |
| [011](./adr-011-light-client-crates-extraction.md) | Light client crates extraction | Proposed |
| [012](./adr-012-handlers-redesign.md) | Handlers validation and execution separation | Proposed |
Loading