diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2018-12-11 04:33:48 +0900 |
---|---|---|
committer | Ryuta Kamizono <kamipo@gmail.com> | 2018-12-11 04:51:54 +0900 |
commit | bde898c1df17722a83e5e8866b2009273c5190c6 (patch) | |
tree | 6925a7791b971874e52cc8f499eed833765520f9 /activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb | |
parent | bad1041b82df941d588ae2565f62424d88104933 (diff) | |
download | rails-bde898c1df17722a83e5e8866b2009273c5190c6.tar.gz rails-bde898c1df17722a83e5e8866b2009273c5190c6.tar.bz2 rails-bde898c1df17722a83e5e8866b2009273c5190c6.zip |
Prevent write queries for `exec_query`
Follow up #34505.
Diffstat (limited to 'activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb')
-rw-r--r-- | activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb b/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb index 1c4e625ead..381d5ab29b 100644 --- a/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb @@ -609,6 +609,10 @@ module ActiveRecord FEATURE_NOT_SUPPORTED = "0A000" #:nodoc: def execute_and_clear(sql, name, binds, prepare: false) + if preventing_writes? && write_query?(sql) + raise ActiveRecord::ReadOnlyError, "Write query attempted while in readonly mode: #{sql}" + end + if without_prepared_statement?(binds) result = exec_no_cache(sql, name, []) elsif !prepare |