Skip to content

Store PostgreSQL user and password in Node-RED credentials#136

Open
mister13337 wants to merge 1 commit into
alexandrainst:mainfrom
mister13337:main
Open

Store PostgreSQL user and password in Node-RED credentials#136
mister13337 wants to merge 1 commit into
alexandrainst:mainfrom
mister13337:main

Conversation

@mister13337
Copy link
Copy Markdown

This updates the PostgreSQL config node to store user and password in Node-RED credentials instead of regular flow properties.

Note that this removes the Env/Global options for providing user and password.

@mister13337 mister13337 force-pushed the main branch 2 times, most recently from ead8269 to 61185b2 Compare April 23, 2026 15:39
Comment thread package.json
{
"name": "node-red-contrib-postgresql",
"version": "0.15.4",
"version": "0.15.5",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"version": "0.15.5",
"version": "0.16.0",

Comment thread postgresql.html
},
label: function () {
return this.name || this.user + '@' + this.host + ':' + this.port + '/' + this.database;
return this.name || this.host + ':' + this.port + '/' + this.database;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn’t it be good to keep the user in this information?

Comment thread postgresql.js
Comment on lines +77 to +78
user: node.credentials.user,
password: node.credentials.password,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you think of a way to fall back to the existing credentials, in order to support a transition for the existing users?

@Alkarex
Copy link
Copy Markdown
Member

Alkarex commented Apr 24, 2026

Thanks for the PR!
I have made a few comments higher up

@Alkarex
Copy link
Copy Markdown
Member

Alkarex commented May 25, 2026

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

Successfully merging this pull request may close these issues.

2 participants