Hi team, Each node has a validationErrors property which contains either the name of the property in error or a validation error message. It seems that config-nodes do not use this validationErrors property if the validation comes from a credential, @knolleary is this a limitation? Is it also poss…

Hi @knolleary , I might be wrong but I feel like there is a bug here: // editor-client -> editor.js // validateNode function - if (node._def._creds) { - validationErrors = validationErrors.concat(validateNodeProperties(node, node._def.credentials, node._def._creds)) - } + if (node.credentials) …

[image] GogoVega: also possible to show that a config node is in error as the following issue suggests? As the issue states, it is not currently possible to flag config nodes as have errors. They simply don't get handled in all the places they should. I haven't started looking at it yet. It…

[image] knolleary: it is not currently possible to flag config nodes as have errors This could be the subject of a new functionality :slightly_smiling_face: What interests me most is being able to get node.validationErrors. And for context :grin: node-red/node-red/blob/fd32ee09ffbc0…

[image] GogoVega: This could be the subject of a new functionality Yes. That is why I asked Joe to raise the issue in the first place.

Hi @knolleary , Do you confirm the issue? I see no other reference to node._def._creds :thinking:

Hi Team, @knolleary , after delving deeper into the subject, I noticed: A validation of node._def._creds is done but what is it? The credentials (node.credentials) are not validated. The reason for the last one seems logical... can I suggest validating the defined properties since they are there: …

Please look at this :slightly_smiling_face: Found this: feature: Allow credentials to be marked as required as part of validation · Issue #3420 · node-red/node-red · GitHub

I forgot to mention it: The problem is that the node credentials are only loaded when the node edit box is opened. Which implies that when loading the workspace the credentials cannot be validated. Do we leave it like this or do we add the loading of credentials to the loading of the workspace? Of…

Node-RED

Node-RED Forum

FR: validationErrors for config-node credentials

Core Development Feature Requests

knolleary 30 November 2023 18:56 5

Yes. That is why I asked Joe to raise the issue in the first place.

2 Likes

Topic		Replies	Views
How can I raise error at 'oneditsave' when my credential is empty? (show red triangle sign) Developing Nodes	13	769	20 February 2022
Client JS in the node config editor? Developing Nodes	6	214	21 May 2023
[Proposal] Make node property validation tooltip more understandable Core Development	12	726	5 April 2022
Config node not showing Developing Nodes	15	356	28 December 2022
Credentials undefined General	7	1134	28 March 2019

Copyright OpenJS Foundation and Node-RED contributors. All rights reserved. The OpenJS Foundation has registered trademarks and uses trademarks. For a list of trademarks of the OpenJS Foundation, please see our Trademark Policy and Trademark List. Trademarks and logos not indicated on the list of OpenJS Foundation trademarks are trademarks™ or registered® trademarks of their respective holders. Use of them does not imply any affiliation with or endorsement by them.

FR: validationErrors for config-node credentials

Related topics