voxpelli / node-pg-pubsub

A Publish/Subscribe implementation on top of PostgreSQL NOTIFY/LISTEN
https://www.npmjs.com/package/pg-pubsub
MIT License
236 stars 16 forks source link

Update pg version to support postgres 12 #89

Closed sigseg1v closed 3 years ago

sigseg1v commented 4 years ago

Updating the pg version lets us use newer versions of postgres. I tested this on postgres 12.3.

The previous version of "pg" in packages.json (7.4.3) just does nothing (doesn't resolve, and doesn't reject, even after 5 mins) when trying to connect to a postgres 12.X server.

Ran the tests as described with docker. All pass and one skips, which seems intended. I did run them a few times and noticed a sporadic unhandled rejection warning, which looks like it's due to the first test that tests an erroneous database user, however that did not fail the test suite.

I updated the "@types/pg" package to the latest available, but I didn't see anything for 8.X. Unsure if that is a problem.

voxpelli commented 3 years ago

Done as part of 0.6.0, sorry for the delay!