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

invalid encoding name: unicode #1049

Open
vohmar opened this issue Dec 5, 2018 · 4 comments
Open

invalid encoding name: unicode #1049

vohmar opened this issue Dec 5, 2018 · 4 comments
Labels

Comments

@vohmar
Copy link
Contributor

vohmar commented Dec 5, 2018

Every time deploy is done this error is caught by errbit

invalid encoding name: unicode

https://errbit.internet.ee/apps/5bcf2b35d9d1af5a21961be2/problems/5c051556d9d1af253512a4ad

Backtrace
--
/home/registry/registry/shared/bundle/ruby/2.4.0/gems/activerecord-4.2.11/lib/active_record/connection_adapters/postgresql_adapter.rb:671 → set_client_encoding /home/registry/registry/shared/bundle/ruby/2.4.0/gems/activerecord-4.2.11/lib/active_record/connection_adapters/postgresql_adapter.rb:671 → configure_connection /home/registry/registry/shared/bundle/ruby/2.4.0/gems/activerecord-4.2.11/lib/active_record/connection_adapters/postgresql_adapter.rb:277 → reconnect! /home/registry/registry/shared/bundle/ruby/2.4.0/gems/activerecord-4.2.11/lib/active_record/connection_adapters/abstract_adapter.rb:340 → verify! /home/registry/registry/shared/bundle/ruby/2.4.0/gems/activerecord-4.2.11/lib/active_record/connection_adapters/abstract/connection_pool.rb:456 → block in checkout_and_verify
@vohmar vohmar added the bug label Dec 5, 2018
@maciej-szlosarczyk
Copy link
Contributor

I made a comment on the Errbit issue earlier. Root cause is the fact that the connection is dropped, and the error message is misleading.

https://bitbucket.org/ged/ruby-pg/issues/267/misleading-error-invalid-encoding-name

Fixed in later version of postgres adapter, which we cannot use yet:

https://bitbucket.org/ged/ruby-pg/commits/a7f98864a7d6


I am not certain if mina should drop a connection like that, and this is probably the fix.

@artur-intech
Copy link
Contributor

artur-intech commented Dec 5, 2018

Maybe it's also worth updating mina first...

@teadur
Copy link
Contributor

teadur commented Dec 5, 2018

Updating mina means rewriting the config for it, so its not trival, but doable if we don't have any other option.

@vohmar
Copy link
Contributor Author

vohmar commented Dec 5, 2018

pushing this way back in the dev pipeline then

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

No branches or pull requests

4 participants