From: | Nathaniel Waisbrot <waisbrot(at)highfleet(dot)com> |
---|---|
To: | Dave Cramer <pg(at)fastcrypt(dot)com> |
Cc: | dmp <danap(at)ttc-cmc(dot)net>, List <pgsql-jdbc(at)postgresql(dot)org> |
Subject: | Re: Bug report: NullPointerException from Driver.connect when passed a Properties with non-string values |
Date: | 2013-01-28 21:18:41 |
Message-ID: | 72B19ADD-5DD7-434F-AB8D-D0F7F5A6760C@highfleet.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-jdbc |
Java 1.6 added a new method to Properties, stringPropertyNames:
http://docs.oracle.com/javase/6/docs/api/java/util/Properties.html#stringPropertyNames%28%29
it does what propertyNames probably ought to: only gives you back keys for which getProperty will return a non-null value. But you'd have to drop Java 1.5 compatibility to use it.
On Jan 28, 2013, at 4:08 PM, Dave Cramer <pg(at)fastcrypt(dot)com> wrote:
> it irks me to have to do that, as the java api for getProperty suggests that you cannot have anything but a string in there, however it's not worth fighting for.
>
> Dave
>
> Dave Cramer
>
> dave.cramer(at)credativ(dot)ca
> http://www.credativ.ca
>
>
> On Mon, Jan 28, 2013 at 4:03 PM, dmp <danap(at)ttc-cmc(dot)net> wrote:
> I think that is what threw me. You are still going to need to
> have Nathaniel's patch in case someone does try to throw in a bogus
> properties that ends up null even thought the url is correct.
>
> danap.
>
>
> Kevin Grittner wrote:
> Dave Cramer<pg(at)fastcrypt(dot)com> wrote:
>
> My proposed patch is this
>
>
> if (url.startsWith("jdbc:postgresql:")) {
> return null;
> }
>
> Right at the top of getConnection, before we do any real work.
>
>
> Thoughts ?
>
> Shouldn't that have a bang to reverse the logic?
>
> -Kevin
>
>
>
> --
> Sent via pgsql-jdbc mailing list (pgsql-jdbc(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-jdbc
>
From | Date | Subject | |
---|---|---|---|
Next Message | dmp | 2013-01-28 21:20:17 | Re: Bug report: NullPointerException from Driver.connect when passed a Properties with non-string values |
Previous Message | Dave Cramer | 2013-01-28 21:08:44 | Re: Bug report: NullPointerException from Driver.connect when passed a Properties with non-string values |