From 49a188635ad13ec023f740e0bad3df30945c7538 Mon Sep 17 00:00:00 2001 From: howdymary <35550889+howdymary@users.noreply.github.com> Date: Wed, 31 Jan 2024 14:20:07 -0500 Subject: [PATCH 1/3] Update arbitrum-integration.md change in fall back on method to ability change in link to sequencer_inbox.go --- developers/arbitrum-integration.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/developers/arbitrum-integration.md b/developers/arbitrum-integration.md index bc6fe813f20..e6034faf94b 100644 --- a/developers/arbitrum-integration.md +++ b/developers/arbitrum-integration.md @@ -51,13 +51,13 @@ which relays commitments to Celestia’s data root to an onchain light client on Ethereum. This allows L2 solutions that settle on Ethereum to benefit from the scalability Celestia’s data availability layer can provide. -As part of this integration, Blobstream has been incorporated into the +As part of this integration, Blobstream is being called from the Arbitrum `SequencerInbox.sol` contract. In the `SequencerInbox.sol` contract, the `validateBatchData` modifier has been designed to authenticate that the data root is on Celestia when reading a batch of data. This is achieved by the -[following code by @Ferret-san](https://gist.github.com/Ferret-san/3d3fc1b5738ee8d77ad112c0eb8bbe5f): +[following code by @Ferret-san]([https://arbnode/sequencer_inbox.go](https://github.com/celestiaorg/nitro/blob/4dac70821b5bbb7c48b6e5c2460663dc0c07e011/arbnode/sequencer_inbox.go)): Note that the data above is the bytes serialized version of this struct in Go: @@ -101,8 +101,8 @@ The [@celestiaorg/nitro](https://github.com/celestiaorg/nitro) integration [uses the same fallback mechanism](https://github.com/celestiaorg/nitro/blob/f01968eb3d4e19329e9c92b050e98a8e5772f1f2/arbnode/batch_poster.go#L845-L857). The fallback logic for Celestia DA is configurable, providing an alternative -to the previous default fallback mechanism. Additionally, a method has been -added to the Arbitrum node software. This method allows the sequencer to call +to the previous default fallback mechanism. Additionally, an ability has been +added to the Arbitrum node software which allows the sequencer to call `VerifyAttestation` to check if a data root has been posted on Blobstream or not, before it sends the sequencer message (data pointer) to the underlying chain. From cf1e5cab04c7787966cf710ea76214907da38545 Mon Sep 17 00:00:00 2001 From: jcstein Date: Wed, 31 Jan 2024 14:59:40 -0500 Subject: [PATCH 2/3] fix: use v2.2.2 release link --- developers/arbitrum-integration.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/developers/arbitrum-integration.md b/developers/arbitrum-integration.md index e6034faf94b..944b643c9ec 100644 --- a/developers/arbitrum-integration.md +++ b/developers/arbitrum-integration.md @@ -57,7 +57,7 @@ Arbitrum `SequencerInbox.sol` contract. In the `SequencerInbox.sol` contract, the `validateBatchData` modifier has been designed to authenticate that the data root is on Celestia when reading a batch of data. This is achieved by the -[following code by @Ferret-san]([https://arbnode/sequencer_inbox.go](https://github.com/celestiaorg/nitro/blob/4dac70821b5bbb7c48b6e5c2460663dc0c07e011/arbnode/sequencer_inbox.go)): +[following code in `arbnode/sequencer_inbox.go`](https://github.com/celestiaorg/nitro/blob/v2.2.2-no-blobstream/arbnode/sequencer_inbox.go): Note that the data above is the bytes serialized version of this struct in Go: From 173a2a906f0474fd8e81f90dc786de8f3375f79b Mon Sep 17 00:00:00 2001 From: Josh Stein <46639943+jcstein@users.noreply.github.com> Date: Wed, 31 Jan 2024 15:12:21 -0500 Subject: [PATCH 3/3] Update developers/arbitrum-integration.md --- developers/arbitrum-integration.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/developers/arbitrum-integration.md b/developers/arbitrum-integration.md index 944b643c9ec..2d91ddceda6 100644 --- a/developers/arbitrum-integration.md +++ b/developers/arbitrum-integration.md @@ -57,7 +57,7 @@ Arbitrum `SequencerInbox.sol` contract. In the `SequencerInbox.sol` contract, the `validateBatchData` modifier has been designed to authenticate that the data root is on Celestia when reading a batch of data. This is achieved by the -[following code in `arbnode/sequencer_inbox.go`](https://github.com/celestiaorg/nitro/blob/v2.2.2-no-blobstream/arbnode/sequencer_inbox.go): +[following code in `arbnode/sequencer_inbox.go`](https://github.com/celestiaorg/nitro-contracts/blob/celestia/blobstream/src/bridge/SequencerInbox.sol#L334-L360): Note that the data above is the bytes serialized version of this struct in Go: