FlowFuse / FlowFuse/nr-tables-nodes

Promote use of parameterised queries by making interface more low-code

Open
#22 1 comment 0 reactions 0 assignees View on GitHub
task
Dominant language
JavaScript
Stars
0
Forks
0
Avg merge
2d 19h
Merged PRs (30d)
1

Description

### Description

> ### Best Practice
> The inline codelens support for SQL editor is great and all, but in this implementation, it is not easy to promote the use of parameterized queries for avoiding SQLi. If we devlop the `nr-tables-nodes` to be more _low-code friendly_ (like the MSSQL-PLUS nodes) where the source of a parameter is defined in the UI, we could pre-populate parameters for prompts like "Update the users table and set the expired property to null when the users ID is equal to the incoming payload number"
>

Originally raised in https://github.com/FlowFuse/nr-assistant/issues/66#issuecomment-3200076483

### Epic/Story

_No response_

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by comparing the nr-tables-nodes SQL editor and inline codelens behavior with the referenced MSSQL-PLUS nodes. Define how a UI-selected parameter source would represent the incoming payload value and verify that the described users-table update can use a parameterized query without inline values.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, sql
Domain
databases, security
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.