Preston Van Loon
2fd6bd8150
Add golang.org/x/tools modernize static analyzer and fix violations ( #15946 )
...
* Ran gopls modernize to fix everything
go run golang.org/x/tools/gopls/internal/analysis/modernize/cmd/modernize@latest -fix -test ./...
* Override rules_go provided dependency for golang.org/x/tools to v0.38.0.
To update this, checked out rules_go, then ran `bazel run //go/tools/releaser -- upgrade-dep -mirror=false org_golang_x_tools` and copied the patches.
* Fix buildtag violations and ignore buildtag violations in external
* Introduce modernize analyzer package.
* Add modernize "any" analyzer.
* Fix violations of any analyzer
* Add modernize "appendclipped" analyzer.
* Fix violations of appendclipped
* Add modernize "bloop" analyzer.
* Add modernize "fmtappendf" analyzer.
* Add modernize "forvar" analyzer.
* Add modernize "mapsloop" analyzer.
* Add modernize "minmax" analyzer.
* Fix violations of minmax analyzer
* Add modernize "omitzero" analyzer.
* Add modernize "rangeint" analyzer.
* Fix violations of rangeint.
* Add modernize "reflecttypefor" analyzer.
* Fix violations of reflecttypefor analyzer.
* Add modernize "slicescontains" analyzer.
* Add modernize "slicessort" analyzer.
* Add modernize "slicesdelete" analyzer. This is disabled by default for now. See https://go.dev/issue/73686 .
* Add modernize "stringscutprefix" analyzer.
* Add modernize "stringsbuilder" analyzer.
* Fix violations of stringsbuilder analyzer.
* Add modernize "stringsseq" analyzer.
* Add modernize "testingcontext" analyzer.
* Add modernize "waitgroup" analyzer.
* Changelog fragment
* gofmt
* gazelle
* Add modernize "newexpr" analyzer.
* Disable newexpr until go1.26
* Add more details in WORKSPACE on how to update the override
* @nalepae feedback on min()
* gofmt
* Fix violations of forvar
2025-11-14 01:27:22 +00:00
Bastin
92bd211e4d
upgrade v6 to v7 ( #15989 )
...
* upgrade v6 to v7
* changelog
* update-go-ssz
2025-11-06 16:16:23 +00:00
terence
774b9a7159
Migrate Prysm repo to Offchain Labs organization ahead of Pectra V6 ( #15140 )
...
* Migrate Prysm repo to Offchain Labs organization ahead of Pectra upgrade v6
* Replace prysmaticlabs with OffchainLabs on general markdowns
* Update mock
* Gazelle and add mock.go to excluded generated mock file
2025-04-10 15:40:39 +00:00
LesCyber
83cf0f8658
refactor: use errors.New to replace fmt.Errorf with no parameters ( #15007 )
...
Signed-off-by: LesCyber <andi4cing@gmail.com >
2025-04-01 19:03:16 +00:00
Preston Van Loon
2aa52fb56a
Add static analyzer to discourage use of panic() ( #15075 )
...
* Implement static analysis to prevent panics
* Add nopanic to nogo
* Fix violations and add exclusions
Fix violations and add exclusions for all
* Changelog fragment
* Use pass.Report instead of pass.Reportf
* Remove strings.ToLower for checking init method name
* Add exclusion for herumi init
* Move api/client/beacon template function to init and its own file
* Fix nopanic testcase
2025-03-19 18:04:15 +00:00
terence
0f39857653
Clean up infinity signature usages ( #15072 )
2025-03-18 19:27:06 +00:00
Preston Van Loon
014dbd5c3a
Lint: Fix violations of non-constant format string in call ( #14974 )
...
* Fix violations of non-constant format string in call
* Changelog fragment
2025-02-21 19:46:19 +00:00
Patrice Vignola
070a765d24
Add stub for VerifySignature when build tag blst_disabled is set ( #12246 )
...
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
2024-06-12 12:46:23 +00:00
Sammy Rosso
4ff91bebf8
Switch gomock library ( #13639 )
...
* Update gomock
* Update mockgen
* Gaz
* Go mod
* Cleanup
* Regenerate gomock
* Manually fix import
2024-02-21 18:37:17 +00:00
terence
5a66807989
Update to V5 ( #13622 )
...
* First take at updating everything to v5
* Patch gRPC gateway to use prysm v5
Fix patch
* Update go ssz
---------
Co-authored-by: Preston Van Loon <pvanloon@offchainlabs.com >
2024-02-15 05:46:47 +00:00
Dhruv Bodani
55a29a4670
Implement beacon committee selections ( #13503 )
...
* implement beacon committee selections
* fix build
* fix lint
* fix lint
* Update beacon-chain/rpc/eth/shared/structs.go
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
* Update validator/client/beacon-api/beacon_committee_selections.go
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
* Update validator/client/beacon-api/beacon_committee_selections.go
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
* Update validator/client/beacon-api/beacon_committee_selections.go
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
* move beacon committee selection structs to validator module
* fix bazel build files
* add support for POST and GET endpoints for get state validators query
* add a handler to return error from beacon node
* move beacon committee selection to validator top-level module
* fix bazel
* re-arrange fields to fix lint
* fix TestServer_InitializeRoutes
* fix build and lint
* fix build and lint
* fix TestSubmitAggregateAndProof_Distributed
---------
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
2024-02-05 15:43:51 +00:00
terencechain
76fec1799e
Replace Empty Slice Literals with Nil Slices ( #13093 )
...
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2023-10-23 16:36:11 +00:00
terencechain
9387a36b66
Refactor Exported Names to Follow Golang Best Practices ( #13075 )
...
* Fix exported names that start with a package name
* A few more renames
* Fix exported names that start with a package name
* A few more renames
* Radek's feedback
* Fix conflict
* fix keymanager test
* Fix comments
---------
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2023-10-20 16:45:33 +00:00
terencechain
823f8ee3a2
Fix redundant type converstion ( #13076 )
...
* Fix redundant type converstion
* Revert generated changes
---------
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2023-10-20 15:07:10 +00:00
Nishant Das
0919b2245f
Avoid Public Key Copies During Aggregation ( #12944 )
...
* Add in optimization
* add better test
---------
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2023-09-26 11:13:19 +00:00
Preston Van Loon
751dd847b8
Update rules go & gazelle ( #12721 )
...
* Update rules_go to v0.40.0
* Update gazelle to v0.26.0
* Update gazelle to v0.27.0
* Update gazelle to v0.28.0
* Update gazelle to v0.29.0
* Update gazelle to v0.30.0
* Update gazelle to v0.31.0
2023-08-11 16:39:35 +00:00
Raul Jordan
e9c8e84618
Integrate Read-Only-Lock-on-Get LRU Cache for Public Keys ( #12646 )
...
* use new lru cache
* update build
---------
Co-authored-by: Nishant Das <nishdas93@gmail.com >
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2023-07-22 04:37:53 +00:00
minh-bq
25ebd335cb
Fix bls signature batch unit test ( #12602 )
...
We randomly observe this failure when running unit test
go test -test.v -run=^TestSignatureBatch_AggregateBatch/common_and_uncommon_messages_in_batch_with_multiple_messages
=== RUN TestSignatureBatch_AggregateBatch
=== RUN TestSignatureBatch_AggregateBatch/common_and_uncommon_messages_in_batch_with_multiple_messages
signature_batch_test.go:643: AggregateBatch() Descriptions got = [test signature bls aggregated signature test signature bls aggregated signature test signature bls aggregated signature], want [bls aggregated signature test signature bls aggregated signature test signature bls aggregated signature test signature]
--- FAIL: TestSignatureBatch_AggregateBatch (0.02s)
--- FAIL: TestSignatureBatch_AggregateBatch/common_and_uncommon_messages_in_batch_with_multiple_messages (0.02s)
The problem is that the signature sort forgets to swap the description when a
swap occurs. This commit adds the description swap when swap occurs.
Co-authored-by: james-prysm <90280386+james-prysm@users.noreply.github.com >
2023-07-07 14:26:02 -05:00
Potuz
be23773924
Don't use max cover on unaggregated atts nor check subgroup of validated signatures ( #12350 )
...
* Don't use max cover on unnaggregated atts
* Do not validate signature on the attestation package
* separate nil error checks
* fix unit tests
2023-05-16 17:06:26 +00:00
Preston Van Loon
4b4e213a24
stategen: Pre-populate bls pubkey cache as part of stategen's Resume function ( #11482 )
...
* Pre-populate bls pubkey cache as part of state gen's Resume function. This adds some helpers and a benchmark to blst
* Do it async
* fix missing import
* lint
---------
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
Co-authored-by: rauljordan <raul@prysmaticlabs.com >
2023-05-10 10:44:15 +00:00
Justin Traglia
4c916403e9
Enable dupword linter & fix findings ( #12271 )
...
* Enable dupword linter & fix findings
* Correct an incorrect fix
* Add nolint comment
* Add another nolint comment
* Revert unintended change to bazel version
---------
Co-authored-by: Preston Van Loon <preston@prysmaticlabs.com >
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
2023-04-18 22:01:27 +00:00
Justin Traglia
8428a79971
Enable whitespace linter & fix findings ( #12273 )
...
* Enable whitespace linter & fix findings
* Fix new finding
* fix new violation
---------
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
Co-authored-by: Preston Van Loon <preston@prysmaticlabs.com >
Co-authored-by: Preston Van Loon <pvanloon@offchainlabs.com >
2023-04-18 18:58:27 +00:00
terencechain
d17996f8b0
Update to V4 🚀 ( #12134 )
...
* Update V3 from V4
* Fix build v3 -> v4
* Update ssz
* Update beacon_chain.pb.go
* Fix formatter import
* Update update-mockgen.sh comment to v4
* Fix conflicts. Pass build and tests
* Fix test
2023-03-17 18:52:56 +00:00
Radosław Kapka
ac4483417d
Redesign voluntary exits pool ( #11898 )
...
Co-authored-by: Nishant Das <nishdas93@gmail.com >
2023-03-01 17:44:00 +01:00
Manu NALEPA
93298dfc56
Fix #11948 and reduces call to GetFeeRecipientByPubKey only in ListFeeRecipientByPubkey in nothing is defined in VC ( #11970 )
...
* Fix #11948 and reduces call to `GetFeeRecipientByPubKey` only in `ListFeeRecipientByPubkey` in nothing is defined in VC
* Fix typo
2023-02-14 04:53:54 +00:00
Potuz
c1f00923c1
Implement bls_to_execution_changes beacon endpoint ( #11845 )
...
* Initial implementation
* fix withrawal unit tests
* add custom hooks api middleware test
* unit tests for endpoint
* update screwed proto file
* double import
* Raul review
Co-authored-by: james-prysm <90280386+james-prysm@users.noreply.github.com >
2023-01-06 05:17:33 +00:00
Patrice Vignola
83f48350b2
Fix a bunch of deepsource warnings ( #11814 )
2022-12-22 09:20:10 +00:00
Ye Ding
e43152102e
Identify invalid signature within batch verification ( #11582 ) ( #11741 )
...
* Identify invalid signature within batch verification (#11582 )
* Fix issues found by linter
* Make deepsource happy
* Add signature description enums
* Make descriptions a mandatory field
* More readable error message of invalid signatures
* Add 'enable-verbose-sig-verification' option
* Fix format
* Move descriptions to package signing
* Add more validation and test cases
* Fix build failure
Co-authored-by: Nishant Das <nishdas93@gmail.com >
2022-12-20 18:41:47 +08:00
Potuz
babfc66c5b
Check BLS changes when requesting from pool ( #11718 )
...
* Check BLS changes when requesting from pool
* Terence's suggestions
* Radek's suggestion
Co-authored-by: terencechain <terence@prysmaticlabs.com >
Co-authored-by: Raul Jordan <raul@prysmaticlabs.com >
2022-12-09 14:41:45 +00:00
Potuz
0aaee51973
Process bls changes ( #11684 )
...
* Implement ProcessBLSToExecutionChanges
* Batch process signatures
* gaz
* Change runtime behavior
* Terence's review
2022-11-24 19:36:12 +00:00
Preston Van Loon
4b033f4cc7
Update go to 1.19.3 ( #11630 )
...
* Update go to 1.19.3
* Update other items to 1.19
* Update golangci-lint to latest release
* Run gofmt -s with go1.19
* Huge gofmt changes
Co-authored-by: Raul Jordan <raul@prysmaticlabs.com >
2022-11-18 19:12:19 +00:00
Nishant Das
5ad36486b0
Remove Skip BLS Verification Feature ( #11233 )
...
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2022-08-17 02:59:13 +00:00
Raul Jordan
d077483577
Add V3 Suffix to All Prysm Packages ( #11083 )
...
* v3 import renamings
* tidy
* fmt
* rev
* Update beacon-chain/core/epoch/precompute/reward_penalty_test.go
* Update beacon-chain/core/helpers/validators_test.go
* Update beacon-chain/db/alias.go
* Update beacon-chain/db/alias.go
* Update beacon-chain/db/alias.go
* Update beacon-chain/db/iface/BUILD.bazel
* Update beacon-chain/db/kv/kv.go
* Update beacon-chain/db/kv/state.go
* Update beacon-chain/rpc/prysm/v1alpha1/validator/attester_test.go
* Update beacon-chain/rpc/prysm/v1alpha1/validator/attester_test.go
* Update beacon-chain/sync/initial-sync/service.go
* fix deps
* fix bad replacements
* fix bad replacements
* change back
* gohashtree version
* fix deps
Co-authored-by: Nishant Das <nishdas93@gmail.com >
Co-authored-by: Potuz <potuz@prysmaticlabs.com >
2022-08-16 12:20:13 +00:00
kasey
044a4ad5a3
Ignore genesis state url and checkpoint sync flags after first run of prysm ( #10881 )
...
* ignore remote genesis url flag if present in db
* ignore checkpoint sync flags if initialized
* lint
Co-authored-by: Kasey Kirkham <kasey@users.noreply.github.com >
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2022-06-14 23:23:25 +00:00
Håvard Anda Estensen
b55ddb5a34
Use go:build lines and remove obsolete +build lines ( #10704 )
...
* Use go:build lines and remove obsolete +build lines
* Run gazelle
* Update crypto/bls/blst/stub.go
* Update crypto/bls/blst/stub.go
Co-authored-by: Raul Jordan <raul@prysmaticlabs.com >
Co-authored-by: Preston Van Loon <preston@prysmaticlabs.com >
Co-authored-by: Nishant Das <nishdas93@gmail.com >
2022-06-14 11:47:27 +00:00
Nishant Das
2cc62cdf40
Fix Fuzzing ( #10798 )
2022-06-01 13:15:50 +00:00
terencechain
092e9e1d19
Clean up various warnings ( #10710 )
...
* Clean up various warnings
* Update beacon-chain/rpc/prysm/v1alpha1/debug/state_test.go
Co-authored-by: Preston Van Loon <preston@prysmaticlabs.com >
* Fix redundant casting genState
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
Co-authored-by: Preston Van Loon <preston@prysmaticlabs.com >
2022-05-19 04:38:04 +00:00
Preston Van Loon
d2dbc13427
fuzz: Fix tests with fuzz tags ( #10619 )
...
* some fixes to fuzz building
* use len
* remove comments
* fix test
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2022-05-04 06:15:09 +00:00
Nishant Das
e5e4dee629
Update Batch Verification Routine ( #10127 )
...
* add changes so far
* a lot of tests
* add flags and comments
* raul's comment
* raul's review
* fix build
* fix
* fix
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2022-05-04 04:47:53 +00:00
Preston Van Loon
f763b35494
fuzz: build with -tags=fuzz,blst_disabled ( #10618 )
2022-05-03 22:19:05 +00:00
Nishant Das
ef8bc97d3e
more efficient ( #10440 )
2022-03-28 20:27:14 +08:00
kasey
0b1a777d62
ran gofmt @ v1.17, these are the resulting changes ( #10362 )
...
* ran gofmt @ v1.17, these are the resulting changes
* fix some flaky tests that are tripping up this PR
* gofmt
Co-authored-by: Kasey Kirkham <kasey@users.noreply.github.com >
2022-03-14 15:58:13 -05:00
Leo Lara
b4b976c28b
Experimental prototype of Apple M1 processor support ( #10192 )
...
* Experimental prototype of Apple M1 processor support
* Enable Apple M1 compilation of herumi MCL by adding a precompiled library
* Renable nogo
* Fix by gazelle
* Update go.mod to reflect go 1.17.6 changes in WORKSPACE
* go mod tidy
Co-authored-by: Raul Jordan <raul@prysmaticlabs.com >
Co-authored-by: prestonvanloon <preston@prysmaticlabs.com >
2022-02-19 18:17:08 +00:00
Nishant Das
b4fa626aff
Use Batch Decompression For Signatures ( #10105 )
...
* use batch decompress method
* terence's review
* gosimple
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
Co-authored-by: prestonvanloon <preston@prysmaticlabs.com >
2022-01-21 22:57:29 +00:00
Preston Van Loon
7df01feb58
Remove libfuzzer / afl based builds in favor of go 1.18 fuzz testing ( #10065 )
...
* Remove libfuzzer / afl based builds in favor of go 1.18 fuzz testing
* rm beacon-fuzz archive
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2022-01-20 04:51:02 +00:00
terence tsao
c69bce5d84
Use fieldparams for BLS public key ( #10042 )
...
* Use fieldparams for pubkey length
* Fix validator tests
* fix more tests
* fix mock validator
* Fix typo
* bunch of typos
* Update bytes.go
* Update BUILD.bazel
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
2022-01-06 17:33:08 +00:00
terence tsao
b38b0186b8
Use signature length from fieldparams ( #10019 )
...
* Use bls sig length from fieldparams
* fmt
* fix tests
* fix tests
* fix tests
* Update tags_test.go
* fix tests
* Update BUILD.bazel
2021-12-15 15:14:30 -05:00
Raul Jordan
b381ad49b5
Add Justifications for Gosec Ignored ( #10005 )
...
* pin gosec
* edit
* go back to master
* justifications
* Update crypto/bls/blst/signature.go
* proper format
* gosec
2021-12-09 19:40:48 +00:00
Nishant Das
6f20d17d15
Rename To Signature Batch ( #9926 )
...
Co-authored-by: Radosław Kapka <rkapka@wp.pl >
2021-11-23 17:57:06 +01:00
terence tsao
ae2c883aaf
Generate secret key from big number test only ( #9816 )
...
* Generate secret key from big number test only
* Gazelle
Co-authored-by: Raul Jordan <raul@prysmaticlabs.com >
2021-10-25 19:14:19 +00:00