cloudflare-email 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b53307e62ade70f522e0fcb5705e11552e9718914d02bea8754a11653b15349a
-  data.tar.gz: 692c09a19fcb4ccd9295ccb692a19bf02aba3176b107c8500e5450b100ce1df3
+  metadata.gz: b10971753d5d8d65bfafc658d3b8bc38553de251215c27097076d53593131813
+  data.tar.gz: b341d781aa6167beecf164915c6672e05c90c8888d76edd90c85e40413a84ad4
 SHA512:
-  metadata.gz: 43da0e85cde52970e696d5a2dfd74f03c0f39f5ea7d66e33cd28ec2bb58e5872c57db7f5dbc010ab85ad43312de9c9abcf2eed31a65e01f78fed8b6de164483b
-  data.tar.gz: a38902643ce082d656a01b7d91a34e70f95f15d66de6fd06e1fe7502461ae6d155a64edcea780155dd9b00b055635a3b10d5a0e9d8cbf2d077e1a7782742b193
+  metadata.gz: ec423d963ffeb2a3a126067360131dafc0bfd0bd4a6f25c4460fdadc03920a7af35bd21d8421c7ea827daa9cb6de36258ea6ed2932a58dbb30c4b92e0e22ab08
+  data.tar.gz: ce03d941227cd672b26c283c973773c1875e5ba63192671f002c9984877db8f4a28c5a40f2def54284bdfd6e031357d8f442da4958c23efb3bbac8c419b336a0

data/CHANGELOG.md

@@ -1,5 +1,33 @@
 # Changelog
 
+## 0.4.2 — 2026-10-12
+
+Released alongside Mailbox Kit 0.1.1, which it now requires.
+
+- Mailbox Kit: stop aborting Rails' production attribute preloader. Without a
+  Mailbox Kit tenant context, `connection_pool` raised
+  `TenantConnectionUnavailable`; Rails 8.1's `active_record.define_attribute_methods`
+  initializer asks every model for `connection_pool.schema_reflection` inside one
+  rescue, so the first Mailbox Kit model ended the loop (printing "Failed to
+  define attribute methods ...") and every model after it — including the host's
+  own — was left to define its attribute methods on first use. `connection_pool`
+  now returns a placeholder that answers only `schema_reflection` (empty, so
+  nothing is defined for Mailbox Kit models) and raises the same
+  `TenantConnectionUnavailable` for anything else. Queries, construction and
+  connection checkout without a context still fail closed.
+- Mailbox Kit: upload the raw email before opening a transaction. `InboundEmail.persist`
+  stored the bytes inside its own transaction, and `Mailboxes.receive(source:)` and
+  `Ingress::VerifiedMessage#receive_into_mailbox!` called it inside the membership
+  transaction, so the storage round trip (an S3 PUT in production) ran while the
+  transaction held the database's write lock. On SQLite with immediate transactions
+  that stalled every other writer to the tenant. Now the bytes upload first and only
+  rows are written under the lock. New `InboundEmail.prepare(...)` returns an
+  uploaded-but-unsaved delivery whose `#persist` writes the rows, for hosts with their
+  own transaction. A duplicate uploads nothing. Bytes no committed row names (a rolled
+  back delivery, or one a concurrent delivery beat) are deleted once the transaction
+  has finished. The duplicate checks under the lock bypass the query cache, which
+  would otherwise repeat the answer from before the upload.
+
 ## 0.4.1 — 2026-10-06
 
 - Report the provider's own reason when a refused request returns a body that

data/lib/cloudflare/email/ingress.rb

@@ -44,9 +44,17 @@ module Cloudflare
         # missing membership without scheduling Rails processing again.
         def receive_into_mailbox!
           result = nil
-          Mailboxes.receive(recipient: envelope.fetch("to")) do
-            result = persist_rails_record
-            result.record
+          prepared = nil
+          recipient = envelope.fetch("to")
+          # Upload before the membership transaction opens (see InboundEmail).
+          Mailboxes.with_recipient(recipient: recipient) { prepared = prepare_rails_record }
+          begin
+            Mailboxes.receive(recipient: recipient) do
+              result = prepared.persist
+              result.record
+            end
+          ensure
+            Mailboxes.for_tenant(prepared.tenant_key) { prepared.discard } if prepared
           end
           result.record if result.created?
         end
@@ -54,8 +62,12 @@ module Cloudflare
         private
 
         def persist_rails_record
+          prepare_rails_record.persist
+        end
+
+        def prepare_rails_record
           require "mailbox_kit/inbound_email"
-          MailboxKit::InboundEmail.persist(source: body, message_checksum: message_checksum,
+          MailboxKit::InboundEmail.prepare(source: body, message_checksum: message_checksum,
             message_id: stable_message_id, metadata: storage_metadata)
         end
 

data/lib/cloudflare/email/version.rb

@@ -1,5 +1,5 @@
 module Cloudflare
   module Email
-    VERSION = "0.4.1"
+    VERSION = "0.4.2"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cloudflare-email
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Cole
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.0
+        version: 0.1.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.0
+        version: 0.1.1
 - !ruby/object:Gem::Dependency
   name: base64
   requirement: !ruby/object:Gem::Requirement