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

fix: 规范REST接口的业务与系统异常的HTTP状态码 #4382

Closed
irwinsun opened this issue Jun 10, 2021 · 0 comments
Closed

fix: 规范REST接口的业务与系统异常的HTTP状态码 #4382

irwinsun opened this issue Jun 10, 2021 · 0 comments
Assignees
@irwinsun @mingshewhe
Labels
area/ci/backend CI 后端issue done Production environment in tencent has been deploy for gray UAT environment in tencent has been deploy for test Test environment in tencent has been deploy grayed uat环境测试通过/test passed for uat stage kind/bug 程序故障Bug，漏洞 tested 测试环境通过/test patest passed for test stage
Milestone
v1.6

Comments

@irwinsun
Copy link
Member

What happened:
早期有一些微服务实现不规范，未区分业务与系统错误，导致有相当一部分业务参数逻辑错误而使用了HTTP 500 作为REST接口的返回码，会被误以为是系统出错，给系统运营带来较大的排查难度
What you expected to happen:
需要规范梳理出系统与业务异常的HTTP状态码，
业务类错误：4XX
系统级错误：5XX
How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

  • bk-ci version (use cat VERSION in installed dir): all
  • Cloud provider or hardware configuration:
  • OS (e.g: cat /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:
@irwinsun irwinsun added the kind/bug 程序故障Bug，漏洞 label Jun 10, 2021
@irwinsun irwinsun added this to the v1.6 milestone Jun 10, 2021
@irwinsun irwinsun added the area/ci/backend CI 后端issue label Jun 10, 2021
@irwinsun irwinsun self-assigned this Jun 10, 2021
@bkci-bot bkci-bot added the for test Test environment in tencent has been deploy label Jun 19, 2021
mingshewhe added a commit to mingshewhe/bk-ci that referenced this issue Jun 23, 2021
@mingshewhe
fix: 规范REST接口的业务与系统异常的HTTP状态码 TencentBlueKing#4382
c443b2d 
ticket和repository模块
@mingshewhe mingshewhe mentioned this issue Jun 23, 2021
Merged
mingshewhe added a commit to mingshewhe/bk-ci that referenced this issue Jun 23, 2021
@mingshewhe
fix: 规范REST接口的业务与系统异常的HTTP状态码 TencentBlueKing#4382
897db3b
mingshewhe added a commit to mingshewhe/bk-ci that referenced this issue Jun 23, 2021
@mingshewhe
fix: 规范REST接口的业务与系统异常的HTTP状态码 TencentBlueKing#4382
3544cc7
@irwinsun irwinsun mentioned this issue Jun 23, 2021
Merged
irwinsun added a commit that referenced this issue Jun 28, 2021
@irwinsun
Merge pull request #4507 from mingshewhe/issue_4382
6cbb87c 
fix: 规范REST接口的业务与系统异常的HTTP状态码 #4382
irwinsun added a commit that referenced this issue Jun 28, 2021
@irwinsun
Merge pull request #4512 from irwinsun/issue_4382
88bfb3b 
fix: 规范REST接口的业务与系统异常的HTTP状态码 #4382
@mingshewhe mingshewhe added the tested 测试环境通过/test patest passed for test stage label Jun 29, 2021
@bkci-bot bkci-bot added the for gray UAT environment in tencent has been deploy label Jun 29, 2021
@irwinsun irwinsun added the grayed uat环境测试通过/test passed for uat stage label Jul 3, 2021
@bkci-bot bkci-bot closed this as completed Jul 8, 2021
@bkci-bot bkci-bot added the done Production environment in tencent has been deploy label Jul 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@irwinsun irwinsun

@mingshewhe mingshewhe

Labels
area/ci/backend CI 后端issue done Production environment in tencent has been deploy for gray UAT environment in tencent has been deploy for test Test environment in tencent has been deploy grayed uat环境测试通过/test passed for uat stage kind/bug 程序故障Bug，漏洞 tested 测试环境通过/test patest passed for test stage
Projects
None yet
Milestone
v1.6
Development

No branches or pull requests
3 participants
@irwinsun @mingshewhe @bkci-bot