Skip to content
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

Monkey Patch Postgres to fix Panic string issue. #182

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rylanb
Copy link
Collaborator

@rylanb rylanb commented Dec 10, 2020

Overview

Heroku upgraded our Postgres version today via regular maintenance (12-10-2020) and it brought down with a message PG::InvalidParameterValue: ERROR: invalid value for parameter “client_min_messages”: “panic”

Found a way to monkey patch and get the app back up via:
https://stackoverflow.com/questions/58763542/pginvalidparametervalue-error-invalid-value-for-parameter-client-min-messag

I think a better option would to re-provision to postgres 9.6 specifically. I couldn't check the version this AM, but best guess is that we are on 10 or 11 version that dropped the support for 'panic'. I don't have a ton of knowledge about this as of now

This blew up the app with error around ‘panic’ as a status
Found via https://stackoverflow.com/questions/58763542/pginvalidparametervalue-error-invalid-value-for-parameter-client-min-messag

Heroku just upgraded Postgres to something … which I can’t tell which version or why this started failing.
@phyninja
Copy link

Hey,

I pulled this PR to test the fix, but found that the error remained upon running the command

bundle exec bin/setup

Did you run a command before the one above, for the app to 'apply' the fix?

@rylanb
Copy link
Collaborator Author

rylanb commented Apr 20, 2021

Probably have to run bundle update pg ?

@phyninja
Copy link

Sorry, that didn't work for me; running bundle update pg resulted in this message being displayed:

Bundler attempted to update pg but its version stayed the same
Bundle updated!

For now, I have used this workaround.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants