feat(relayer): ccip provider and contract transmitter #81
Conversation
pkg/ccip/ocr/contract_transmitter.go
Outdated
| return fmt.Errorf("failed to generate ed25519 call data: %w", err) | ||
| } | ||
| } else { | ||
| return errors.New("no calldata function") |
There was a problem hiding this comment.
Add a constructor and check this there?
pkg/ccip/ocr/contract_transmitter.go
Outdated
| Mode: wallet.PayGasSeparately, | ||
| FromWallet: *c.fromWallet, | ||
| ContractAddress: *address.MustParseAddr(contract), | ||
| // Body: , fill me |
There was a problem hiding this comment.
I think the report will be encoded as BOC, we'll likely need to decode it, then pack it together with other args (signatures)
pkg/ccip/ocr/contract_transmitter.go
Outdated
| } | ||
| var sigBytes [96]byte | ||
| copy(sigBytes[:], sig) | ||
| signatures = append(signatures, sigBytes) |
There was a problem hiding this comment.
You you should construct types from the bindings, so the payload can be serialized: https://github.com/smartcontractkit/chainlink-ton/blob/main/pkg/ccip/bindings/plugin/commitreport.go
I think we're still missing the top level type for Commit/Execute that will hold report+signatures
briansztamfater
force-pushed
the
NONEVM-2035-relayer-setup
branch
5 times, most recently
from
37e39c6 to
8003f24
Compare
briansztamfater
force-pushed
the
NONEVM-2205-ccip-provider-transmitter
branch
from
156bf07 to
f187dc0
Compare
briansztamfater
force-pushed
the
NONEVM-2205-ccip-provider-transmitter
branch
from
8265c71 to
b780161
Compare
briansztamfater
force-pushed
the
NONEVM-2205-ccip-provider-transmitter
branch
from
b780161 to
ba10142
Compare
briansztamfater
changed the title
| cp := &Provider{ | ||
| lggr: logger.Named(lggr, CCIPProviderName), | ||
| ct: ct, | ||
| } |
There was a problem hiding this comment.
nit: want to toss a TODO: implement in here for chain accessor ca?
| } | ||
|
|
||
| func NewCCIPProvider(lggr logger.Logger, txm txm.TxManager) (*Provider, error) { | ||
| ct, err := ocr.NewCCIPTransmitter(txm, lggr) |
There was a problem hiding this comment.
It's still a little unclear to me how we're going to split between commit vs exec ToEd25519CalldataFunc if we're only creating one transmitter and we're not passing it in here or during Transmit(). cc @archseer
There was a problem hiding this comment.
I guess we can define some type inside of contract_transmitter.go to distinguish them? But we'd still need to set that somehow
There was a problem hiding this comment.
Probably we'll need to have ton_contract_transmitter_factory.go in https://github.com/smartcontractkit/chainlink/tree/develop/core/capabilities/ccip/ocrimpls ?
| FromWallet: w, | ||
| ContractAddress: *address.MustParseAddr(c.offrampAddress), | ||
| Body: body, | ||
| Amount: tlb.MustFromTON("0.05"), |
There was a problem hiding this comment.
Would we want to have an ability to configure this?
| } | ||
|
|
||
| func NewCCIPProvider(lggr logger.Logger, txm txm.TxManager) (*Provider, error) { | ||
| ct, err := ocr.NewCCIPTransmitter(txm, lggr) |
There was a problem hiding this comment.
Probably we'll need to have ton_contract_transmitter_factory.go in https://github.com/smartcontractkit/chainlink/tree/develop/core/capabilities/ccip/ocrimpls ?
| services.StateMachine | ||
| } | ||
|
|
||
| func NewCCIPProvider(lggr logger.Logger, txm txm.TxManager) (*Provider, error) { |
There was a problem hiding this comment.
iirc, do we also need to initialize TON accessor here as well?
| type ccipTransmitter struct { | ||
| txm txm.TxManager | ||
| offrampAddress string | ||
| toEd25519CalldataFn ToEd25519CalldataFunc |
There was a problem hiding this comment.
Also why do we need this fn? Why don't we just configure the transmitter with the address and method, and encode input args ourselves as this is a TON-CCIP-specific transmitter?
5 participants
NONEVM-2205
CCIPProvider and ContractTransmitter implementation