pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Nathan Bossart <nathan@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Remove pg_authid's TOAST table.
Date: Sat, 21 Sep 2024 20:19:25 +0000
Message-ID: <E1ss6Zg-000Jed-JY@gemulon.postgresql.org> (raw)

Remove pg_authid's TOAST table.

pg_authid's only varlena column is rolpassword, which unfortunately
cannot be de-TOASTed during authentication because we haven't
selected a database yet and cannot read pg_class.  By removing
pg_authid's TOAST table, attempts to set password hashes that
require out-of-line storage will fail with a "row is too big"
error instead.  We may want to provide a more user-friendly error
in the future, but for now let's just remove the useless TOAST
table.

Bumps catversion.

Reported-by: Alexander Lakhin
Reviewed-by: Tom Lane, Michael Paquier
Discussion: https://postgr.es/m/89e8649c-eb74-db25-7945-6d6b23992394%40gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/6aa44060a3c94ee10273bb8a89e98a5bb2fbbacb

Modified Files
--------------
src/backend/catalog/catalog.c              |  4 +---
src/include/catalog/catversion.h           |  2 +-
src/include/catalog/pg_authid.h            |  2 --
src/test/regress/expected/create_index.out | 14 +++++++-------
src/test/regress/expected/misc_sanity.out  |  5 ++++-
src/test/regress/expected/tablespace.out   | 12 ++++++------
src/test/regress/sql/create_index.sql      | 10 +++++-----
src/test/regress/sql/misc_sanity.sql       |  2 ++
src/test/regress/sql/tablespace.sql        |  8 ++++----
9 files changed, 30 insertions(+), 29 deletions(-)



Message-ID: <E1ss6Zg-000Jed-JY@gemulon.postgresql.org>
Permalink:  ../E1ss6Zg-000Jed-JY@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1ss6Zg-000Jed-JY@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: nathan@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Remove pg_authid's TOAST table.
  In-Reply-To: <E1ss6Zg-000Jed-JY@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox