denodrivers / postgres

PostgreSQL driver for Deno

Home Page:https://denodrivers.github.io/postgres

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Support postgresql:// for protocol

nalanj opened this issue · comments

I'm not sure if it was an oversight or otherwise, but the library supports connection strings beginning with postgres:// but not postgresql://, despite the fact that postgresql:// is the official libpq way of doing things. Not too big of a deal, but it definitely threw me off a bit at first when postgresql wasn't working.

I'm simply asking here first - I'm more than happy to put together a quick PR to handle this.

commented

a PR would be very welcome

@Soremwar this seem reasonable to you?

@nalanj Feel free to open a PR if you want, though I don't think it will be a deal breaker for most

Yep, definitely not a deal breaker. I more just saw it as a rough corner that I could smooth out super easily.

Closed by #260