Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cP88S-0006Kw-AP for pgsql-hackers@arkaria.postgresql.org; Thu, 05 Jan 2017 13:31:16 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1cP88R-00017f-Ta for pgsql-hackers@arkaria.postgresql.org; Thu, 05 Jan 2017 13:31:15 +0000 Received: from makus.postgresql.org ([2001:4800:1501:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1cP88R-00017Z-Dg for pgsql-hackers@postgresql.org; Thu, 05 Jan 2017 13:31:15 +0000 Received: from out2-smtp.messagingengine.com ([66.111.4.26]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1cP88O-000491-ID for pgsql-hackers@postgresql.org; Thu, 05 Jan 2017 13:31:14 +0000 Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id 8DC2F20BB3; Thu, 5 Jan 2017 08:31:11 -0500 (EST) Received: from frontend1 ([10.202.2.160]) by compute7.internal (MEProxy); Thu, 05 Jan 2017 08:31:11 -0500 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc:x-sasl-enc; s= smtpout; bh=5/0xcaDRMa+fdMFuq0IY/laemeQ=; b=k3u+7Y8TDblLk+BhE8EX +oz9M8PGn00dXKs4BDvcsgX7hpM6mHB7j7ogfZu3wqdxx4nLlfFnjgY1MMrMZiY+ JSeTcjy7Yiz93MxNIsG8ebYQe4nzMaeJXzPXw9IlmP5HOEkjHFhRB5qxz/0Te04k 2aw4Aw7bxPGpNerH6+iW64E= X-ME-Sender: X-Sasl-enc: moR7K5JBWNzkgukC2WLT2btMsYxhJnMdmHcm2BF8OofT 1483623071 Received: from april.local (c-73-13-66-39.hsd1.pa.comcast.net [73.13.66.39]) by mail.messagingengine.com (Postfix) with ESMTPA id 0D2B27EA48; Thu, 5 Jan 2017 08:31:11 -0500 (EST) Subject: Re: pg_authid.rolpassword format (was Re: Password identifiers, protocol aging and SCRAM protocol) To: Heikki Linnakangas , Michael Paquier References: <21eecdb8-365b-8347-1ae2-5278d1fe798c@iki.fi> <762f4875-a87e-cf0b-c06b-b357294d8be1@iki.fi> <2d07165c-1793-e243-a2a9-e45b624c7580@iki.fi> <9a7263eb-0980-2072-4424-440bb2513dc7@iki.fi> <20161216013114.GA1400@paquier.xyz> <6831df67-7641-1a66-4985-268609a4821f@iki.fi> Cc: Magnus Hagander , Robert Haas , Andres Freund , David Steele , Tom Lane , David Fetter , Alvaro Herrera , Julian Markwort , Stephen Frost , PostgreSQL mailing lists , Valery Popov From: Peter Eisentraut Organization: 2ndQuadrant Message-ID: <49dca243-5d04-55cf-660e-bca52c3955df@2ndquadrant.com> Date: Thu, 5 Jan 2017 08:31:14 -0500 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <6831df67-7641-1a66-4985-268609a4821f@iki.fi> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Pg-Spam-Score: -2.6 (--) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgsql-hackers Precedence: bulk Sender: pgsql-hackers-owner@postgresql.org On 1/3/17 9:09 AM, Heikki Linnakangas wrote: > Since not everyone agrees with this approach, I split this patch into > two. The first patch refactors things, replacing the isMD5() function > with get_password_type(), without changing the representation of > pg_authid.rolpassword. That is hopefully uncontroversial. And the second > patch adds the "plain:" prefix, which not everyone agrees on. > > Barring objections I'm going to at least commit the first patch. I think > we should commit the second one too, but it's not as critical, and the > first patch matters more for the SCRAM patch, too. Is there currently anything to review here for the commit fest? -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers