mail_dude 0.1.1 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1d1597d0569b3ed20a9b14713f909ffce570bb26ed17fde3d2070453ff5b9fdd
-  data.tar.gz: 1cb89746067bb096290891ef62f928154ccc326ce2d39243902364ab339bbe5c
+  metadata.gz: 38404e9ab732d38243310741c269a4ec27e02a3076c479c891fe7bd532474d97
+  data.tar.gz: c66340b339905d47989e60b49d86073aab27785eae7a9da526fd910914db81ff
 SHA512:
-  metadata.gz: a18a26ad181887b44096287fc7b1a78fc007d5fe60c085eb55fd0fddf6af6c8c641fad576a30daea6929b0d8d4143ffd6f2aaa260039c97c96d94cd76568484c
-  data.tar.gz: ec195556d3bd1f3cc28bcf799af9f3ce2fc9b32b4601f7a9f2a446dabd70551929c9bab8b8ccef2369bde1a2be732ada4b4479bbc644437da488b8e65843222b
+  metadata.gz: 3d28003d59309eccded62419a0d28573facd4a027045c9df28eefa7491206f5815bc9906f60cb2631a8ada54921bbf59c9b3c418fa28a89462eca055946ee2ef
+  data.tar.gz: 7d8011434a33827a7f795f3bf865b7995dfec41d999aab8ef9cfd8d99e2f8b9d3ecf725c75f74deddbb4702fb9f8353666e66565254ca4e23c21f8e1a89b9be2

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.1.3
+
+- Fix CID image rendering for Content-ID attachments without inline disposition.
+- Strip attachment payloads from stored raw source when attachment capture is disabled.
+
+## 0.1.2
+
+- Add official Ruby 3.1 support.
+
 ## 0.1.1
 
 - Add compatibility with Rails >= 7.0.3.1.

data/README.md

@@ -8,7 +8,7 @@ Local and QA applications often need realistic email delivery flows without risk
 
 ## Requirements
 
-MailDude supports Ruby 3.2+ and Rails >= 7.0.3.1, < 8.0. CI covers Rails 7.0.3.1, 7.1, and 7.2.
+MailDude supports Ruby 3.1+ and Rails >= 7.0.3.1, < 8.0. CI covers Ruby 3.1 and 3.2 across Rails 7.0.3.1, 7.1, and 7.2.
 
 ## Installation
 

data/lib/mail_dude/attachment_locator.rb

@@ -2,6 +2,26 @@
 
 module MailDude
   class AttachmentLocator
+    class << self
+      def attachment_part?(part)
+        !part.multipart? && (part.attachment? || part.filename.present? || inline_renderable_part?(part))
+      end
+
+      def content_disposition(part)
+        part.content_disposition.to_s.split(';').first.to_s.downcase
+      end
+
+      def inline_renderable_part?(part)
+        inline_part?(part) || part.content_id.present?
+      end
+
+      private
+
+      def inline_part?(part)
+        content_disposition(part) == 'inline' && part.content_id.present?
+      end
+    end
+
     Attachment = Struct.new(:id, :part, :metadata, keyword_init: true) do
       def content_type
         metadata['content_type']
@@ -35,6 +55,14 @@ module MailDude
       end
     end
 
+    def attachments_count
+      attachment_parts.length
+    end
+
+    def attachments_present?
+      attachments_count.positive?
+    end
+
     def find(attachment_id)
       raise AttachmentNotFoundError, 'Attachment not found' unless attachment_id.to_s.match?(/\Aa\d+\z/)
 
@@ -44,6 +72,8 @@ module MailDude
 
     def find_inline_by_cid(content_id)
       normalized = normalize_content_id(content_id)
+      return nil if normalized.blank?
+
       attachments.find { |attachment| attachment.metadata['content_id'] == normalized }
     end
 
@@ -64,19 +94,7 @@ module MailDude
     def attachment_parts
       return [] unless mail
 
-      mail.all_parts.select { |part| attachment_part?(part) }
-    end
-
-    def attachment_part?(part)
-      !part.multipart? && (part.attachment? || part.filename.present? || inline_part?(part))
-    end
-
-    def content_disposition(part)
-      part.content_disposition.to_s.split(';').first.to_s.downcase
-    end
-
-    def inline_part?(part)
-      content_disposition(part) == 'inline' && part.content_id.present?
+      mail.all_parts.select { |part| self.class.attachment_part?(part) }
     end
 
     def mail
@@ -96,8 +114,8 @@ module MailDude
         'filename' => sanitize_filename(part.filename, fallback: "attachment-#{id}"),
         'content_type' => part.mime_type.presence || 'application/octet-stream',
         'content_id' => normalize_content_id(part.content_id),
-        'disposition' => content_disposition(part).presence || 'attachment',
-        'inline' => content_disposition(part) == 'inline',
+        'disposition' => self.class.content_disposition(part).presence || 'attachment',
+        'inline' => self.class.inline_renderable_part?(part),
         'size_bytes' => decoded_size(part)
       }
     end

data/lib/mail_dude/attachment_scrubber.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module MailDude
+  class AttachmentScrubber
+    RAW_SOURCE_OMITTED = "MailDude omitted raw message source because attachment capture is disabled.\n"
+
+    def initialize(mail)
+      @mail = mail
+    end
+
+    def raw_source
+      source = mail&.to_s
+      return RAW_SOURCE_OMITTED if source.blank?
+
+      sanitized = Mail.read_from_string(source)
+      remove_attachments!(sanitized)
+      sanitized.to_s
+    rescue StandardError
+      RAW_SOURCE_OMITTED
+    end
+
+    private
+
+    attr_reader :mail
+
+    def remove_attachments!(message)
+      return remove_single_part_attachment!(message) unless message.multipart?
+
+      message.parts.recursive_delete_if { |part| AttachmentLocator.attachment_part?(part) }
+    end
+
+    def remove_single_part_attachment!(message)
+      message.body = '' if AttachmentLocator.attachment_part?(message)
+    end
+  end
+end

data/lib/mail_dude/message_broadcast.rb

@@ -41,7 +41,7 @@ module MailDude
         sender: presenter.sender_summary,
         recipients: presenter.recipient_summary,
         captured_at: presenter.captured_at_label,
-        attachments_count: presenter.attachments.length,
+        attachments_count: presenter.attachment_count,
         attachment_count_label: presenter.attachment_count_label,
         mailer_label: presenter.mailer_label
       }

data/lib/mail_dude/message_presenter.rb

@@ -80,12 +80,12 @@ module MailDude
       raw_source.split(/\r?\n\r?\n/, 2).first.to_s
     end
 
-    def has_attachments?
-      attachments.any?
-    end
+    def has_attachments? = metadata_value('has_attachments') == true || attachments.any?
+
+    def attachment_count = metadata_value('attachments_count').presence&.to_i || attachments.length
 
     def attachment_count_label
-      count = attachments.length
+      count = attachment_count
       "#{count} #{'attachment'.pluralize(count)}"
     end
 

data/lib/mail_dude/message_serializer.rb

@@ -35,8 +35,8 @@ module MailDude
         'mailer_action' => internal_header(INTERNAL_ACTION_HEADER),
         'has_html' => part_present?('text/html'),
         'has_text' => part_present?('text/plain'),
-        'has_attachments' => attachments.any?,
-        'attachments_count' => attachments.length,
+        'has_attachments' => attachment_locator.attachments_present?,
+        'attachments_count' => attachment_locator.attachments_count,
         'attachments' => attachments,
         'size_bytes' => raw_source.bytesize
       }
@@ -55,7 +55,11 @@ module MailDude
     end
 
     def attachments
-      @attachments ||= AttachmentLocator.new(mail).attachments.map(&:metadata)
+      @attachments ||= attachment_locator.attachments.map(&:metadata)
+    end
+
+    def attachment_locator
+      @attachment_locator ||= AttachmentLocator.new(mail)
     end
 
     def content_type

data/lib/mail_dude/stores/base.rb

@@ -35,11 +35,17 @@ module MailDude
       private
 
       def build_record(mail, id: generate_id, captured_at: Time.now.utc)
-        raw_source = mail.to_s
+        raw_source = raw_source_for(mail)
         metadata = MessageSerializer.new(mail, id: id, captured_at: captured_at, raw_source: raw_source).metadata
         MessageRecord.new(id: id, metadata: metadata, raw_source: raw_source)
       end
 
+      def raw_source_for(mail)
+        return mail.to_s if MailDude.configuration.capture_attachments
+
+        AttachmentScrubber.new(mail).raw_source
+      end
+
       def generate_id
         time = Time.now.utc
         "#{time.strftime('%Y%m%dT%H%M%S')}#{format('%06d', time.usec)}Z-#{SecureRandom.hex(8)}"

data/lib/mail_dude/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MailDude
-  VERSION = '0.1.1'
+  VERSION = '0.1.3'
 end

data/lib/mail_dude.rb

@@ -17,6 +17,7 @@ require_relative 'mail_dude/pagination'
 require_relative 'mail_dude/message_serializer'
 require_relative 'mail_dude/message_presenter'
 require_relative 'mail_dude/attachment_locator'
+require_relative 'mail_dude/attachment_scrubber'
 require_relative 'mail_dude/html_body_renderer'
 require_relative 'mail_dude/message_broadcast'
 require_relative 'mail_dude/stores/base'

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: mail_dude
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.3
 platform: ruby
 authors:
 - MailDude contributors
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-05-08 00:00:00.000000000 Z
+date: 2026-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -185,6 +184,7 @@ files:
 - lib/generators/mail_dude/templates/initializer.tt
 - lib/mail_dude.rb
 - lib/mail_dude/attachment_locator.rb
+- lib/mail_dude/attachment_scrubber.rb
 - lib/mail_dude/configuration.rb
 - lib/mail_dude/dashboard.rb
 - lib/mail_dude/delivery_method.rb
@@ -208,7 +208,6 @@ licenses:
 - MIT
 metadata:
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -216,15 +215,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.2'
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.1
-signing_key:
+rubygems_version: 3.6.2
 specification_version: 4
 summary: A Rails Action Mailer capture engine for development and QA.
 test_files: []