PostgreSQL, Replace static connection param list by the one built int…

…o libpq.

This makes the connection adapter future-proof regarding to new parameters.

To maintain backward compatibility, :requiressl is added by hand. It is
deprecated by PostgreSQL since 2003, but still accepted by libpq.

activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb

@@ -20,12 +20,6 @@
 
 module ActiveRecord
  module ConnectionHandling # :nodoc:
- VALID_CONN_PARAMS = [:host, :hostaddr, :port, :dbname, :user, :password, :connect_timeout,
- :client_encoding, :options, :application_name, :fallback_application_name,
- :keepalives, :keepalives_idle, :keepalives_interval, :keepalives_count,
- :tty, :sslmode, :requiressl, :sslcompression, :sslcert, :sslkey,
- :sslrootcert, :sslcrl, :requirepeer, :krbsrvname, :gsslib, :service]
-
  # Establishes a connection to the database that's used by all Active Record objects
  def postgresql_connection(config)
  conn_params = config.symbolize_keys
@@ -37,7 +31,8 @@ def postgresql_connection(config)
  conn_params[:dbname] = conn_params.delete(:database) if conn_params[:database]
 
  # Forward only valid config params to PGconn.connect.
- conn_params.keep_if { |k, _| VALID_CONN_PARAMS.include?(k) }
+ valid_conn_param_keys = PGconn.conndefaults_hash.keys + [:requiressl]
+ conn_params.slice!(*valid_conn_param_keys)
 
  # The postgres drivers don't allow the creation of an unconnected PGconn object,
  # so just pass a nil connection object for the time being.