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

the database url is missing ":\\" when sqlite or postgresql is not used #5491

Open
qq745639151 opened this issue Dec 31, 2024 · 2 comments
Open
Labels
bug Something isn't working

Comments

@qq745639151
Copy link

Bug Description

If I set the LANGFLOW_DATABASE_URL start with "mysql+aiomysql://", the function _create_engine split it to judge the database will be used. But if sqlite or postgresql is not used. the url is missing "://"

Reproduction

set the environment variable LANGFLOW_DATABASE_URL with "mysql+aiomysql://"。 then execute cmd python -m langflow run

Expected behavior

langflow works normallly

Who can help?

No response

Operating System

windows

Langflow Version

1.1.1

Python Version

3.11

Screenshot

image

Flow File

No response

@qq745639151 qq745639151 added the bug Something isn't working label Dec 31, 2024
Copy link

dosubot bot commented Dec 31, 2024

The issue you're encountering with the database URL losing "://" when using non-sqlite/postgresql databases like MySQL is related to the logic in the _create_engine function. This function splits the URL and reconstructs it without the "://" for databases other than SQLite and PostgreSQL, which leads to the error you're seeing [1].

To address this, you might need to modify the _create_engine function to ensure that the "://" is preserved for all database types. This involves adjusting the logic that reconstructs the database_url to include "://" for MySQL and other databases.

Additionally, the AttributeError related to the TelemetryService suggests that there might be a missing attribute or misconfiguration in the telemetry setup. This could be a separate issue that needs to be addressed in the TelemetryService implementation.

For further assistance, you might want to check if there are any updates or patches available for Langflow that address these issues.

To continue talking to Dosu, mention @dosu.


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

@liquor96
Copy link

Did you solve the MissingGreenlet: greenlet_spawn has not been called; can't call await_only() here. Was IO attempted in an unexpected placeerror using MySQL?
image

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

No branches or pull requests

2 participants