Skip to content

Commit 9498614

Browse files
committed
Merge branch 'develop' into feat/CRE-1035-extend-standard-cap
2 parents 2842ddb + e478a40 commit 9498614

132 files changed

Lines changed: 3453 additions & 3941 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.changeset/eight-tips-bathe.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"chainlink": minor
3+
---
4+
5+
#internal Optimize beholder validator in system tests (part 2)

.changeset/plenty-results-laugh.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"chainlink": patch
3+
---
4+
5+
#updated CCIP changesets to deploy v1.5.1 TokenPoolFactory instead of latest version

.changeset/slick-drinks-like.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"chainlink": minor
3+
---
4+
5+
#internal Update PoR workflow to use chainlink BalanceReader bindings

.changeset/thirty-lamps-own.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"chainlink": minor
3+
---
4+
5+
Bump dependency of chainlink-evm #internal

.github/CODEOWNERS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,7 @@ core/scripts/gateway @smartcontractkit/dev-services
8888
/deployment/common @smartcontractkit/devex-tooling
8989
/deployment/ccip @smartcontractkit/ccip-tooling @smartcontractkit/ccip-offchain @smartcontractkit/core @smartcontractkit/cld-team
9090
/deployment/ccip/changeset/globals @smartcontractkit/ccip-offchain
91+
/deployment/crib @smartcontractkit/ccip-offchain
9192
/deployment/ccip/changeset/ton @smartcontractkit/bix-build @smartcontractkit/core @smartcontractkit/cld-team
9293
/deployment/keystone @smartcontractkit/keystone @smartcontractkit/core @smartcontractkit/cld-team
9394
/deployment/data-feeds @smartcontractkit/data-feeds-engineers @smartcontractkit/core @smartcontractkit/cld-team

.github/integration-in-memory-tests.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ runner-test-matrix:
4949
- Nightly Integration CCIP Tests
5050
test_cmd: cd smoke/ccip && go test -run "Test_CCIPMessaging_EVM2Solana" -timeout 18m -test.parallel=4 -count=1 -json
5151
test_go_project_path: integration-tests
52+
install_plugins_public: true
5253

5354
- id: smoke/ccip/ccip_messaging_test.go:Test_CCIPMessaging_Solana2EVM
5455
path: integration-tests/smoke/ccip/ccip_messaging_test.go
@@ -60,6 +61,7 @@ runner-test-matrix:
6061
- Nightly Integration CCIP Tests
6162
test_cmd: cd smoke/ccip && go test -run "Test_CCIPMessaging_Solana2EVM" -timeout 18m -test.parallel=4 -count=1 -json
6263
test_go_project_path: integration-tests
64+
install_plugins_public: true
6365

6466
- id: smoke/ccip/ccip_messaging_test.go:Test_CCIPMessaging_MultiExecReports_EVM2Solana
6567
path: integration-tests/smoke/ccip/ccip_messaging_test.go
@@ -222,6 +224,7 @@ runner-test-matrix:
222224
- Nightly Integration CCIP Tests
223225
test_cmd: go test smoke/ccip/ccip_token_transfer_test.go -timeout 16m -test.parallel=2 -count=1 -json
224226
test_go_project_path: integration-tests
227+
install_plugins_public: true
225228

226229
- id: smoke/ccip/ccip_cs_update_rmn_config_test.go:*
227230
path: integration-tests/smoke/ccip/ccip_cs_update_rmn_config_test.go
@@ -600,8 +603,7 @@ runner-test-matrix:
600603
triggers:
601604
- PR Integration CCIP Tests
602605
- Nightly Integration CCIP Tests
603-
test_cmd:
604-
cd smoke/ccip && go test ccip_ton_messaging_test.go -timeout 20m -test.parallel=1 -count=1 -json
606+
test_cmd: cd smoke/ccip && go test ccip_ton_messaging_test.go -timeout 20m -test.parallel=1 -count=1 -json
605607
test_go_project_path: integration-tests
606608
install_plugins_public: true
607609
# END: CCIP tests

.mockery.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,9 @@ packages:
379379
config:
380380
mockname: "Mock{{ .InterfaceName }}"
381381
filename: all_price_getter_mock.go
382-
github.com/smartcontractkit/chainlink/v2/core/services/relay/evm/statuschecker:
382+
github.com/smartcontractkit/chainlink-evm/pkg/statuschecker:
383+
config:
384+
dir: "core/services/relay/evm/mocks/statuschecker"
383385
interfaces:
384386
CCIPTransactionStatusChecker:
385387
github.com/smartcontractkit/chainlink/v2/core/services/ocr2/plugins/ccip/internal/rpclib:

