Budibase / budibase

Low code platform for building business apps and workflows in minutes. Supports PostgreSQL, MySQL, MariaDB, MSSQL, MongoDB, Rest API, Docker, K8s, and more 🚀
https://budibase.com
Other
21.9k stars 1.51k forks source link

Initial implementation of solving SQS issue with case insensitivity #14038

Closed mike12345567 closed 3 months ago

mike12345567 commented 3 months ago

Description

Fix for issues with columns named after other internal columns, like type or createdAt - these are not supposed to be usable but the check was case sensitive, with SQS this is a problem.

Quite a few instances of columns named things like Type exist, which we have to support, we can't migrate these easily without breaking bindings. This PR introduces a change where all user columns are prefixed as data_... in the SQLite table, meaning they can't overlap with anything.

We already did case sensitivity checks vs other columns in the table, but just didn't handle the columns which are internal to the table.