Skip to content
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.

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

Relative WS path for comfyui #13061

Open
5 tasks done
bikevit2008 opened this issue Jan 26, 2025 · 1 comment · May be fixed by #13029
Open
5 tasks done

Relative WS path for comfyui #13061

bikevit2008 opened this issue Jan 26, 2025 · 1 comment · May be fixed by #13029
Labels
🐞 bug Something isn't working

Comments

@bikevit2008
Copy link

Self Checks

  • This is only for bug report, if you would like to ask a question, please head to Discussions.
  • I have searched for existing issues search for existing issues, including closed ones.
  • I confirm that I am using English to submit this report (我已阅读并同意 Language Policy).
  • [FOR CHINESE USERS] 请务必使用英文提交 Issue,否则会被关闭。谢谢!:)
  • Please do not modify this template :) and fill in all the required fields.

Dify version

0.15.1

Cloud or Self Hosted

Self Hosted (Docker)

Steps to reproduce

By default now we can use links like abc.com/ but abc.com/foo/bar/comfy is not valid

✔️ Expected Behavior

By default now we can use links like abc.com/ but abc.com/foo/bar/comfy is ok and valid

❌ Actual Behavior

By default now we can use links like abc.com/ but abc.com/foo/bar/comfy is not valid

Copy link

dosubot bot commented Jan 26, 2025

Hello @bikevit2008! I'm here to help you with any bugs, questions, or contributions you have for the project. I'll assist you while we wait for a human maintainer.

The issue you're encountering with URL paths like abc.com/foo/bar/comfy not being valid in your self-hosted Dify setup might be related to the URL path validation logic or configuration settings.

In Dify, URL path validation logic is implemented in the api/tests/unit_tests/libs/test_yarl.py file. This logic ensures correct URL path concatenations and raises a ValueError for invalid paths [1].

For self-hosted Docker environments, you can configure URL handling using environment variables such as CONSOLE_API_URL, CONSOLE_WEB_URL, SERVICE_API_URL, APP_API_URL, APP_WEB_URL, and FILES_URL. These variables allow you to customize how URLs are handled and displayed [2][3].

You might want to check these configurations to ensure they are set correctly for your desired URL paths. If the issue persists, reviewing the URL path validation logic in the codebase might provide further insights.

To continue talking to Dosu, mention @dosu.


Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other

@bikevit2008 bikevit2008 linked a pull request Jan 26, 2025 that will close this issue
5 tasks
@dosubot dosubot bot added the 🐞 bug Something isn't working label Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant