-
Notifications
You must be signed in to change notification settings - Fork 10
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.
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
Chores grooming #11
Merged
Merged
Chores grooming #11
Changes from 39 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
46a4190
change sever to operator and clint to initiator
pavelkrolevets 8f83e7a
update readme about operator`s priv key
pavelkrolevets 64ee75f
change plaintext password to file
pavelkrolevets a5b52c5
add logs about message signature verification
pavelkrolevets 44e019f
change fork hex to name
pavelkrolevets a6d122f
add identity to initiator
pavelkrolevets 9ddfa77
add security notes to readme
pavelkrolevets aa1c0a7
update flags + docker
pavelkrolevets ba097fb
rename project
pavelkrolevets 8935dad
readme update
pavelkrolevets ecf4e91
fix tests + work on comments
pavelkrolevets 4ad85a0
add test of initiator identity
pavelkrolevets ac06e54
update test operator
pavelkrolevets a71153e
add linter + fixes
pavelkrolevets 18c0b58
comment fixes
pavelkrolevets 79805c5
update configs and paths to store
pavelkrolevets 1174c1f
readme update
pavelkrolevets aecffb6
add .dockerignore
pavelkrolevets 211e779
add logging to file same as ssv
pavelkrolevets 954e17f
update board logger
pavelkrolevets 5d7d73b
readme update
pavelkrolevets 859819e
- Break dkg function into smaller functions
y0sher e02bc02
update tests
pavelkrolevets 1aec967
update tests
pavelkrolevets 70432af
add unhappy flow
pavelkrolevets bf6ce6c
add threshold tests
pavelkrolevets 114b3d8
clean up
pavelkrolevets c3748e6
Merge remote-tracking branch 'origin/chores_grooming' into refactor/l…
pavelkrolevets ff259b4
fix makefile
pavelkrolevets 260398a
Merge pull request #13 from bloxapp/refactor/logging_and_codestyle
pavelkrolevets 8808ccd
Merge branch 'main' into chores_grooming
pavelkrolevets c00be40
add more threshold checks for 4 ops
pavelkrolevets 2495c75
groom log reports
pavelkrolevets 9675a4d
minor fix
pavelkrolevets 4ab3607
add test shares order ar ssv payload
pavelkrolevets d7a439c
move NewID to crypto
pavelkrolevets 7949342
remove VerifyFunc
pavelkrolevets 8765162
move VerifyInitiatorMessage to state
pavelkrolevets c3c1dcd
removed rsa priv from local owner
pavelkrolevets e1caae3
logs path
y0sher e6fd291
change payload strcuture
y0sher 8c9050c
fix payload to v4
y0sher 8704e08
correct help for flag
y0sher 2fad7d7
set default nonce to 0
y0sher 8a21497
fixed integration tests to new format
y0sher 28c4c0a
added single operator/initiator to Makefile
y0sher e79c08b
change withdrawl address to be ETH1
y0sher b0a1ae6
fix test to look for the ETH1 addr
y0sher f09a35a
add hexToAddress function with error handling
pavelkrolevets 62077ed
add error wrapper at state
pavelkrolevets b623cbc
fix operator test
pavelkrolevets a2b65cf
- change ssvpayload name to keyshares
y0sher 695177e
change error to show bad sigs
y0sher File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
./bin/ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,3 @@ | ||
.idea/ | ||
bin/ | ||
bin/ | ||
.vscode/ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,15 @@ | ||
FROM golang:1.20 | ||
FROM golang:1.20-alpine3.17 | ||
RUN apk add build-base | ||
|
||
WORKDIR / | ||
|
||
COPY go.mod go.sum ./ | ||
RUN go mod download | ||
RUN go mod download && go mod verify | ||
|
||
COPY ./ ./ | ||
|
||
# Build | ||
RUN CGO_ENABLED=1 GOOS=linux go build -o /app /cmd/dkgcli/dkgcli.go | ||
RUN CGO_ENABLED=1 GOOS=linux go build -o /app /cmd/ssv-dkg/ssv-dkg.go | ||
|
||
|
||
EXPOSE 3030 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,45 +2,60 @@ | |
# with Go source code. If you know what GOPATH is then you probably | ||
# don't need to bother with make. | ||
|
||
.PHONY: dkgcli test clean build docker-build | ||
.PHONY: install clean build test docker-build-image docker-operators docker-initiator mockgen-install lint-prepare lint | ||
|
||
GOBIN = ./build/bin | ||
GO ?= latest | ||
GORUN = env GO111MODULE=on go run | ||
GOINSTALL = env GO111MODULE=on go install -v | ||
GOTEST = env GO111MODULE=on go test -v | ||
# Name of the Go binary output | ||
BINARY_NAME=./bin/dkgcli | ||
BINARY_NAME=./bin/ssv-dkg | ||
# Docker image name | ||
DOCKER_IMAGE=ssv-dkg-tool | ||
DOCKER_IMAGE=ssv-dkg | ||
|
||
install: | ||
$(GOINSTALL) cmd/dkgcli/dkgcli.go | ||
$(GOINSTALL) cmd/ssv-dkg/ssv-dkg.go | ||
@echo "Done building." | ||
@echo "Run dkgcli to launch the tool." | ||
@echo "Run ssv-dkg to launch the tool." | ||
|
||
clean: | ||
env GO111MODULE=on go clean -cache | ||
|
||
# Recipe to compile the Go program | ||
build: | ||
@echo "Building Go binary..." | ||
go build -o $(BINARY_NAME) ./cmd/dkgcli/dkgcli.go | ||
go build -o $(BINARY_NAME) ./cmd/ssv-dkg/ssv-dkg.go | ||
|
||
# Recipe to run tests | ||
test: | ||
@echo "running tests" | ||
go test -p 1 ./... | ||
go test -v -p 1 ./... | ||
|
||
# Recipe to build the Docker image | ||
docker-build: | ||
docker-build-image: | ||
@echo "Building Docker image..." | ||
docker build -t $(DOCKER_IMAGE) . | ||
|
||
docker-servers: | ||
@echo "Running servers in docker demo" | ||
docker-compose up --build server1 server2 server3 server4 | ||
docker-operators: | ||
@echo "Running operators in docker demo" | ||
docker-compose up --build operator1 operator2 operator3 operator4 | ||
|
||
docker-client: | ||
@echo "Running client in docker demo" | ||
docker-compose up --build client | ||
docker-initiator: | ||
@echo "Running initiator in docker demo" | ||
docker-compose up --build initiator | ||
|
||
mockgen-install: | ||
go install github.com/golang/mock/[email protected] | ||
@which mockgen || echo "Error: ensure `go env GOPATH` is added to PATH" | ||
|
||
lint-prepare: | ||
@echo "Preparing Linter" | ||
curl -sfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh| sh -s latest | ||
|
||
lint: | ||
./bin/golangci-lint run -v ./... | ||
@if [ ! -z "${UNFORMATTED}" ]; then \ | ||
echo "Some files requires formatting, please run 'go fmt ./...'"; \ | ||
exit 1; \ | ||
fi |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove tool
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.