-
-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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
[FEATURE] Postgres as a vector store support SSL #1085
Comments
can you try this PR here - #837 ? |
@HenryHengZJ, I did have a look at that PR, which will solve the issue for the flowise application, to use postgres as its datastore. However this PR doesn't address the issue for the postgres vector store nodes, used within a "chat flow". Similar options to what is being done in PR #837 will also need to be applied to the Postgres Vector Store Nodes. |
opened - #1107 PR merged! |
legend!!! |
Hi Guys,
Firstly, thanks this is a great project, the work your are doing is fantastic, and the rate at which you are rolling out new features is admirable!
When using a Hosted / Managed Postgres database with providers like Digital Ocean, they require ssl-mode to be turned on.
There is already a pull request to solve this issue for the main flowise application feature/pg sslmode I look forward to this being merged and released 😃
however this change will not solve the issue, on the "Postgres Upsert Document" or "Postgres Load Existing Index" vector store options.
Suggestion
Add options in the additional parameters section to allow for setting the SSL Mode, as well as adding the providers ca-cert and/or the option to set: "rejectUnauthorized: false"
See: stack overflow: cannot connect an SSL secured database to typeorm
If this is something that will be super low priority / not really something you guys are keen to look at.
I may be able to work it out, my node skills are a little rusty, but I have more or less worked out where to make the required changes, so might be able to submit a PR, though will need to setup my machine for node development 😝
I look forward to your thoughts on this.
The text was updated successfully, but these errors were encountered: