-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update pg package to version 8 or allow ssl configuration? #28
Comments
@rosstroha I would also like to use @dfarinella-tda seems like the latest contributor to the project; is this possible? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi there! First, thank you all for taking the time to create this package. It's a huge timesaver, even after dealing with a two day long problem I just figured out. My brain is still swimming with all the different variables and troubleshooting things I had tried, so I apologize if these suggested changes aren't your responsibility!
I'm using Heroku to host the bot and a Heroku Postgres addon. Heroku's documentation states that an ssl config object is needed, as seen here. The problem is that
pg@8
is the version that allows one to do it in this exact way. See the breaking changes here. or, as an alternative, add an env varPGSSLMODE=no-verify
. This env var only works ifpg@8
is installed, apparently.My problem was this: No connection was being made to the db through
botbuilder-storage-postgres
. There were no errors. I solved it by explicitly installingpg@8
as well as addingPGSSLMODE=no-verify
because I didn't have access to the ssl configuration object through this package. As the lazy developer that I am I relied onpg
being installed as a dependency of this package, which it is. However, it's one major version behind; the one that doesn't jive withPGSSLMODE
.or
pg@8
?Many thanks!
The text was updated successfully, but these errors were encountered: