Skip to content

fix: ZIP files always use forward slashes / regardless of the operati…#616

Merged
fatelei merged 1 commit into
langgenius:mainfrom
fatelei:issue-32650
Feb 27, 2026
Merged

fix: ZIP files always use forward slashes / regardless of the operati…#616
fatelei merged 1 commit into
langgenius:mainfrom
fatelei:issue-32650

Conversation

@fatelei

@fatelei fatelei commented Feb 27, 2026

Copy link
Copy Markdown
Contributor

…ng system

Description

fix langgenius/dify#32650

Root Cause: In pkg/bundle_packager/memory_zip.go, the code used filepath.Separator (which is \ on Windows) to check for the _assets/ directory prefix in ZIP file paths. ZIP files always use forward slashes / regardless of the operating system, causing the asset detection to fail on Windows.

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • Other

Essential Checklist

Testing

  • I have tested the changes locally and confirmed they work as expected
  • I have added unit tests where necessary and they pass successfully

Bug Fix (if applicable)

  • I have used GitHub syntax to close the related issue (e.g., Fixes #123 or Closes #123)

Additional Information

Please provide any additional context that would help reviewers understand the changes.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working Windows labels Feb 27, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Feb 27, 2026
@fatelei fatelei merged commit 72288c3 into langgenius:main Feb 27, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files. Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When I install the model provider, an error occurs

2 participants