On 02/07/2017 04:20 AM, Michael Paquier wrote:
On Tue, Feb 7, 2017 at 3:12 AM, Aleksander Alekseev <a.aleks...@postgrespro.ru> wrote:No, I'm afraid `make distclean` doesn't help. I've re-checked twice.Hm. I can see the failure on macos and python2 builds as well with the set of patches applied. And the master branch is working properly. This needs some investigation.
Ah, found it. It was because of this change:
--- a/src/backend/utils/errcodes.txt +++ b/src/backend/utils/errcodes.txt @@ -247,6 +247,7 @@ Section: Class 28 - Invalid Authorization Specification 28000 E ERRCODE_INVALID_AUTHORIZATION_SPECIFICATION invalid_authorization_specification 28P01 E ERRCODE_INVALID_PASSWORD invalid_password +28P01 E ERRCODE_INVALID_NONCE invalid_nonce
Having two error codes with the same SQLSTATE is not cool, and tripped the assertion in PL/python. I removed the new error code, it was only used in one place, and ERRCODE_PROTOCOL_VIOLATIOn was more appropriate there anyway.
Attached is a new set of patches, with that fixed. Thanks for the report Aleksander!
- Heikki
0001-Refactor-SHA2-functions-and-move-them-to-src-common.patch.gz
Description: application/gzip
0002-Add-encoding-routines-for-base64-without-whitespace-.patch.gz
Description: application/gzip
0003-Add-clause-PASSWORD-val-USING-protocol-to-CREATE-ALT.patch.gz
Description: application/gzip
0004-Support-for-SCRAM-SHA-256-authentication-RFC-5802-an.patch.gz
Description: application/gzip
0005-Add-regression-tests-for-passwords.patch.gz
Description: application/gzip
0006-Add-TAP-tests-for-authentication-methods.patch.gz
Description: application/gzip
0007-Introduce-WIP-for-UTF-8-normalization.patch.gz
Description: application/gzip
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers