Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[venus-auth] add more check in api / 增加接口验证检查 #5820

Closed
1 of 11 tasks
Tracked by #5810
hunjixin opened this issue Mar 13, 2023 · 0 comments · Fixed by ipfs-force-community/sophon-auth#160
Closed
1 of 11 tasks
Tracked by #5810

[venus-auth] add more check in api / 增加接口验证检查 #5820

hunjixin opened this issue Mar 13, 2023 · 0 comments · Fixed by ipfs-force-community/sophon-auth#160
Assignees
@diwufeiwen
Labels
C-enhancement Category: feature enhancement design Venus feature/system design issues P2 Medium - we should get to this soon V-venus-auth

Comments

@hunjixin
Copy link
Contributor

链服务模块 / Chain Service Components

  • venus
  • venus-auth
  • venus-gateway
  • venus-messager
  • venus-miner
  • 文档 / docs

订单服务模块 / Deal Service Components

  • venus-market
  • 文档 / docs

算力服务模块 / Storage Power Service Components

  • venus-sector-manager
  • venus-worker
  • 文档 / docs

描述 / Description

venus-auth目前实现中，没有对数据关系做检查， 比如generate token并不会检查 user是否存在， 无论何时都会直接生成一个。 需要重新梳理关系， 在对应的接口中增加更多的检查。
@hunjixin hunjixin added P2 Medium - we should get to this soon design Venus feature/system design issues C-enhancement Category: feature enhancement V-venus-auth labels Mar 13, 2023
@diwufeiwen diwufeiwen mentioned this issue Mar 22, 2023
Closed
39 tasks
@diwufeiwen diwufeiwen self-assigned this Mar 22, 2023
@diwufeiwen diwufeiwen moved this to In Progress in Venus Project Mar 22, 2023
@diwufeiwen diwufeiwen mentioned this issue Mar 22, 2023
Merged
5 tasks
@diwufeiwen diwufeiwen moved this from In Progress to In Review in Venus Project Mar 22, 2023
@github-project-automation github-project-automation bot moved this from In Review to Done in Venus Project Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@diwufeiwen diwufeiwen

Labels
C-enhancement Category: feature enhancement design Venus feature/system design issues P2 Medium - we should get to this soon V-venus-auth
Projects
Venus Project
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: return detailed error infor after authentication failure ipfs-force-community/sophon-auth
2 participants
@diwufeiwen @hunjixin