Skip to content

fix(dify): add GET method support to /dify/retrieval for health check #13837

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

Open
Lntanohuang wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(dify): add GET method support to /dify/retrieval for health check #13837

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions api/apps/sdk/dify_retrieval.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,19 +15,25 @@
#
import logging

from quart import jsonify
from quart import jsonify, request

Check failure on line 18 in api/apps/sdk/dify_retrieval.py

View workflow job for this annotation

GitHub Actions / ragflow_tests

ruff (F401) 

api/apps/sdk/dify_retrieval.py:18:28: F401 `quart.request` imported but unused
  help: Remove unused import: `quart.request`

from api.db.services.document_service import DocumentService
from api.db.services.doc_metadata_service import DocMetadataService
from api.db.services.knowledgebase_service import KnowledgebaseService
from api.db.services.llm_service import LLMBundle
from api.db.joint_services.tenant_model_service import get_model_config_by_id, get_model_config_by_type_and_name, get_tenant_default_model_by_type
from common.metadata_utils import meta_filter, convert_conditions
from api.utils.api_utils import apikey_required, build_error_result, get_request_json, validate_request
from api.utils.api_utils import apikey_required, build_error_result, get_json_result, get_request_json, validate_request
from rag.app.tag import label_question
from common.constants import RetCode, LLMType
from common import settings

@manager.route('/dify/retrieval', methods=['GET']) # noqa: F821
async def retrieval_health_check():
"""Health check endpoint for Dify external knowledge base connectivity verification."""
return get_json_result(data=True)


@manager.route('/dify/retrieval', methods=['POST']) # noqa: F821
@apikey_required
@validate_request("knowledge_id", "query")
Expand Down
Loading