Skip to content

fix: Add authorization check for knowledge_delete node#1865

Merged
lvxinyu-1117 merged 1 commit intomainfrom
fix/wf/knowledge_delete
Aug 29, 2025
Merged

fix: Add authorization check for knowledge_delete node#1865
lvxinyu-1117 merged 1 commit intomainfrom
fix/wf/knowledge_delete

Conversation

@lvxinyu-1117
Copy link
Copy Markdown
Collaborator

What type of PR is this?

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Add documentation if the current PR requires user awareness at the usage level.

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en: Add authorization check for knowledge_delete node
zh(optional): 知识库删除节点增加文档是否属于所选知识库的鉴权校验

(Optional) Which issue(s) this PR fixes:

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 37.14286% with 22 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
backend/crossdomain/impl/knowledge/knowledge.go 0.00% 13 Missing ⚠️
...flow/internal/nodes/knowledge/knowledge_deleter.go 59.09% 6 Missing and 3 partials ⚠️
Files with missing lines Coverage Δ
...ckend/api/model/crossdomain/knowledge/knowledge.go 0.00% <ø> (ø)
...flow/internal/nodes/knowledge/knowledge_deleter.go 60.37% <59.09%> (ø)
backend/crossdomain/impl/knowledge/knowledge.go 0.00% <0.00%> (ø)

... and 1873 files with indirect coverage changes

🚀 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.

@lvxinyu-1117 lvxinyu-1117 added this pull request to the merge queue Aug 29, 2025
Merged via the queue into main with commit a1da673 Aug 29, 2025
10 checks passed
@lvxinyu-1117 lvxinyu-1117 deleted the fix/wf/knowledge_delete branch August 29, 2025 03:54
futeng pushed a commit to futeng/coze-studio that referenced this pull request Sep 7, 2025
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.

3 participants