bouncy 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 25620e7678f2db3d1aeae8f633654ae92205277b171eea5bddbbd3c3d4d898e1
-  data.tar.gz: f898b5f8180190136a031469ca2a6bd62f4930929378afedd0cae4d3f9a19f4a
+  metadata.gz: 454c9ceb5482a94e7885dd6d648b9716e05d24c6ffe6d11cae8e4957297be373
+  data.tar.gz: 36c35cc2d4b7b39d63722dd04cdb36a773cd0814dca1b7f8fd12be463ac55222
 SHA512:
-  metadata.gz: d9ebee34cdc50548f939a80ab07cfc3e26cd0284d0c46f5b608f032dd8b41b9b92d7e5101a77b2d7b0107d8fd977aaa2f58da8e822b580362692900f1fd345d2
-  data.tar.gz: 16c591e6c5e1d51d08bd34d8273a2afbc9d08e605f30520635e509b5519104e3b1f13917a740378e88106e3890ef8743606756117a0338087b11a8ee48f5fff1
+  metadata.gz: 8ab0266ad476a1b157171125d6062c3eda54b02b078aaaa244e6e22dee750c7ec19512cd36bec481270a8debe5b578b6502d3267bc01375450ec07990c33398e
+  data.tar.gz: '088f91735a153cdf3c6f9c922dbc775641743d375b11733f98438403d50ff0afbfa064e136486a416ccbcad97dc7e5e9887b134a229eb3caa860112f8f4151bc'

data/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 ## [Unreleased]
 
+## [0.2.2] - 2026-10-26
+
+One fix that an app on the fleet found with the PostGIS adapter. There is no new migration.
+
+### Fixed
+
+- Bouncy knows a database by its Active Record adapter class, so an adapter that subclasses a Rails adapter gets the locks of that database whatever name it reports, because the PostGIS adapter (`activerecord-postgis-adapter`) is a `PostgreSQLAdapter` whose `adapter_name` is `"PostGIS"`, and Bouncy matched the name `"PostgreSQL"` exactly: it raised "Bouncy supports PostgreSQL, MySQL and SQLite", the receiver answered 503 to every signed SNS message, and `Bouncy.hold`, `Bouncy.sync!` and every other write raised. The name is still the fallback for an adapter that only renames itself. One helper, `Bouncy::Database`, now makes this choice for every lock and read.
+- The install migration gives a PostGIS database JSONB columns and native UUID keys, as it gives PostgreSQL, because it looked for `"postgres"` in the adapter name and made JSON columns and 36-character string keys there. Tables made that way keep working and need no change.
+
 ## [0.2.1] - 2026-10-25
 
 Three gaps that the first app to move its own suppression table to Bouncy found. There is no new migration.

data/README.md

