Dave,
The inclusion of:
if (!url.startsWith("jdbc:postgresql:")) {
return null;
}
in the patch yesterday does not appear to have
been necessary because it looks to be checked
in parseURL().
danap.
It's the placement of it that is important. It checks there before it does anything of consequence at all. I left it in parseURL in case there is a path that I wasn't aware of, and it is pretty inconsequential