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

[FEATURE] Postgres as a vector store support SSL #1085

Closed
primalfear opened this issue Oct 18, 2023 · 4 comments
Closed

[FEATURE] Postgres as a vector store support SSL #1085

primalfear opened this issue Oct 18, 2023 · 4 comments

Comments

@primalfear
Copy link

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.

@primalfear primalfear changed the title Postgres as a vector store support SSL [FEATURE] Postgres as a vector store support SSL Oct 18, 2023
@HenryHengZJ
Copy link
Contributor

can you try this PR here - #837 ?

@primalfear
Copy link
Author

@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".

image

Similar options to what is being done in PR #837 will also need to be applied to the Postgres Vector Store Nodes.

@HenryHengZJ
Copy link
Contributor

HenryHengZJ commented Oct 20, 2023

opened - #1107

PR merged!

@primalfear
Copy link
Author

legend!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants