diff options
| author | Noah Misch <noah@leadboat.com> | 2025-05-01 16:51:59 -0700 |
|---|---|---|
| committer | Noah Misch <noah@leadboat.com> | 2025-05-01 16:51:59 -0700 |
| commit | c6a26e4ccd1bf090ab755df2f597103798284d3a (patch) | |
| tree | 892711e95bac21c274b989172063f44845a7aa94 | |
| parent | 0064020680c1293ec6ce9fe2588e33d6aebdb015 (diff) | |
| download | postgresql-c6a26e4ccd1bf090ab755df2f597103798284d3a.tar.gz postgresql-c6a26e4ccd1bf090ab755df2f597103798284d3a.zip | |
Doc: stop implying recommendation of insecure search_path value.
SQL "SET search_path = 'pg_catalog, pg_temp'" is silently equivalent to
"SET search_path = pg_temp, pg_catalog, "pg_catalog, pg_temp"" instead
of the intended "SET search_path = pg_catalog, pg_temp". (The intent
was a two-element search path. With the single quotes, it instead
specifies one element with a comma and a space in the middle of the
element.) In addition to the SET statement, this affects SET clauses of
CREATE FUNCTION, ALTER ROLE, and ALTER DATABASE. It does not affect the
set_config() SQL function.
Though the documentation did not show an insecure command, remove single
quotes that could entice a reader to write an insecure command.
Back-patch to v13 (all supported versions).
Reported-by: Sven Klemm <sven@timescale.com>
Author: Sven Klemm <sven@timescale.com>
Backpatch-through: 13
| -rw-r--r-- | doc/src/sgml/extend.sgml | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/doc/src/sgml/extend.sgml b/doc/src/sgml/extend.sgml index 4915c1985e0..065bc49c973 100644 --- a/doc/src/sgml/extend.sgml +++ b/doc/src/sgml/extend.sgml @@ -1344,8 +1344,8 @@ SELECT * FROM pg_extension_update_paths('<replaceable>extension_name</replaceabl secure <varname>search_path</varname>; do <emphasis>not</emphasis> trust the path provided by <command>CREATE/ALTER EXTENSION</command> to be secure. Best practice is to temporarily - set <varname>search_path</varname> to <literal>'pg_catalog, - pg_temp'</literal> and insert references to the extension's + set <varname>search_path</varname> to <literal>pg_catalog, + pg_temp</literal> and insert references to the extension's installation schema explicitly where needed. (This practice might also be helpful for creating views.) Examples can be found in the <filename>contrib</filename> modules in |