Skip to content

fix: 修复网关资源配置错误 --story=130003551#744

Merged
guohelu merged 1 commit into
TencentBlueKing:developfrom
guohelu:develop_20
May 22, 2026
Merged

fix: 修复网关资源配置错误 --story=130003551#744
guohelu merged 1 commit into
TencentBlueKing:developfrom
guohelu:develop_20

Conversation

@guohelu

@guohelu guohelu commented May 22, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Summary

变更验证通过。api-resources.yml/sdk/api/plugin_service/logs/ 的 HTTP 方法从 get 修正为 post,与实际视图定义 plugin_service/api.py:236 中的 @api_view(["POST"]) 一致。

  • 网关资源定义与代码实现对齐 ✅
  • API 文档 (sdk_get_plugin_service_logs.md) 无需同步更新(未显式声明 HTTP 方法)✅
  • apigw-docs.zip 无内容变更,无需重新生成 ✅

LGTM,无问题。

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.87%. Comparing base (ea70462) to head (18365ec).
⚠️ Report is 27 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #744      +/-   ##
===========================================
+ Coverage    82.14%   82.87%   +0.72%     
===========================================
  Files          296      296              
  Lines        17925    17963      +38     
===========================================
+ Hits         14725    14886     +161     
+ Misses        3200     3077     -123     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@guohelu guohelu merged commit 5ea5c40 into TencentBlueKing:develop May 22, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants