filecoin-project / filecoin-project/venus

venus公共组建部分token配置的优化建议

Open
#5,823 0 comments 0 reactions 0 assignees View on GitHub
C-enhancement CU-force-community P3
Dominant language
Go
Stars
2.1k
Forks
493
PR merge metrics
No merged PRs in 30d

Description

### 链服务模块 / Chain Service Components

- [ ] venus
- [ ] venus-auth
- [ ] venus-gateway
- [X] venus-messager
- [X] venus-miner
- [ ] 文档 / docs

### 订单服务模块 / Deal Service Components

- [ ] venus-market
- [ ] 文档 / docs

### 算力服务模块 / Storage Power Service Components

- [ ] venus-sector-manager
- [ ] venus-worker
- [ ] 文档 / docs

### 版本 / Version

```text
/dev/1.11.0
```

### 描述 / Describe the Bug
```
-----------------config.toml-----------------
[FullNode]
Addr = "/ip4/127.0.0.1/tcp/3453"
Token = "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJuYW1lIjoiZGVmYXVsdExvY2FsVG9rZW4iLCJwZXJtIjoiYWRtaW4iLCJleHQiOiIifQ.2HZGrV_BjXkm8XDzhOilo7m0jPYyMR2JW5FSP2e7V0w"

[Gateway]
ListenAPI = ["/ip4/127.0.0.1/tcp/45132"]
Token = "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJuYW1lIjoiZGVmYXVsdExvY2FsVG9rZW4iLCJwZXJtIjoiYWRtaW4iLCJleHQiOiIifQ.2HZGrV_BjXkm8XDzhOilo7m0jPYyMR2JW5FSP2e7V0w"

[Auth]
Addr = "http://127.0.0.1:8989"
Token = "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJuYW1lIjoiZGVmYXVsdExvY2FsVG9rZW4iLCJwZXJtIjoiYWRtaW4iLCJleHQiOiIifQ.2HZGrV_BjXkm8XDzhOilo7m0jPYyMR2JW5FSP2e7V0w"
---------------------------------
```
1. 以venus-miner的云上组建的配置文件为例,存在3个token配置的地方
2. venus-miner是云上组建,token的角色应改只是用admin,不应该存在其他权限的使用场景
3. 如果同样都是admin的token情况下,也就没有必要分别配置多个admin的场景
4. 所以token的配置应该仅需要一个参数就可以,不需要相同的token填写多次
5. 建议其他的云上组建也都简化一下,增强易用性

### 日志 / Logging Information

```text

```

### 重现步骤 / Repo Steps

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.