diff options
author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2017-04-28 15:04:02 +0300 |
---|---|---|
committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2017-04-28 15:22:38 +0300 |
commit | d981074c24d2f1e4f44bc6d80e967e523ce64f50 (patch) | |
tree | aca39492e91899c6fca0e7a23e72b0894c438eed /src/backend/parser/parse_utilcmd.c | |
parent | b9a3ef55b253d885081c2d0e9dc45802cab71c7b (diff) | |
download | postgresql-d981074c24d2f1e4f44bc6d80e967e523ce64f50.tar.gz postgresql-d981074c24d2f1e4f44bc6d80e967e523ce64f50.zip |
Misc SCRAM code cleanups.
* Move computation of SaltedPassword to a separate function from
scram_ClientOrServerKey(). This saves a lot of cycles in libpq, by
computing SaltedPassword only once per authentication. (Computing
SaltedPassword is expensive by design.)
* Split scram_ClientOrServerKey() into two functions. Improves
readability, by making the calling code less verbose.
* Rename "server proof" to "server signature", to better match the
nomenclature used in RFC 5802.
* Rename SCRAM_SALT_LEN to SCRAM_DEFAULT_SALT_LEN, to make it more clear
that the salt can be of any length, and the constant only specifies how
long a salt we use when we generate a new verifier. Also rename
SCRAM_ITERATIONS_DEFAULT to SCRAM_DEFAULT_ITERATIONS, for consistency.
These things caught my eye while working on other upcoming changes.
Diffstat (limited to 'src/backend/parser/parse_utilcmd.c')
0 files changed, 0 insertions, 0 deletions