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: 修复日志相关 Sentry 异常 #1579

Merged
merged 4 commits into from
Sep 9, 2024
Merged

fix: 修复日志相关 Sentry 异常 #1579

merged 4 commits into from
Sep 9, 2024

Conversation

jiayuan929
Copy link
Collaborator

  • 限制日志查询的最大窗口，避免查询 10,000 条以上的日志报错
  • 修复插件查询日志时，未初始化配置导致异常问题
  • 用户输入语法错误，API 返回错误码，记录错误日志即可，不抛 500 异常
  • 日志平台 API 返回 result=false，记录错误日志，返回错误码，避免后续抛出 500 异常
  • 修复插件查询日志时未指定主模块导致异常问题

以下是符合预期的异常，不用处理

  • 查询标准输出日志时，scroll id 失效等导致 ES 查询异常
  • 调用 bklog API 504 、连接超时等

@jiayuan929 jiayuan929 merged commit 6e84722 into TencentBlueKing:main Sep 9, 2024
7 checks passed
songzxc789 pushed a commit to songzxc789/blueking-paas that referenced this pull request Sep 24, 2024
@jiayuan929 @songzxc789
fix: 修复日志相关 Sentry 异常 (TencentBlueKing#1579)
ff89578
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SheepSheepChen SheepSheepChen SheepSheepChen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jiayuan929 @SheepSheepChen