[drupal-devel] [bug] PostgreSQL Socket Connection

[iamtheari]

Issue status update for 
http://drupal.org/node/26836
Post a follow up: 
http://drupal.org/project/comments/add/26836

 Project:      Drupal
 Version:      4.6.2
 Component:    postgresql database
 Category:     bug reports
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  iamtheari
 Updated by:   iamtheari
 Status:       patch (code needs review)

Postgres almost prefers Unix socket connections to TCP/IP.  It is easier
to set up working Unix sockets than TCP/IP connections with Postgres.


The PHP pg_connect() function's connection string will connect via Unix
socket if the "host=" element is omitted.  That is how my patch works,
but Drupal's means of handling the database URL prevented a blank
hostname from being accepted.  Also, using a made-up hostname to
signify a Unix socket is kludgy, at best.


Note that no path needs to be specified.  The way PHP connects via Unix
socket is to use the local Postgres libraries, and it seems to find the
socket just fine if it exists.




iamtheari



Previous comments:
------------------------------------------------------------------------

Sat, 16 Jul 2005 16:52:16 +0000 : iamtheari

Attachment: http://drupal.org/files/issues/drupal-pgsql-socket.patch (917 bytes)

Drupal does not allow for PostgreSQL connections via unix socket.  I
have written a patch that allows it to do so.  The effect of this patch
is that a hostname of 'localhost' in a pgsql:// $db_url will use Unix
sockets unless a port it set in the URL.  A hostname of '127.0.0.1'
will use TCP/IP as before.


Examples:
Unix socket: pgsql://user:pass@localhost/db
TCP/IP: pgsql://user:pass@localhost:4444/db
TCP/IP: pgsql://user:pass@127.0.0.1/db
TCP/IP: pgsql://user:pass@127.0.0.1:4444/db


The patch is attached to this bug report.  From a top-level Drupal
directory, type 'patch -p0 < /path/to/drupal-pgsql-socket.patch' to
apply it (omit the single-quotes).




------------------------------------------------------------------------

Sat, 16 Jul 2005 17:08:09 +0000 : chx

First, welcome among contributors and thanks for the patch, it looks
useful. There are a few general problems: only bugfixes go into 4.6.x
everything else, like this should be for HEAD. Other problem is with
coding style, we use 



<?php
if () {
}
else {
}
?>




also you spacing issues, like spaces are a dot. The rule, as explained
to me by Steven is that there is never a space between a dot and a
quote always otherwise. There is also coding style guideline in CVS.




------------------------------------------------------------------------

Sun, 17 Jul 2005 05:06:39 +0000 : iamtheari

I don't run the CVS version, but somehow I doubt that the file in
question has changed significantly enough to prevent the patch from
working.  As to the coding style, see now that Drupal's guidelines
require the space between "if" and "(", but the extra spaces around the
dots was from the original code in the patched file, as you can see from
the lines of the patch beginning with a "-".  Regardless, I hope that my
code can be hammered into shape so as to fit into Drupal.


Just be sure to warn people that the behavior of
'pgsql://user:password@localhost/db' has changed, if this makes it in.




------------------------------------------------------------------------

Mon, 25 Jul 2005 17:32:51 +0000 : Steven

Are Unix sockets something that is standard supported and enabled for
PGSQL? If not, it is confusing to change the behaviour just because you
enter "localhost" with no port. Why not make a special reserved hostname
(e.g. 'unixsocket') for this? It can be documented easily in
settings.php.




------------------------------------------------------------------------

Wed, 27 Jul 2005 20:12:07 +0000 : Cvbge

AFAIK sockets are supported by both mysql and postgres... but I think
they might not allways be accessible (well, the same can be said about
tcp  connections).


I also think that we should not break existing configurations and use
other syntax for specifying. Also you might need to provide path to
unix socket.