Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dKJt1-0007Rn-8k for pgsql-pkg-yum@arkaria.postgresql.org; Mon, 12 Jun 2017 07:35:43 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1dKJsz-0001id-PW for pgsql-pkg-yum@arkaria.postgresql.org; Mon, 12 Jun 2017 07:35:41 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1dKJsv-0001br-LA for pgsql-pkg-yum@postgresql.org; Mon, 12 Jun 2017 07:35:37 +0000 Received: from [46.21.97.138] (helo=mail.zonezero.se) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1dKJss-0002D4-CR for pgsql-pkg-yum@postgresql.org; Mon, 12 Jun 2017 07:35:35 +0000 Received: from localhost (localhost [127.0.0.1]) by mail.zonezero.se (Postfix) with ESMTP id 7DBD214BA3F; Mon, 12 Jun 2017 09:35:32 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at mail.zonezero.se Authentication-Results: mail.zonezero.se (amavisd-new); dkim=pass header.i=@zonezero.se Received: from mail.zonezero.se ([127.0.0.1]) by localhost (mail.zonezero.se [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Dtj3e0cub5Ug; Mon, 12 Jun 2017 09:35:18 +0200 (CEST) Received: from webmail.zonezero.se (localhost [127.0.0.1]) by mail.zonezero.se (Postfix) with ESMTP id 76BCB13FC70; Mon, 12 Jun 2017 09:35:18 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=zonezero.se; s=zonezero.se; t=1497252918; bh=/0UPzqKSgkZ6UxwCAGEboItPZn2OZlPkMqaXavdPCu0=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=OggKvk3O6eBk2C/8Lo6dgRtqWFFpYx4m2NFTtgDe4liX+ntvaBvGZN6z1AQdAaElR sqKY/gnabj4jJh7YouS2ldrtViLKn+Obv43Gb8wE46IuP3YvYWhSW3bIRFpd7j7GSt 8qXqQyf2QLh2SFXggC+a0pjeHmmyWFq2P0i+Jl8k= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Date: Mon, 12 Jun 2017 09:35:18 +0200 From: Jesper St John To: =?UTF-8?Q?Devrim_G=C3=BCnd=C3=BCz?= Cc: pgsql-pkg-yum@postgresql.org Subject: Re: Possible bug for pam auth in at least 9.6 and 10 rpms on CentOS 7 In-Reply-To: <1497004856.4355.8.camel@gunduz.org> References: <4529a5e9d4d40abd2e0df7ab49c94118@zonezero.se> <1497004856.4355.8.camel@gunduz.org> Message-ID: <3e4f7809f23d8831aa36dcfe94576d38@zonezero.se> X-Sender: admin@zonezero.se User-Agent: Roundcube Webmail/1.1.0 X-Host-Lookup-Failed: Reverse DNS lookup failed for 46.21.97.138 (failed) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgsql-pkg-yum Precedence: bulk Sender: pgsql-pkg-yum-owner@postgresql.org That is awesome news. Thanks! //Jesper 2017-06-09 12:40 skrev Devrim Gündüz: > Hi Jesper, > > Seems reasonable. I'll apply this patch early next week. > > Regards, Devrim > > On Thu, 2017-06-08 at 14:11 +0200, Jesper St John wrote: >> When installing postgresql-server package, it puts a file in >> /etc/pam.d/  >> called postgresql%{majorversion}. >> >> Postgres source says this (src/backend/libpq/auth.c): >> #define PGSQL_PAM_SERVICE "postgresql"  /* Service name passed to PAM >> */ >> >> That creates the need to create a new file named >> /etc/pam.d/postgresql  >> to make pam auth work for me. >> >> I've tested this with both 9.6 and 10 beta on CentOS 7 latest. >> >> >> Best regards, >> Jesper St John >> admin@zonezero.se >> >> >> >> Suggested patch below: >> >> >> diff --git a/rpm/redhat/10/postgresql/master/postgresql-10.spec  >> b/rpm/redhat/10/postgresql/master/postgresql-10.spec >> index d6e53b0c..32542462 100644 >> --- a/rpm/redhat/10/postgresql/master/postgresql-10.spec >> +++ b/rpm/redhat/10/postgresql/master/postgresql-10.spec >> @@ -781,7 +781,7 @@ install -m 755 %{sname}.init  >> %{buildroot}%{_initrddir}/%{sname}-%{pgmajorversion >> >>   %if %pam >>   install -d %{buildroot}/etc/pam.d >> -install -m 644 %{SOURCE14}  >> %{buildroot}/etc/pam.d/%{sname}%{pgmajorversion} >> +install -m 644 %{SOURCE14} %{buildroot}/etc/pam.d/%{sname} >>   %endif >> >>   # Create the directory for sockets. >> @@ -1231,7 +1231,7 @@ fi >>   %config(noreplace) %{_initrddir}/%{sname}-%{pgmajorversion} >>   %endif >>   %if %pam >> -%config(noreplace) /etc/pam.d/%{sname}%{pgmajorversion} >> +%config(noreplace) /etc/pam.d/%{sname} >>   %endif >>   %attr (755,root,root) %dir /etc/sysconfig/pgsql >>   %{pgbaseinstdir}/bin/initdb >> >> -- Sent via pgsql-pkg-yum mailing list (pgsql-pkg-yum@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-pkg-yum