GHSA-p46v-f2x8-qp98 pREST has a Systemic SQL Injection Vulnerability
SummarypREST provides a simple way for users to expose access their database via a REST-full API. The project is implemented using the Go programming language and is designed to expose access to Postgres database tables.During an independent review of the project, Doyensec engineers found that SQL injection is a systemic problem in the current implementation (version `v2.0.0-rc2`). Even though there are several instances of attempts to sanitize user input and mitigate injection attempts, we have found that on most code-paths, the protection is faulty or non-existent.Core EndpointsThe main functionality providing REST operations on the data stored in the Postgres database is exposed via the following endpoints:`GET /{database}/{schema}/{table}``POST /{database}/{schema}/{table}``PUT|PATCH /{database}/{schema}/{table}``DELETE /{database}/{schema}/{table}`Handlers for the above endpoints execute very similar logic. At a high-level they:Perform authentication and authorizationBuild the SQL query based on the incoming requestExecute the query on the databaseReturn the data to the userThe query construction logic uses data from the request (e.g query, body or path parameters) and incorporates them in the SQL query.As an example, let us look at the `GET` request or the read operation. After completing the authentication and authorization steps, the `SelectFromTables` function will first compile a list of all columns/fields, that will be returned in the HTTP response.The `SelectFields` function will validate the requested columns using the `chkInvalidIdentifier` function, and will ultimately return the beginning of the generated SQL statement. Assuming the request specifies that only the `id` and `task` columns should be returned, the generated SQL will look something like:The next step involves generating the table name, from which the data will be queried.The `SelectSQL` function will receive the `database`, `schema` and `table` values directly from the request and use them to construct the next part of the SQL statement using simple string concatenation.If we assume that the `GET` request is made to the following path `/db001/api/todos`, the resulting query will look similar to:This step performs processing on values, specifically `schema` and `table`, which do not undergo any input validation, and ultimately allow for SQL injection.The description above is only a single instance of this issue. The list below contains code paths that we believe is a comprehensive list of all code paths affected by this issue:https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L243https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L245https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L559https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L643https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1538https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1559https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1581https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1583https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1585https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1601https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1606https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1611https://github.com/prest/prest/blob/main/adapters/postgres/postgres.go#L1616https://github.com/prest/prest/blob/main/controllers/tables.go#L394https://github.com/prest/prest/blob/main/controllers/tables.go#L465ReproductionThe reproduction steps require a working environment which can be set up using the instructions below.With that, the issue can be verified using the following HTTP request:The value provided as the `schema` path parameter contains the injection payload and contains SQL which will be added to the existing SQL statement and will inject a nested query that calls the `pg_sleep()` function, delaying the response by 5 seconds. The statement shown below will be the one that is ultimately executed on the database server.Missing Validation on tsquery PredicatesUsers with permission to read data from tables have the ability to specify `tsquery` predicates, allowing them to perform more complex filtering on the data. An example usage of `tsquery` can be seen below:pREST will parse the request, and if it detects that a `tsquery` needs to be generated, the following code will be executed:In this example, the value of the `value` variable is used directly from the request without any validation, which ultimately allows another path to perform SQL injection.ReproductionThe reproduction steps require a working environment which can be set up using the instructions below.With that, the issue can be verified using make the following HTTP request:As with the previous example, the request above will use Postgres' `pg_sleep()` function to delay the response for 5 seconds, proving the injection was successful.Script TemplatespREST users can define templates for complex SQL queries, that can be reached using the `/_QUERIES/{queriesLocation}/{script}` endpoint. The scripts are read directly from the file system. Their content is passed to the `text/template` Go library, which will render any dynamic data, sourced from the request, directly on to the script template and return the result.The `text/template` library is used to render pure text and does not implement any validation or sanitization functionality out-of-the-box. This allows for yet another path from SQL injection.ReproductionThe reproduction steps require a working environment which can be set up using the instructions below. In addition, the script below should be saved under the `{{project_root}}/_active` path as `get_todo.read.sql`.Before running pREST, make sure the configuration specifies the script template's directory on the root of the project.With that, the issue can be verified by simply making the following request:The `todo_id` value contains the value: `2 OR true` in percent-encoded format. This value will be interpolated in the template and result in the following query being executed:This will ultimately return all values in from the target table.Issues with the Current ValidationpREST implements input validation via the `chkInvalidIdentifier` function, with an attempt to mitigate potential SQL injection attacks. The function will verify that a supplied variable contains only characters from a pre-defined allow list. In addition, the performed validation makes sure that the number of double quotes (`"`) in the validated value are divisible by 2, with the goal of preventing the user to escape the context of a Postgres identifier.The quotation validation logic ultimately proves to be faulty, and can also be abused to perform injection attacks. Namely, Postgres' SQL parser allows identifiers to be enclosed in double-quotes, which acts as a soft of field separator. This enables the construction of queries without any spaces. Combined with the set of allowed characters by the `chkInvalidIdentifier` function, the following request can be made to the server:The request will ultimately execute the following SQL query:The nested `SELECT` statement will impact the output returned to the user. If the nested query evaluates to `true`, the user will see all entries in the `todos` table. On the other hand, if the nested query evaluates to `false`, the user will only see the entry with its `id` column set to `1`.This injection path is ultimately limited by the validation preformed in `chkInvalidIdentifier`, which limits the size of identifiers to 62 characters.Impact**Critical**. Executing arbitrary commands on the database can allow for unauthorized access and modification of the data stored. Additionally, feature-rich database engines such as Postgres allow access to files stored on the underlining file-system, and may even allow for arbitrary command execution.In pREST's case, the query generation procedure will invoke the `Prepare` function from the `sqlx` ORM, which prevents using stacked queries, also preventing execution of arbitrary operations.However, nested queries and file access operations can be performed. The request shown below will read and return the contents of the `/etc/passwd` file.Note that using forward slashes (`/`) will brake the path parsing performed by the API server. That limitation can be bypassed by specifying the forward slash using `CHR(47)`. This technique can be used to read environment variables, which often contain sensitive information such as API keys, or read other sensitive files such as SSH private keys or Postgres-specific certificates used for host-based authentication.Nested queries can be used to access information from internal Postgres tables. The example below will retrieve the password hash of the current Postgres user.Finally, the pREST's official [Docker container ](https://hub.docker.com/r/prest/prest/) uses with the `prest` user the database to establish the database connection. This user does have "superuser" permissions, which increases the likelihood of users running pREST with overly permissioned database users which in turn exposes them to the attacks described above.Complexity**Low**. With access to a running instance, basic web application security knowledge is required to find and exploit this issue. Furthermore, the pREST project is open source, removing any guess work that a potentially attacker might need to do if they were attacking an unknown system.RemediationThe injection proved to be systemic and impacts the majority of the exposed endpoint. We recommend overhauling how dynamic query generation is implemented. Unfortunately, the used `sqlx` library does not appear allow database identifiers to be parametrized, which is a core feature of pREST. This means that validation needs to be perform manually.Start off by preventing all string concatenation operations that use unvalidated or unsanitized user input. All user-controllable values that represent database identifiers (e.g. database and table names) should only contain alpha-numeric characters and optionally dashed (`-`) and underscores (`_`).Also consider removing the double-quote from the list of allowed character when performing validation and make sure they are placed in the correct position on the server-side. This will prevent the limited injection mentioned above.Finally, consider updating how query scripts are created and processed. One way of doing this is by recommending the users to write scripts in a parametrized form. pREST can then read the script from the disk and build a parametrized query using `sqlx`. Any dynamic parameters can be read from the request object and set on the query object. In this implementation, escaping user-controlled values will be handled by the library itself.It is worth noting that the injection issue was pointed out by [GHSA-wm25-j4gw-6vr3](https://github.com/prest/prest/security/advisories/GHSA-wm25-j4gw-6vr3). However, the submitter did not highlight the impact which is likely why the issue was left unpatched.Reproduction Environment SetupThe base environment used to verify the existence of the vulnerability uses a running instance of the deploying the official pREST [Docker container](https://hub.docker.com/r/prest/prest/). For simplicity, all reproduction steps assume that JWT-based authentication is disabled.The database contains one table under the `api` namespace, named `todos` with the following schema:pREST can be ran using the following configuration:
CVSS v4.0 score: 9.3