MIME-Version: 1.0
From: Nisha Moond <nisha.moond412@gmail.com>
Date: Mon, 1 Jun 2026 19:14:12 +0530
Message-ID: 
 <CABdArM4uA=6nA0BunJwudiEoY1BcWUS_oj_2pkEq_d-YdiBJhw@mail.gmail.com>
Subject: Fix column privileges for pg_subscription.subwalrcvtimeout
To: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>,
 Fujii Masao <masao.fujii@gmail.com>
Content-Type: multipart/mixed; boundary="00000000000080222a0653316477"
Archived-At: 
 <https://www.postgresql.org/message-id/CABdArM4uA%3D6nA0BunJwudiEoY1BcWUS_oj_2pkEq_d-YdiBJhw%40mail.gmail.com>
Precedence: bulk

--00000000000080222a0653316477
Content-Type: text/plain; charset="UTF-8"

Hi Hackers,

IIUC, all columns of pg_subscription, except subconninfo, are intended
to be readable by non-superusers as well. A comment in
system_views.sql also states:
"-- All columns of pg_subscription except subconninfo are publicly readable."

However, 'subwalrcvtimeout' is currently not accessible:
Test:
postgres=# CREATE ROLE nisha LOGIN PASSWORD 'testpass';
CREATE ROLE
postgres=# SET SESSION AUTHORIZATION nisha;
SET
postgres=> select subwalrcvtimeout from pg_subscription;
ERROR:  permission denied for table pg_subscription

It appears the column-level privileges for pg_subscription were not
updated when subwalrcvtimeout was added.

Attached is a small fix patch to grant public access to this column,
consistent with the existing behavior of the other pg_subscription
columns.

CC: Fujii-san (subwalrcvtimeout was introduced by commit fb80f38).

--
Thanks,
Nisha

--00000000000080222a0653316477
Content-Type: application/octet-stream;
	name="v1-0001-Fix-pg_subscription-column-privileges-for-subwalr.patch"
Content-Disposition: attachment;
	filename="v1-0001-Fix-pg_subscription-column-privileges-for-subwalr.patch"
Content-Transfer-Encoding: base64
Content-ID: <f_mpv98cjz0>
X-Attachment-Id: f_mpv98cjz0

RnJvbSBjMDMyOWMzNDRkNGJlNTJmMGM5MzFmZDEzYWMzZTFhMDJjMTNjZDdhIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBOaXNoYSBNb29uZCA8bmlzaGEubW9vbmQ0MTJAZ21haWwuY29t
PgpEYXRlOiBNb24sIDEgSnVuIDIwMjYgMTg6NDM6NDIgKzA1MzAKU3ViamVjdDogW1BBVENIIHYx
XSBGaXggcGdfc3Vic2NyaXB0aW9uIGNvbHVtbiBwcml2aWxlZ2VzIGZvciBzdWJ3YWxyY3Z0aW1l
b3V0CgpUaGUgc3Vid2FscmN2dGltZW91dCBjb2x1bW4gd2FzIGFkZGVkIGJ5IGNvbW1pdCBmYjgw
ZjM4LCBidXQgdGhlCmNvbHVtbi1sZXZlbCBwcml2aWxlZ2VzIG9uIHBnX3N1YnNjcmlwdGlvbiB3
ZXJlIG5vdCB1cGRhdGVkLiBBcyBhCnJlc3VsdCwgbm9uLXN1cGVydXNlcnMgY2Fubm90IHJlYWQg
dGhlIGNvbHVtbiwgdW5saWtlIHRoZSBvdGhlcgpwdWJsaWNseSByZWFkYWJsZSBwZ19zdWJzY3Jp
cHRpb24gY29sdW1ucy4KClRoaXMgcGF0Y2ggZ3JhbnQgU0VMRUNUIHByaXZpbGVnZSBvbiBzdWJ3
YWxyY3Z0aW1lb3V0IHRvIFBVQkxJQy4KLS0tCiBzcmMvYmFja2VuZC9jYXRhbG9nL3N5c3RlbV92
aWV3cy5zcWwgfCAzICsrLQogMSBmaWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygrKSwgMSBkZWxl
dGlvbigtKQoKZGlmZiAtLWdpdCBhL3NyYy9iYWNrZW5kL2NhdGFsb2cvc3lzdGVtX3ZpZXdzLnNx
bCBiL3NyYy9iYWNrZW5kL2NhdGFsb2cvc3lzdGVtX3ZpZXdzLnNxbAppbmRleCA3M2ExYzFjNDY3
MC4uOGYxMjliYWVjOTAgMTAwNjQ0Ci0tLSBhL3NyYy9iYWNrZW5kL2NhdGFsb2cvc3lzdGVtX3Zp
ZXdzLnNxbAorKysgYi9zcmMvYmFja2VuZC9jYXRhbG9nL3N5c3RlbV92aWV3cy5zcWwKQEAgLTE1
MjcsNyArMTUyNyw4IEBAIEdSQU5UIFNFTEVDVCAob2lkLCBzdWJkYmlkLCBzdWJza2lwbHNuLCBz
dWJuYW1lLCBzdWJvd25lciwgc3ViZW5hYmxlZCwKICAgICAgICAgICAgICAgc3ViYmluYXJ5LCBz
dWJzdHJlYW0sIHN1YnR3b3BoYXNlc3RhdGUsIHN1YmRpc2FibGVvbmVyciwKIAkJCSAgc3VicGFz
c3dvcmRyZXF1aXJlZCwgc3VicnVuYXNvd25lciwgc3ViZmFpbG92ZXIsCiAgICAgICAgICAgICAg
IHN1YnJldGFpbmRlYWR0dXBsZXMsIHN1Ym1heHJldGVudGlvbiwgc3VicmV0ZW50aW9uYWN0aXZl
LAotICAgICAgICAgICAgICBzdWJzZXJ2ZXIsIHN1YnNsb3RuYW1lLCBzdWJzeW5jY29tbWl0LCBz
dWJwdWJsaWNhdGlvbnMsIHN1Ym9yaWdpbikKKyAgICAgICAgICAgICAgc3Vic2VydmVyLCBzdWJz
bG90bmFtZSwgc3Vic3luY2NvbW1pdCwgc3Vid2FscmN2dGltZW91dCwKKyAgICAgICAgICAgICAg
c3VicHVibGljYXRpb25zLCBzdWJvcmlnaW4pCiAgICAgT04gcGdfc3Vic2NyaXB0aW9uIFRPIHB1
YmxpYzsKIAogQ1JFQVRFIFZJRVcgcGdfc3RhdF9zdWJzY3JpcHRpb25fc3RhdHMgQVMKLS0gCjIu
NTAuMSAoQXBwbGUgR2l0LTE1NSkKCg==
--00000000000080222a0653316477--