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

services/horizon: Support new CAP-21 transaction conditions #4297

Merged
merged 22 commits into from
Mar 25, 2022
Merged

services/horizon: Support new CAP-21 transaction conditions #4297

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
d28c2ca
Add cap-21 transaction preconditions
Mar 17, 2022
cd8277a
Replace xdr.TransactionEnvelope.TimeBounds -> .Preconditions function
Mar 21, 2022
ac3fb54
Rename Signers -> ExtraSigners for more consistent naming
Mar 21, 2022
3d73d12
Store the new precondition fields in the DB
Mar 21, 2022
fc1057c
Render the new precondition fields in the resourceadapter
Mar 21, 2022
64ce98e
./gogenerate.sh
Mar 21, 2022
2b13140
use a pointer so omitempty works
Mar 21, 2022
cbe15b7
more consistent naming
Mar 21, 2022
fffacc5
Keep Timebounds helper
Mar 21, 2022
bc02876
PR feedback
Mar 22, 2022
9174f76
tidy transaction precondition helpers
Mar 22, 2022
3cdb667
typo fixes
Mar 22, 2022
b646879
remove dead param
Mar 22, 2022
3104a13
use bigint so we have enough space for a uint32
Mar 23, 2022
4574288
use text instead of varchar, to avoid future migrations
Mar 23, 2022
de9bca3
Updating scenario sqls
Mar 23, 2022
c7115b2
need to load the preconditions from the db
Mar 24, 2022
4e61c0f
Temporary passing data for v2 preconditions transaction insertion test
Mar 25, 2022
4248a0c
Update xdr/transaction_envelope_test.go
Mar 25, 2022
a400725
Update services/horizon/internal/db2/history/transaction_ledger_bound…
Mar 25, 2022
7290be4
PR feedback
Mar 25, 2022
b7807a2
Merge remote-tracking branch 'origin/horizon-protocol-19' into update…
Mar 25, 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
Next Next commit
Add cap-21 transaction preconditions
  • Loading branch information
Paul Bellamy committed Mar 21, 2022
commit d28c2ca0195013c4fd5569ebdc1582f7fd1bb504
75 changes: 47 additions & 28 deletions protocols/horizon/main.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -499,34 +499,53 @@ type Transaction struct {
// When TransactionSuccess is removed from the SDKs we can remove this HAL link
Transaction hal.Link `json:"transaction"`
} `json:"_links"`
ID string `json:"id"`
PT string `json:"paging_token"`
Successful bool `json:"successful"`
Hash string `json:"hash"`
Ledger int32 `json:"ledger"`
LedgerCloseTime time.Time `json:"created_at"`
Account string `json:"source_account"`
AccountMuxed string `json:"account_muxed,omitempty"`
AccountMuxedID uint64 `json:"account_muxed_id,omitempty,string"`
AccountSequence string `json:"source_account_sequence"`
FeeAccount string `json:"fee_account"`
FeeAccountMuxed string `json:"fee_account_muxed,omitempty"`
FeeAccountMuxedID uint64 `json:"fee_account_muxed_id,omitempty,string"`
FeeCharged int64 `json:"fee_charged,string"`
MaxFee int64 `json:"max_fee,string"`
OperationCount int32 `json:"operation_count"`
EnvelopeXdr string `json:"envelope_xdr"`
ResultXdr string `json:"result_xdr"`
ResultMetaXdr string `json:"result_meta_xdr"`
FeeMetaXdr string `json:"fee_meta_xdr"`
MemoType string `json:"memo_type"`
MemoBytes string `json:"memo_bytes,omitempty"`
Memo string `json:"memo,omitempty"`
Signatures []string `json:"signatures"`
ValidAfter string `json:"valid_after,omitempty"`
ValidBefore string `json:"valid_before,omitempty"`
FeeBumpTransaction *FeeBumpTransaction `json:"fee_bump_transaction,omitempty"`
InnerTransaction *InnerTransaction `json:"inner_transaction,omitempty"`
ID string `json:"id"`
PT string `json:"paging_token"`
Successful bool `json:"successful"`
Hash string `json:"hash"`
Ledger int32 `json:"ledger"`
LedgerCloseTime time.Time `json:"created_at"`
Account string `json:"source_account"`
AccountMuxed string `json:"account_muxed,omitempty"`
AccountMuxedID uint64 `json:"account_muxed_id,omitempty,string"`
AccountSequence string `json:"source_account_sequence"`
FeeAccount string `json:"fee_account"`
FeeAccountMuxed string `json:"fee_account_muxed,omitempty"`
FeeAccountMuxedID uint64 `json:"fee_account_muxed_id,omitempty,string"`
FeeCharged int64 `json:"fee_charged,string"`
MaxFee int64 `json:"max_fee,string"`
OperationCount int32 `json:"operation_count"`
EnvelopeXdr string `json:"envelope_xdr"`
ResultXdr string `json:"result_xdr"`
ResultMetaXdr string `json:"result_meta_xdr"`
FeeMetaXdr string `json:"fee_meta_xdr"`
MemoType string `json:"memo_type"`
MemoBytes string `json:"memo_bytes,omitempty"`
Memo string `json:"memo,omitempty"`
Signatures []string `json:"signatures"`
// Action needed in release: horizon-v3.0.0: remove valid_(after|before)
ValidAfter string `json:"valid_after,omitempty"`
ValidBefore string `json:"valid_before,omitempty"`
Preconditions TransactionPreconditions `json:"preconditions,omitempty"`
FeeBumpTransaction *FeeBumpTransaction `json:"fee_bump_transaction,omitempty"`
InnerTransaction *InnerTransaction `json:"inner_transaction,omitempty"`
}

type TransactionPreconditions struct {
Timebounds struct {
MinTime string `json:"min_time,omitempty"`
MaxTime string `json:"max_time,omitempty"`
} `json:"timebounds,omitempty"`
paulbellamy marked this conversation as resolved.
Show resolved Hide resolved
Ledgerbounds struct {
MinLedger *uint32 `json:"min_ledger,omitempty"`
MaxLedger *uint32 `json:"max_ledger,omitempty"`
} `json:"ledgerbounds,omitempty"`

MinAccountSequence string `json:"min_account_sequence,omitempty"`
MinSequenceAge string `json:"min_account_sequence_age,omitempty"`
MinSequenceLedgerGap uint32 `json:"min_account_sequence_ledger_gap,omitempty"`

Signers []string `json:"extra_signers,omitempty"`
}

// FeeBumpTransaction contains information about a fee bump transaction
Expand Down