Fix dumping role comments when using --no-role-passwords
authorDaniel Gustafsson <dgustafsson@postgresql.org>
Thu, 21 Mar 2024 22:31:57 +0000 (23:31 +0100)
committerDaniel Gustafsson <dgustafsson@postgresql.org>
Thu, 21 Mar 2024 22:31:57 +0000 (23:31 +0100)
commitd82cb467bbc292375283b40f5cc65868024c82b4
tree8cbb5ba6523181b0560e295513592cda05f33006
parent58efabdc0ce932c9e927d8616f25e0e1f651b601
Fix dumping role comments when using --no-role-passwords

Commit 9a83d56b38c added support for allowing pg_dumpall to dump
roles without including passwords, which accidentally made dumps
omit COMMENTs on roles.  This fixes it by using pg_authid to get
the comment.

Backpatch to all supported versions. Patch simultaneously written
independently by Γlvaro and myself.

Author: Γlvaro Herrera <alvherre@alvh.no-ip.org>
Author: Daniel Gustafsson <daniel@yesql.se>
Reported-by: Bartosz ChroΕ‚ <bartosz.chrol@handen.pl>
Discussion: https://postgr.es/m/AS8P194MB1271CDA0ADCA7B75FCD8E767F7332@AS8P194MB1271.EURP194.PROD.OUTLOOK.COM
Discussion: https://postgr.es/m/CAEP4nAz9V4H41_4ESJd1Gf0v%3DdevkqO1%3Dpo91jUw-GJSx8Hxqg%40mail.gmail.com
Backpatch-through: v12
src/bin/pg_dump/pg_dumpall.c