Skip to content

fix: fix plugin miss version and checksum#36083

Merged
fatelei merged 2 commits into
langgenius:mainfrom
fatelei:issue-36081
May 13, 2026
Merged

fix: fix plugin miss version and checksum#36083
fatelei merged 2 commits into
langgenius:mainfrom
fatelei:issue-36081

Conversation

@fatelei
Copy link
Copy Markdown
Contributor

@fatelei fatelei commented May 12, 2026

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

fix #36081

when call https://marketplace.dify.ai/api/v1/plugins/download?unique_identifier=langgenius/deepseek

it need unique_identifier it author/plugin:version@checksum format, but currently it miss version@checksum, so it will fail {"code":-1,"data":null,"msg":"plugin unique identifier is invalid"}

and call

2026-05-12 11:39:46,89 ERROR [base.py:242] 787f6b0685 Failed to request plugin daemon, status: 400, url: plugin/f067162a-a141-4128-9836-ee2ad3623e94/management/fetch/manifest
Traceback (most recent call last):
  File "/Users/fatelei/dify/dify/api/core/plugin/impl/base.py", line 240, in _request_with_plugin_daemon_response
    response.raise_for_status()
  File "/Users/fatelei/dify/dify/api/.venv/lib/python3.12/site-packages/httpx/_models.py", line 829, in raise_for_status
    raise HTTPStatusError(message, request=request, response=self)
httpx.HTTPStatusError: Client error '400 Bad Request' for url 'http://127.0.0.1:10000/plugin/f067162a-a141-4128-9836-ee2ad3623e94/management/fetch/manifest?plugin_unique_identifier=langgenius%2Fdeepseek'

latest_package_identifier is the right format

image

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran make lint && make type-check (backend) and cd web && pnpm exec vp staged (frontend) to appease the lint gods

@fatelei fatelei requested a review from iamjoel as a code owner May 12, 2026 11:59
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label May 12, 2026
@github-actions github-actions Bot added the web This relates to changes on the web. label May 12, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 13, 2026
@fatelei fatelei added this pull request to the merge queue May 13, 2026
Merged via the queue into langgenius:main with commit 7e56a24 May 13, 2026
28 checks passed
@fatelei fatelei deleted the issue-36081 branch May 13, 2026 01:46
zhangtaodemama added a commit to zhangtaodemama/langgenius-dify-bfaadcb0c706 that referenced this pull request May 13, 2026
BeautyyuYanli pushed a commit that referenced this pull request May 13, 2026
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files. web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bundle with marketplace/github dependencies fails to install despite 200 OK response

3 participants