Skip to content

Commit

Permalink
Merge pull request #253 from filecoin-project/fix/tanlang/auth-url-no…
Browse files Browse the repository at this point in the history
…-more-necessary

Fix/tanlang/auth url no more necessary / 去掉 auth url 的检查
  • Loading branch information
diwufeiwen authored Dec 2, 2022
2 parents 0ff1ca5 + eb7643c commit 443e34c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 35 deletions.
10 changes: 4 additions & 6 deletions cmd/venus-market/run.go
Original file line number Diff line number Diff line change
Expand Up @@ -158,19 +158,17 @@ func runDaemon(cctx *cli.Context) error {
return fmt.Errorf("prepare run failed: %w", err)
}

// Configuration sanity check
if len(cfg.AuthNode.Url) == 0 {
return fmt.Errorf("pool mode have to configure auth node")
}

if len(cfg.Signer.Url) == 0 {
return fmt.Errorf("the signer node must be configured")
}

ctx := cctx.Context

// 'NewAuthClient' never returns an error, no needs to check
authClient, _ := jwtclient.NewAuthClient(cfg.AuthNode.Url)
var authClient *jwtclient.AuthClient
if len(cfg.AuthNode.Url) != 0 {
authClient, _ = jwtclient.NewAuthClient(cfg.AuthNode.Url)
}

resAPI := &impl.MarketNodeImpl{}
shutdownChan := make(chan struct{})
Expand Down
29 changes: 0 additions & 29 deletions docker/script/compose.sh

This file was deleted.

0 comments on commit 443e34c

Please sign in to comment.