@@ -41,7 +41,7 @@ Check out my other 💎 Ruby gems: [`nondisposable`](https://github.com/rameerez
 ## Requirements
 
 - Ruby **3.3+** and Rails **7.2.3.2+**, below Rails 9. Tested on Ruby 3.3/3.4/4.0 and Rails 7.2/8.0/8.1.
-- **PostgreSQL, MySQL or SQLite**, with bigint or UUID primary keys.
+- **PostgreSQL** (including the PostGIS adapter), **MySQL** or **SQLite**, with bigint or UUID primary keys.
 - **Amazon SES**, one account and region. SES SMTP is supported; suppression management needs AWS API credentials.
 
 Start in observation mode and verify your sending setup before enabling dropping. See [compatibility and validation](guides/compatibility.md) for the tested matrix and remaining validation limits.

data/guides/compatibility.md

@@ -6,6 +6,12 @@ The v0.1.0 release run on September 9, 2026 passed all 54 combinations using Rub
 
 The v0.2.0 local run on October 25, 2026 passed all 54 combinations with the same Ruby, Rails, PostgreSQL and MySQL versions and SQLite 3.54.1, and the same 18 again with Rails 8.0.2, the oldest 8.0 release the transactional-fixture support was checked on: 201 tests on SQLite (one skipped there: a second pool cannot share an in-memory database), 220 on PostgreSQL and 219 on MySQL with Rails 8.1, one fewer with Rails 7.2 and 8.0, and at least 96.5% line and 91.5% branch coverage. It adds the address-lock tests with real connections on PostgreSQL and MySQL. No live SES run was part of it.
 
+The v0.2.2 local run on October 26, 2026 used Ruby 3.3.5 and Rails 8.1 only: 236 tests on SQLite (one skipped, as before), 256 on PostgreSQL and 255 on MySQL, all passing. It adds tests for adapters that subclass a Rails adapter under another name, with the suite's own connection renamed `"PostGIS"` on PostgreSQL. No live SES run and no run on the PostGIS adapter itself was part of it.
+
+## Database adapters
+
+Bouncy knows a database by its Active Record adapter class. An adapter that subclasses `PostgreSQLAdapter`, `AbstractMysqlAdapter` or `SQLite3Adapter` belongs to that database whatever `adapter_name` it reports. The PostGIS adapter (`activerecord-postgis-adapter`) reports `"PostGIS"` and gets everything PostgreSQL gets: advisory locks, the isolation check, JSONB and native UUIDs. An adapter that is no such subclass is known by its name: one that contains `postg`, `mysql`, `trilogy` or `sqlite`. Any other adapter raises `Bouncy::ConfigurationError` at its first lock. The suite runs MySQL through `mysql2` only; the Trilogy adapter shares `AbstractMysqlAdapter` with it and is recognised by its class and its name, but no suite run uses it.
+
 The runtime JSON gem is constrained below version 3 because the supported Rails releases still pass a positional options hash to `JSON.parse`. Optional AWS SDKs load lazily and are host dependencies. There is no database or AWS lookup during boot.
 
 ## Migrations
@@ -25,11 +31,11 @@ Without a setting, Rails chooses its standard primary key. Explicit bigint/integ
 
 | Database | JSON storage | JSON defaults | UUID storage |
 |---|---|---|---|
-| PostgreSQL | JSONB | Database and model defaults | Native UUID |
+| PostgreSQL, PostGIS included | JSONB | Database and model defaults | Native UUID |
 | MySQL 8.4 | JSON | Model defaults; no literal SQL JSON default | 36-character string |
 | SQLite | JSON | Database and model defaults | 36-character string |
 
-The gem's JSON attributes inherit the database's native type. Every new record gets an independent hash/array default. No adapter probe or attribute-schema query is performed at boot. Unsupported database adapters raise before reconciliation.
+The gem's JSON attributes inherit the database's native type. Every new record gets an independent hash/array default. No adapter probe or attribute-schema query is performed at boot. Unsupported database adapters raise before reconciliation. Before 0.2.2 the migration made JSON columns and 36-character string keys on PostGIS; those tables keep working.
 
 MySQL identity columns explicitly use `utf8mb4_bin`. Case/accent-insensitive equality must not collapse addresses beyond Bouncy's trimmed lowercase policy. Scope and address limits keep compound indexes within supported database key limits. Use an application-normalized host column for model scopes.
 

data/lib/bouncy/address_lock.rb

@@ -170,7 +170,7 @@ module Bouncy
     # stays invisible. Hold the address outside the transaction instead; the lock then lasts for
     # the whole block, and the transaction's snapshot starts after it.
     def require_read_committed(connection)
-      return unless connection.adapter_name == "PostgreSQL" && connection.transaction_open?
+      return unless Database.family(connection) == :postgresql && connection.transaction_open?
 
       isolation = connection.uncached { connection.select_value("SHOW transaction_isolation") }
       return if ["read committed", "read uncommitted"].include?(isolation)
@@ -181,11 +181,10 @@ module Bouncy
 
     # Takes the lock now and returns what releases it, or nil when the database releases it.
     def take(connection, key, exclusive:)
-      case connection.adapter_name
-      when "PostgreSQL" then take_advisory(connection, key, exclusive)
-      when /Mysql|Trilogy/i then take_named(connection, key)
-      when "SQLite" then take_file(connection, exclusive ? File::LOCK_EX : File::LOCK_SH)
-      else raise ConfigurationError, "Bouncy supports PostgreSQL, MySQL and SQLite"
+      case Database.family!(connection)
+      when :postgresql then take_advisory(connection, key, exclusive)
+      when :mysql then take_named(connection, key)
+      when :sqlite then take_file(connection, exclusive ? File::LOCK_EX : File::LOCK_SH)
       end
     end
 

data/lib/bouncy/database.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module Bouncy
+  # Names the database family of a connection, which decides the lock Bouncy takes. A subclass
+  # of a Rails adapter belongs to the family of its parent class: the PostGIS adapter is a
+  # PostgreSQLAdapter whose adapter_name is "PostGIS", and the Trilogy and Mysql2 adapters share
+  # AbstractMysqlAdapter. The adapter_name is the fallback for an adapter that only renames
+  # itself. Each class is looked up only once its adapter is loaded, so no driver gem loads.
+  module Database
+    FAMILIES = {
+      postgresql: ["PostgreSQLAdapter", /postg/i],
+      mysql: ["AbstractMysqlAdapter", /mysql|trilogy/i],
+      sqlite: ["SQLite3Adapter", /sqlite/i]
+    }.freeze
+
+    module_function
+
+    # :postgresql, :mysql, :sqlite, or nil for any other database.
+    def family(connection)
+      FAMILIES.find { |_, (adapter, _)| subclass?(connection, adapter) }&.first ||
+        FAMILIES.find { |_, (_, name)| connection.adapter_name.match?(name) }&.first
+    end
+
+    def family!(connection)
+      family(connection) || raise(ConfigurationError, "Bouncy supports PostgreSQL, MySQL and SQLite")
+    end
+
+    def subclass?(connection, adapter)
+      ActiveRecord::ConnectionAdapters.const_defined?(adapter, false) &&
+        connection.is_a?(ActiveRecord::ConnectionAdapters.const_get(adapter, false))
+    end
+  end
+end

data/lib/bouncy/scope_lock.rb

@@ -11,8 +11,8 @@ module Bouncy
     def synchronize
       Record.connection_pool.with_connection do |connection|
         key = Digest::SHA256.hexdigest(Bouncy.scope)[0, 15].to_i(16)
-        case connection.adapter_name
-        when "PostgreSQL"
+        case Database.family!(connection)
+        when :postgresql
           # Uncached: a cached answer would report the lock without taking it.
           acquired = connection.uncached { connection.select_value("SELECT pg_try_advisory_lock(#{key})") }
           raise ProviderError, "Another sync holds this scope lock" unless [true, "t"].include?(acquired)
@@ -22,7 +22,7 @@ module Bouncy
           ensure
             connection.execute("SELECT pg_advisory_unlock(#{key})")
           end
-        when /Mysql|Trilogy/i
+        when :mysql
           name = Digest::SHA256.hexdigest("bouncy:#{connection.pool.db_config.database}:#{Bouncy.scope}")
           acquired = connection.uncached { connection.select_value("SELECT GET_LOCK('#{name}', 0)") }
           raise ProviderError, "Another sync holds this scope lock" unless acquired == 1
@@ -32,7 +32,7 @@ module Bouncy
           ensure
             connection.execute("SELECT RELEASE_LOCK('#{name}')")
           end
-        when "SQLite"
+        when :sqlite
           database = connection.pool.db_config.database
           path = database == ":memory:" ? File.join(Dir.tmpdir, "bouncy-#{Process.pid}-#{key}.lock") : "#{File.expand_path(database)}.bouncy-#{key}.lock"
           File.open(path, File::RDWR | File::CREAT, 0o600) do |file|
@@ -44,8 +44,6 @@ module Bouncy
               file.flock(File::LOCK_UN)
             end
           end
-        else
-          raise ConfigurationError, "Bouncy supports PostgreSQL, MySQL and SQLite"
         end
       end
     end

data/lib/bouncy/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bouncy
-  VERSION = "0.2.1"
+  VERSION = "0.2.2"
 end

data/lib/bouncy.rb

@@ -173,7 +173,7 @@ module Bouncy
     # committed state: a REPEATABLE READ snapshot from before the hold cannot hide a newer block
     # or release. PostgreSQL at READ COMMITTED reads the latest state with every statement.
     def current_state(relation)
-      return relation unless AddressLock.holding? && Record.connection_pool.with_connection(&:adapter_name).match?(/Mysql|Trilogy/i)
+      return relation unless AddressLock.holding? && Record.connection_pool.with_connection { |connection| Database.family(connection) } == :mysql
 
       relation.lock("LOCK IN SHARE MODE")
     end
@@ -201,6 +201,7 @@ end
 
 require_relative "bouncy/configuration"
 require_relative "bouncy/identity"
+require_relative "bouncy/database"
 require_relative "bouncy/record"
 require_relative "bouncy/suppression"
 require_relative "bouncy/event"

data/lib/generators/bouncy/templates/create_bouncy_tables.rb.erb

@@ -87,11 +87,25 @@ class CreateBouncyTables < ActiveRecord::Migration<%= migration_version %>
     mysql? ? { collation: "utf8mb4_bin" } : {}
   end
 
-  def postgresql?
-    connection.adapter_name.downcase.include?("postgres")
+  def postgresql? = adapter_family == :postgresql
+  def mysql? = adapter_family == :mysql
+
+  # Mirrors Bouncy::Database.family. It is inlined because the migration must not depend on
+  # Bouncy's code when it runs. Every adapter class comes before any name: a subclass of a Rails
+  # adapter keeps the family of its parent class whatever it reports, as the PostGIS adapter
+  # reports "PostGIS". The name is the fallback for an adapter that only renames itself.
+  def adapter_family
+    families = {
+      postgresql: ["PostgreSQLAdapter", /postg/i],
+      mysql: ["AbstractMysqlAdapter", /mysql|trilogy/i],
+      sqlite: ["SQLite3Adapter", /sqlite/i]
+    }
+    families.find { |_, (adapter, _)| adapter_subclass?(adapter) }&.first ||
+      families.find { |_, (_, name)| connection.adapter_name.match?(name) }&.first
   end
 
-  def mysql?
-    connection.adapter_name.downcase.match?(/mysql|trilogy/)
+  def adapter_subclass?(adapter)
+    ActiveRecord::ConnectionAdapters.const_defined?(adapter, false) &&
+      connection.is_a?(ActiveRecord::ConnectionAdapters.const_get(adapter, false))
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bouncy
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - rameerez
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-10-25 00:00:00.000000000 Z
+date: 2026-10-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -79,6 +79,7 @@ files:
 - lib/bouncy/address_lock.rb
 - lib/bouncy/body_limit.rb
 - lib/bouncy/configuration.rb
+- lib/bouncy/database.rb
 - lib/bouncy/engine.rb
 - lib/bouncy/event.rb
 - lib/bouncy/identity.rb