Date: Tue, 3 Jun 2025 13:01:58 -0500
From: Nathan Bossart <nathandbossart@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: pgsql-hackers@postgresql.org
Subject: Re: fix notes about password encryption in pg_authid docs
Message-ID: <aD84lmS0uLheXrk7@nathan>
References: <aD2yKkZro4nbl5ol@nathan>
 <aD2zc-nj_UW5njzN@nathan>
 <aD59Y1IXEDpq_j5C@paquier.xyz>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="oa/oIHxiwZpWg/Ua"
Content-Disposition: inline
In-Reply-To: <aD59Y1IXEDpq_j5C@paquier.xyz>
Archived-At: <https://www.postgresql.org/message-id/aD84lmS0uLheXrk7%40nathan>
Precedence: bulk


--oa/oIHxiwZpWg/Ua
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, Jun 03, 2025 at 01:43:15PM +0900, Michael Paquier wrote:
> You are missing one reference in doc/src/sgml/system-views.sgml for
> pg_shadow.passwd, no?

Yup.  Here is an updated patch.

-- 
nathan

--oa/oIHxiwZpWg/Ua
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment;
	filename=v2-0001-doc-Fix-notes-about-password-encryption-in-pg_aut.patch

From 2d41fa2cff14b548905e2dfdd98b992976137e61 Mon Sep 17 00:00:00 2001
From: Nathan Bossart <nathan@postgresql.org>
Date: Mon, 2 Jun 2025 09:08:24 -0500
Subject: [PATCH v2 1/1] doc: Fix notes about password encryption in pg_authid.

---
 doc/src/sgml/catalogs.sgml     | 7 +------
 doc/src/sgml/system-views.sgml | 2 +-
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/doc/src/sgml/catalogs.sgml b/doc/src/sgml/catalogs.sgml
index cbd4e40a320..d53e7e39b59 100644
--- a/doc/src/sgml/catalogs.sgml
+++ b/doc/src/sgml/catalogs.sgml
@@ -1582,7 +1582,7 @@
        <structfield>rolpassword</structfield> <type>text</type>
       </para>
       <para>
-       Password (possibly encrypted); null if none. The format depends
+       Encrypted password; null if none. The format depends
        on the form of encryption used.
       </para></entry>
      </row>
@@ -1627,11 +1627,6 @@ SCRAM-SHA-256$<replaceable>&lt;iteration count&gt;</replaceable>:<replaceable>&l
    <replaceable>ServerKey</replaceable> are in Base64 encoded format. This format is
    the same as that specified by <ulink url="https://datatracker.ietf.org/doc/html/rfc5803">RFC 5803</ulink>.
   </para>
-
-  <para>
-    A password that does not follow either of those formats is assumed to be
-    unencrypted.
-  </para>
  </sect1>
 
 
diff --git a/doc/src/sgml/system-views.sgml b/doc/src/sgml/system-views.sgml
index b58c52ea50f..986ae1f543d 100644
--- a/doc/src/sgml/system-views.sgml
+++ b/doc/src/sgml/system-views.sgml
@@ -3932,7 +3932,7 @@ SELECT * FROM pg_locks pl LEFT JOIN pg_prepared_xacts ppx
        <structfield>passwd</structfield> <type>text</type>
       </para>
       <para>
-       Password (possibly encrypted); null if none.  See
+       Encrypted password; null if none.  See
        <link linkend="catalog-pg-authid"><structname>pg_authid</structname></link>
        for details of how encrypted passwords are stored.
       </para></entry>
-- 
2.39.5 (Apple Git-154)


--oa/oIHxiwZpWg/Ua--