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

Attachment: 0001-Refactor-SHA2-functions-and-move-them-to-src-common.patch.gz
Description: application/gzip

Attachment: 0002-Add-encoding-routines-for-base64-without-whitespace-.patch.gz
Description: application/gzip

Attachment: 0003-Add-clause-PASSWORD-val-USING-protocol-to-CREATE-ALT.patch.gz
Description: application/gzip

Attachment: 0004-Support-for-SCRAM-SHA-256-authentication-RFC-5802-an.patch.gz
Description: application/gzip

Attachment: 0005-Add-regression-tests-for-passwords.patch.gz
Description: application/gzip

Attachment: 0006-Add-TAP-tests-for-authentication-methods.patch.gz
Description: application/gzip

Attachment: 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

Reply via email to