Thread: [SQLObject] port option being ignored in connection uri?
SQLObject is a Python ORM.
Brought to you by:
ianbicking,
phd
From: Jamie W. <ja...@sp...> - 2005-08-30 00:37:59
|
Hi! I've got a test postgresql database running on port 54320, and my connection string looks like "postgresql://127.0.0.1:54320/database". However, I get a connectoin failure from psycopg, which suggests that the port isn't being used: File "sqlobject/sqlobject/postgres/pgconnection.py", line 84, in makeConnection raise self.module.OperationalError("%s; used connection string %r" % (e, self.dsn) psycopg.OperationalError: FATAL: IDENT authentication failed for user "jaq" ; used connection string 'dbname=database host=127.0.0.1' I'm using a fresh checkout of sqlobject from about an hour ago, and psycopg 1.1.18 with python 2.4 on Ubuntu hoary. |
From: Jamie W. <ja...@sp...> - 2005-08-30 00:42:21
|
This one time, at band camp, Jamie Wilkinson wrote: >I've got a test postgresql database running on port 54320, and my connection >string looks like "postgresql://127.0.0.1:54320/database". However, I get a >connectoin failure from psycopg, which suggests that the port isn't being >used: Here's the patch :) Index: sqlobject/postgres/pgconnection.py =================================================================== --- sqlobject/postgres/pgconnection.py (revision 965) +++ sqlobject/postgres/pgconnection.py (working copy) @@ -56,6 +56,8 @@ dsn.append('user=%s' % user) if passwd: dsn.append('password=%s' % passwd) + if port: + dsn.append('port=%s' % port) if host: # @@: right format? dsn.append('host=%s' % host) |
From: Oleg B. <ph...@ph...> - 2005-08-30 09:15:26
|
On Tue, Aug 30, 2005 at 10:41:56AM +1000, Jamie Wilkinson wrote: > Here's the patch :) > > Index: sqlobject/postgres/pgconnection.py > =================================================================== > --- sqlobject/postgres/pgconnection.py (revision 965) > +++ sqlobject/postgres/pgconnection.py (working copy) > @@ -56,6 +56,8 @@ > dsn.append('user=%s' % user) > if passwd: > dsn.append('password=%s' % passwd) > + if port: > + dsn.append('port=%s' % port) Thank you. You've forgotted to fix pygresql case. I did it and committed at revision 966. Oleg. -- Oleg Broytmann http://phd.pp.ru/ ph...@ph... Programmers don't die, they just GOSUB without RETURN. |