Commit 7e65ad1
committed
Fix dumping role comments when using --no-role-passwords
Commit 9a83d56 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: v121 parent 485f0aa commit 7e65ad1Copy full SHA for 7e65ad1
File tree
Expand file treeCollapse file tree
1 file changed
+10
-7
lines changedOpen diff view settings
Filter options
- src/bin/pg_dump
Expand file treeCollapse file tree
1 file changed
+10
-7
lines changedOpen diff view settings
Collapse file
src/bin/pg_dump/pg_dumpall.c
Copy file name to clipboardExpand all lines: src/bin/pg_dump/pg_dumpall.c+10-7Lines changed: 10 additions & 7 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
769 | 769 | |
770 | 770 | |
771 | 771 | |
772 | | - |
| 772 | + |
| 773 | + |
| 774 | + |
| 775 | + |
773 | 776 | |
774 | 777 | |
775 | 778 | |
776 | 779 | |
777 | 780 | |
778 | 781 | |
779 | | - |
| 782 | + |
780 | 783 | |
781 | 784 | |
782 | 785 | |
783 | | - |
| 786 | + |
784 | 787 | |
785 | 788 | |
786 | 789 | |
787 | 790 | |
788 | 791 | |
789 | 792 | |
790 | | - |
| 793 | + |
791 | 794 | |
792 | 795 | |
793 | | - |
| 796 | + |
794 | 797 | |
795 | 798 | |
796 | 799 | |
797 | 800 | |
798 | 801 | |
799 | 802 | |
800 | 803 | |
801 | | - |
| 804 | + |
802 | 805 | |
803 | 806 | |
804 | | - |
| 807 | + |
805 | 808 | |
806 | 809 | |
807 | 810 | |
|
0 commit comments