search

DASSL / ClassDB

An open-source system to let students experiment with relational data
https://dassl.github.io/ClassDB/
Other
8 stars 2 forks source link

testClassDBRolesMgmt.sql relies on MD5 password hashing (W) #213

Open wildtayne opened 6 years ago

wildtayne commented 6 years ago

checkEncryptedPwd() in testClassDBRolesMgmt.sql only checks for md5 passwords. If an instance is configured to use scram-sha-256, the test will fail.

EDIT: Other parts for testClassDBRolesMgmt.sql also seem to fail when using scram-sha-256().

ERROR:  Invalid argument: server role "<role>" already exists
CONTEXT:  PL/pgSQL function createrole(idnamedomain,character varying,boolean,idnamedomain,character varying,boolean,boolean,character varying)

Is thrown on the following lines (does not occur when password_encryption = `md5'):

smurthys commented 6 years ago

I think this should be covered in the documentation for M3 if the release is qualified for a Posgtres version in which where scram-sha-256 is supported. Let us leave the milestone unassigned until the Postgres version is determined.