core/capabilities/ccip/ccip_integration_tests/usdcreader/usdcreader_test.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import (
1919
"github.com/stretchr/testify/mock"
2020
"github.com/stretchr/testify/require"
2121

22+
"github.com/smartcontractkit/chainlink-ccip/chains/evm/gobindings/generated/latest/usdc_reader_tester"
2223
typepkgmock "github.com/smartcontractkit/chainlink-ccip/mocks/pkg/types/ccipocr3"
2324
ccipocr3common "github.com/smartcontractkit/chainlink-common/pkg/types/ccipocr3"
2425
"github.com/smartcontractkit/chainlink-evm/pkg/config"
@@ -29,15 +30,12 @@ import (
2930
"github.com/smartcontractkit/chainlink-ccip/pkg/reader"
3031
cciptypes "github.com/smartcontractkit/chainlink-ccip/pkg/types/ccipocr3"
3132
"github.com/smartcontractkit/chainlink-ccip/pluginconfig"
32-
33+
"github.com/smartcontractkit/chainlink-common/pkg/logger"
3334
"github.com/smartcontractkit/chainlink-evm/pkg/client"
3435
"github.com/smartcontractkit/chainlink-evm/pkg/heads/headstest"
3536
"github.com/smartcontractkit/chainlink-evm/pkg/logpoller"
3637
"github.com/smartcontractkit/chainlink-evm/pkg/utils"
3738
ubig "github.com/smartcontractkit/chainlink-evm/pkg/utils/big"
38-
39-
"github.com/smartcontractkit/chainlink-ccip/chains/evm/gobindings/generated/latest/usdc_reader_tester"
40-
"github.com/smartcontractkit/chainlink-common/pkg/logger"
4139
evmconfig "github.com/smartcontractkit/chainlink/v2/core/capabilities/ccip/configs/evm"
4240
"github.com/smartcontractkit/chainlink/v2/core/internal/testutils"
4341
"github.com/smartcontractkit/chainlink/v2/core/internal/testutils/pgtest"

core/capabilities/ccip/ccipevm/msghasher.go

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,15 @@ import (
44
"context"
55
"errors"
66
"fmt"
7+
"math"
78
"math/big"
89
"strings"
910

1011
"github.com/ethereum/go-ethereum/accounts/abi"
1112
"github.com/ethereum/go-ethereum/common"
1213
"github.com/ethereum/go-ethereum/common/hexutil"
1314
chainsel "github.com/smartcontractkit/chain-selectors"
15+
1416
"github.com/smartcontractkit/chainlink-ccip/chains/evm/gobindings/generated/v1_6_0/message_hasher"
1517
"github.com/smartcontractkit/chainlink-ccip/pkg/logutil"
1618
cciptypes "github.com/smartcontractkit/chainlink-ccip/pkg/types/ccipocr3"
@@ -295,11 +297,16 @@ func extractDestGasAmountFromMap(input map[string]any) (uint32, error) {
295297
lowercase := strings.ToLower(fieldName)
296298
switch lowercase {
297299
case "destgasamount":
298-
// Expect uint32
299-
if val, ok := fieldValue.(uint32); ok {
300-
return val, nil
301-
} else {
302-
return 0, errors.New("invalid type for destgasamount, expected uint32")
300+
switch v := fieldValue.(type) {
301+
case uint32:
302+
return v, nil
303+
case int64: // LOOP converts expected uint32 to int64
304+
if v > math.MaxUint32 {
305+
return 0, fmt.Errorf("destGasAmount exceeds uint32 max, got %d", v)
306+
}
307+
return uint32(v), nil //nolint:gosec // G115: validated to be within uint32 max above
308+
default:
309+
return 0, errors.New("invalid type for destgasamount, expected uint32 or int64")
303310
}
304311
default:
305312
}

core/capabilities/ccip/ccipsolana/executecodec.go

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import (
66
"encoding/binary"
77
"errors"
88
"fmt"
9+
"math"
910
"math/big"
1011
"strings"
1112

@@ -58,7 +59,7 @@ func (e *ExecutePluginCodecV1) Encode(ctx context.Context, report ccipocr3.Execu
5859
return nil, fmt.Errorf("empty amount for token: %s", tokenAmount.DestTokenAddress)
5960
}
6061

61-
if tokenAmount.Amount.Int.Sign() < 0 {
62+
if tokenAmount.Amount.Sign() < 0 {
6263
return nil, fmt.Errorf("negative amount for token: %s", tokenAmount.DestTokenAddress)
6364
}
6465

@@ -222,14 +223,18 @@ func extractDestGasAmountFromMap(input map[string]any) (uint32, error) {
222223
lowercase := strings.ToLower(fieldName)
223224
switch lowercase {
224225
case "destgasamount":
225-
// Expect uint32
226-
if v, ok := fieldValue.(uint32); ok {
226+
switch v := fieldValue.(type) {
227+
case uint32:
227228
return v, nil
228-
} else {
229-
return 0, errors.New("invalid type for destgasamount, expected uint32")
229+
case int64: // LOOP converts expected uint32 to int64
230+
if v > math.MaxUint32 {
231+
return 0, fmt.Errorf("destGasAmount exceeds uint32 max, got %d", v)
232+
}
233+
return uint32(v), nil //nolint:gosec // G115: validated to be within uint32 max above
234+
default:
235+
return 0, errors.New("invalid type for destgasamount, expected uint32 or int64")
230236
}
231237
default:
232-
233238
}
234239
}
235240

0 commit comments

Comments
 (0)