| 961 |
User-defined routines that have the same name as some system-defined routine. |
Avoid creating user-defined routines that have the same name as some system-defined routine because it may cause confusion. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-02-25 17:30 |
MIT License |
View |
| 962 |
User-defined routines that implement UPSERT operation |
Find user-defioned routines that implement UPSERT operation. Make sure that it is consistent with the contracts of database operations. |
General |
INFORMATION_SCHEMA+system catalog base tables |
2021-10-25 17:02 |
MIT License |
View |
| 963 |
User-defined routines that produce a temporary table |
Find user user-defined routines that produce a temporary table |
General |
INFORMATION_SCHEMA+system catalog base tables |
2021-10-25 16:25 |
MIT License |
View |
| 964 |
User-defined routines that read data |
Find user-defined routines that contain SELECT … FROM or PERFORM … FROM operations. PostgreSQL uses multiversion concurrency control (MVCC). Therefore, SELECTs do not block modifications and vice versa. One has to take steps to achieve correct behaviour of data access code. In addition, one should not ask data with multiple queries if it is possible to achieve the result with only one query. |
General |
INFORMATION_SCHEMA+system catalog base tables |
2023-12-21 15:00 |
MIT License |
View |
| 965 |
User-defined routines that use dynamic SQL to execute data manipulation statements |
Find user-defined routines that use dynamic SQL to execute data manipulation statements (SELECT, INSERT, UPDATE, DELETE). |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-11-04 12:39 |
MIT License |
View |
| 966 |
User-defined routines that use keyword DECLARE but do not declare anything |
Find user-defined routines that use keyword DECLARE but do not declare anything. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-11-05 14:47 |
MIT License |
View |
| 967 |
User-defined routines that use md5 hash for other purposes than generating test data |
Find user-defined routines that use md5 hashes for the security purposes. Nowadays such hashes can be calculated too quickly and its use should be avoided at least for hashing passwords. Exclude routines that invoke both md5 function and generate_series function and are thus probably used to generate test data. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-10-31 15:23 |
MIT License |
View |
| 968 |
User-defined routines that use positional references to parameters |
Use parameter names instead of positional references to improve code evolvability. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-11-05 13:18 |
MIT License |
View |
| 969 |
User-defined routines that use xmin hidden column |
Find routines that contain a UPDATE or a DELETE statement that search condition refers to the xmin column. If one uses optimistic approach for dealing with the concurrent modifications of data, then xmin values should be presented by views and used in routines that modify or delete rows. |
General |
INFORMATION_SCHEMA+system catalog base tables |
2021-11-04 11:30 |
MIT License |
View |
| 970 |
User-defined routines with dynamic SQL |
Find routines that use dynamic SQL. Make sure that dynamic SQL is indeed needed, i.e., the task cannot be solved with static SQL. Make sure that the routine is protected against attacks that use SQL injection method. |
General |
INFORMATION_SCHEMA+system catalog base tables |
2021-11-04 11:09 |
MIT License |
View |
| 971 |
User-defined routines with dynamic SQL that are potential targets of the SQL injection attack |
Find routines that have at least one input parameter, use dynamic SQL but do not escape the input arguments at all. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-11-04 11:58 |
MIT License |
View |
| 972 |
User-defined routines with the same parameters (same name and type) regardless of the order of parameters |
Find routines with the same parameters (same name and type) regardless of the order of parameters. Make sure that there is no accidental duplication. The query helps users to group together routines that probably have related tasks. |
General |
INFORMATION_SCHEMA+system catalog base tables |
2020-11-06 14:51 |
MIT License |
View |
| 973 |
Username is not unique |
Find textual columns that potentially contain usernames (including columns that potentially contain e-mail addresses) that do not have a unique constraint or a unique index that involves only this column. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-02-25 17:29 |
MIT License |
View |
| 974 |
Using AFTER triggers to enforce constraints |
Do not let the system to do extra work. Checking a constraint with an AFTER trigger means that the trigger procedure will be executed after the data modification and if the check fails, then the system has to do extra work to roll back the changes. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-02-25 17:29 |
MIT License |
View |
| 975 |
Using an internal data type - name |
Find base table columns that use type name that is used in system catalog tables. It is not a problem if the column is meant for recording identifiers of database objects. |
General |
INFORMATION_SCHEMA only |
2021-03-30 13:36 |
MIT License |
View |
| 976 |
Using BEFORE triggers to log data changes |
Do not let the system to do extra work. Logging changes with a BEFORE trigger means extra work for rolling back the changes in case the logged data modification fails. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-02-25 17:29 |
MIT License |
View |
| 977 |
Using conditionals to determine the returned value |
Use SQL language instead of PL/pgSQL where possible. Instead of using an IF statement, you can check as to whether the data modification succeeded or not by using the RETURNING clause in the data modification statement. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-11-04 11:52 |
MIT License |
View |
| 978 |
Using in some way reserved (in PostgreSQL) SQL keywords as the names of a database object (aggregate view) |
"Names in software are 90 percent of what make software readable. You need to take the time to choose them wisely and keep them relevant. Names are too important to treat carelessly. Names should not cause confusion." (Robert C. Martin, Clean Code) Names should not cause confusion. Find the distinct names (identifiers) of user-defined objects that are SQL keywords that are not completely unreserved in PostgreSQL, i.e., these either never cannot be used as regular identifiers or cannot be used in case of some type of database objects. In PostgreSQL "there are several different classes of tokens ranging from those that can never be used as an identifier to those that have absolutely no special status in the parser as compared to an ordinary identifier. " (PostgreSQL manual) Moreover, such identifiers are often too general, i.e., do not provide enough information about the named object. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2024-12-21 17:17 |
MIT License |
View |
| 979 |
Using in some way reserved (in PostgreSQL) SQL keywords as the names of a database object (detailed view) |
"Names in software are 90 percent of what make software readable. You need to take the time to choose them wisely and keep them relevant. Names are too important to treat carelessly. Names should not cause confusion." (Robert C. Martin, Clean Code) Names should not cause confusion. Find the names (identifiers) of user-defined objects that are SQL keywords that are not completely unreserved in PostgreSQL, i.e., these either never cannot be used as regular identifiers or cannot be used in case of some type of database objects. In PostgreSQL "there are several different classes of tokens ranging from those that can never be used as an identifier to those that have absolutely no special status in the parser as compared to an ordinary identifier. " (PostgreSQL manual) Moreover, such identifiers are often too general, i.e., do not provide enough information about the named object. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2024-12-21 17:21 |
MIT License |
View |
| 980 |
Using routine name in front of a parameter name in a routine body to refer to the parameter of the routine |
Write code that is easy to understand and not unnecessarily long. A routine cannot have two or more parameters with the same name. In this case using longer identifier in the form routine_name.parameter name is unnecessary. |
Problem detection |
INFORMATION_SCHEMA+system catalog base tables |
2021-11-05 13:40 |
MIT License |
View |