]> granicus.if.org Git - postgresql/commit
Change the on-disk format of SCRAM verifiers to conform to RFC 5803.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Fri, 21 Apr 2017 19:51:57 +0000 (22:51 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Fri, 21 Apr 2017 19:51:57 +0000 (22:51 +0300)
commit68e61ee72eb6914f493f08be98363c2f980ee242
tree258e46c2790aa1341a2676aa1d424b6a78005092
parentc29a752c683d9b08ee1376709b825532e94c2709
Change the on-disk format of SCRAM verifiers to conform to RFC 5803.

It doesn't make any immediate difference to PostgreSQL, but might as well
follow the standard, since one exists. (I looked at RFC 5803 earlier, but
didn't fully understand it back then.)

The new format uses Base64 instead of hex to encode StoredKey and
ServerKey, which makes the verifiers slightly smaller. Using the same
encoding for the salt and the keys also means that you only need one
encoder/decoder instead of two. Although we have code in the backend to
do both, we are talking about teaching libpq how to create SCRAM verifiers
for PQencodePassword(), and libpq doesn't currently have any code for hex
encoding.

Bump catversion, because this renders any existing SCRAM verifiers in
pg_authid invalid.

Discussion: https://www.postgresql.org/message-id/351ba574-85ea-d9b8-9689-8c928dd0955d@iki.fi
doc/src/sgml/catalogs.sgml
src/backend/libpq/auth-scram.c
src/backend/libpq/crypt.c
src/include/catalog/catversion.h
src/test/regress/expected/password.out
src/test/regress/sql/password.sql