gitlab-mail_room 0.0.24 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0e4c047999e1f2e3d77862ea0e65f146fe92c89de4d0367892649d518de8ad66
-  data.tar.gz: 36000daee0622b345eea3be14143652ce335ea9441ab012a4088d0038b7bef8c
+  metadata.gz: 0e8a5af986fff787ca0cd1ba45293f44a2f5130ddc8f1a2e762258d397d3eb7d
+  data.tar.gz: 6056b4c396bb63fd769ddd26c0860f65095045f84c334e7531f40b465c7f2326
 SHA512:
-  metadata.gz: 9693c9ee292e8e45025d251c720d4bda9f80b121aa0524c6c2bad0924b5ec731fcb017f91641c27adf2c7a3321a59345d1f0b80177bbde998ac5b42bc080a13d
-  data.tar.gz: d8c733b9eaef28122e6b2533e1e5fca6a0aba6581b07be93189f498f38ad4845ef624016258c339a2e6cd8ac56e7eb2fbe3c52a9724d5d9363deb58ed1d8d431
+  metadata.gz: c100b13d4179d816a39c308c36f0bb47e4c43bba80d549ef0a7f37cc70fb73d03dc3ecd554a50b4f350beb527bcff7cbeee999d737dcb7527d872d1606e204aa
+  data.tar.gz: d483eb80b21ee39e54b3a08ca18c3dad015027684dbb2d87b2b2d8b0c71a6bd72931932408034a6014b013ee653698703d9571a07385a162cfee0054fcd1fe8b

data/.github/workflows/ci.yml

@@ -26,12 +26,13 @@ jobs:
       matrix:
         ruby-version:
           - head
+          - '4.0'
+          - '3.4'
+          - '3.3'
           - '3.2'
           - '3.1'
-          - '3.0'
-          - '2.7'
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Set up Ruby ${{ matrix.ruby-version }}
       uses: ruby/setup-ruby@v1
       with:
@@ -43,7 +44,7 @@ jobs:
   rubocop:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Set up Ruby ${{ matrix.ruby-version }}
       uses: ruby/setup-ruby@v1
       with:

data/.gitlab-ci.yml

@@ -10,11 +10,10 @@ services:
     paths:
       - vendor/ruby
   variables:
-    REDIS_URL: redis://redis:6379/0
+    REDIS_URL: redis://redis:6379
   before_script:
     - apt update && apt install -y libicu-dev
     - ruby -v                                   # Print out ruby version for debugging
-    - gem install bundler --no-document         # Bundler is not installed with the image
     - bundle config set --local path 'vendor'
     - bundle install -j $(nproc)
   script:
@@ -23,8 +22,8 @@ services:
 rspec:
   parallel:
     matrix:
-      - RUBY_VERSION: [ "2.6", "2.7", "3.0", "3.1", "3.2" ]
+      - RUBY_VERSION: [ "3.1", "3.2", "3.3", "3.4", "4.0" ]
   <<: *test
 
 include:
-  - template: Security/Dependency-Scanning.gitlab-ci.yml
+  - template: Security/Dependency-Scanning.gitlab-ci.yml

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2023-03-15 01:41:21 UTC using RuboCop version 1.48.1.
+# on 2026-01-14 08:16:06 UTC using RuboCop version 1.82.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -8,24 +8,20 @@
 
 # Offense count: 5
 # This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: TreatCommentsAsGroupSeparators, ConsiderPunctuation, Include.
-# Include: **/*.gemspec
+# Configuration parameters: TreatCommentsAsGroupSeparators, ConsiderPunctuation.
 Gemspec/OrderedDependencies:
   Exclude:
     - 'mail_room.gemspec'
 
 # Offense count: 1
-# Configuration parameters: Severity, Include.
-# Include: **/*.gemspec
 Gemspec/RequiredRubyVersion:
   Exclude:
     - 'mail_room.gemspec'
 
-# Offense count: 5
+# Offense count: 1
 # This cop supports safe autocorrection (--autocorrect).
 Layout/BlockEndNewline:
   Exclude:
-    - 'spec/lib/delivery/postback_spec.rb'
     - 'spec/lib/delivery/que_spec.rb'
 
 # Offense count: 5
@@ -51,14 +47,13 @@ Layout/EmptyLinesAroundAccessModifier:
     - 'lib/mail_room/coordinator.rb'
     - 'lib/mail_room/delivery/que.rb'
 
-# Offense count: 4
+# Offense count: 3
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyle.
 # SupportedStyles: empty_lines, no_empty_lines
 Layout/EmptyLinesAroundBlockBody:
   Exclude:
     - 'spec/lib/crash_handler_spec.rb'
-    - 'spec/lib/delivery/sidekiq_spec.rb'
     - 'spec/lib/logger/structured_spec.rb'
 
 # Offense count: 1
@@ -69,13 +64,12 @@ Layout/EmptyLinesAroundClassBody:
   Exclude:
     - 'lib/mail_room/logger/structured.rb'
 
-# Offense count: 15
+# Offense count: 7
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyle, IndentationWidth.
 # SupportedStyles: special_inside_parentheses, consistent, align_braces
 Layout/FirstHashElementIndentation:
   Exclude:
-    - 'spec/lib/delivery/postback_spec.rb'
     - 'spec/lib/delivery/que_spec.rb'
     - 'spec/lib/logger/structured_spec.rb'
 
@@ -91,19 +85,26 @@ Layout/HashAlignment:
 
 # Offense count: 2
 # This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: AllowDoxygenCommentStyle, AllowGemfileRubyComment.
+# Configuration parameters: AllowDoxygenCommentStyle, AllowGemfileRubyComment, AllowRBSInlineAnnotation, AllowSteepAnnotation.
 Layout/LeadingCommentSpace:
   Exclude:
     - 'lib/mail_room/mailbox.rb'
     - 'spec/lib/arbitration/redis_spec.rb'
 
-# Offense count: 5
+# Offense count: 1
 # This cop supports safe autocorrection (--autocorrect).
 Layout/MultilineBlockLayout:
   Exclude:
-    - 'spec/lib/delivery/postback_spec.rb'
     - 'spec/lib/delivery/que_spec.rb'
 
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle, IndentationWidth.
+# SupportedStyles: aligned, indented, indented_relative_to_receiver
+Layout/MultilineMethodCallIndentation:
+  Exclude:
+    - 'lib/mail_room/imap/connection.rb'
+
 # Offense count: 3
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyle.
@@ -123,8 +124,9 @@ Layout/SpaceAroundKeyword:
 
 # Offense count: 2
 # This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: AllowForAlignment, EnforcedStyleForExponentOperator.
+# Configuration parameters: AllowForAlignment, EnforcedStyleForExponentOperator, EnforcedStyleForRationalLiterals.
 # SupportedStylesForExponentOperator: space, no_space
+# SupportedStylesForRationalLiterals: space, no_space
 Layout/SpaceAroundOperators:
   Exclude:
     - 'lib/mail_room/mailbox.rb'
@@ -141,7 +143,7 @@ Layout/SpaceBeforeBlockBraces:
     - 'spec/lib/crash_handler_spec.rb'
     - 'spec/lib/mailbox_spec.rb'
 
-# Offense count: 51
+# Offense count: 47
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters.
 # SupportedStyles: space, no_space
@@ -153,7 +155,6 @@ Layout/SpaceInsideBlockBraces:
     - 'spec/lib/configuration_spec.rb'
     - 'spec/lib/delivery/letter_opener_spec.rb'
     - 'spec/lib/delivery/logger_spec.rb'
-    - 'spec/lib/delivery/postback_spec.rb'
     - 'spec/lib/delivery/que_spec.rb'
     - 'spec/lib/imap/connection_spec.rb'
     - 'spec/lib/mailbox_watcher_spec.rb'
@@ -208,17 +209,18 @@ Lint/RescueException:
 
 # Offense count: 1
 # This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods, IgnoreNotImplementedMethods.
+# Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods, IgnoreNotImplementedMethods, NotImplementedExceptions.
+# NotImplementedExceptions: NotImplementedError
 Lint/UnusedMethodArgument:
   Exclude:
     - 'lib/mail_room/logger/structured.rb'
 
-# Offense count: 5
+# Offense count: 7
 # Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
 Metrics/AbcSize:
-  Max: 27
+  Max: 34
 
-# Offense count: 32
+# Offense count: 34
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 # AllowedMethods: refine
 Metrics/BlockLength:
@@ -229,12 +231,12 @@ Metrics/BlockLength:
 Metrics/ClassLength:
   Max: 169
 
-# Offense count: 1
+# Offense count: 3
 # Configuration parameters: AllowedMethods, AllowedPatterns.
 Metrics/CyclomaticComplexity:
   Max: 8
 
-# Offense count: 13
+# Offense count: 16
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 Metrics/MethodLength:
   Max: 22
@@ -244,14 +246,13 @@ Metrics/MethodLength:
 Metrics/ModuleLength:
   Max: 172
 
-# Offense count: 1
-Naming/AccessorMethodName:
-  Exclude:
-    - 'lib/mail_room/configuration.rb'
+# Offense count: 2
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/PerceivedComplexity:
+  Max: 9
 
 # Offense count: 1
-# This cop supports unsafe autocorrection (--autocorrect-all).
-Security/YAMLLoad:
+Naming/AccessorMethodName:
   Exclude:
     - 'lib/mail_room/configuration.rb'
 
@@ -263,7 +264,7 @@ Style/Alias:
   Exclude:
     - 'lib/mail_room/coordinator.rb'
 
-# Offense count: 16
+# Offense count: 12
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyle, ProceduralMethods, FunctionalMethods, AllowedMethods, AllowedPatterns, AllowBracesOnProceduralOneLiners, BracesRequiredMethods.
 # SupportedStyles: line_count_based, semantic, braces_for_chaining, always_braces
@@ -273,7 +274,6 @@ Style/Alias:
 Style/BlockDelimiters:
   Exclude:
     - 'spec/lib/arbitration/redis_spec.rb'
-    - 'spec/lib/delivery/postback_spec.rb'
     - 'spec/lib/delivery/que_spec.rb'
     - 'spec/lib/delivery/sidekiq_spec.rb'
 
@@ -289,6 +289,7 @@ Style/Documentation:
     - 'lib/mail_room/arbitration/redis.rb'
     - 'lib/mail_room/connection.rb'
     - 'lib/mail_room/crash_handler.rb'
+    - 'lib/mail_room/health_check.rb'
     - 'lib/mail_room/delivery.rb'
     - 'lib/mail_room/imap.rb'
     - 'lib/mail_room/imap/connection.rb'
@@ -349,7 +350,7 @@ Style/GuardClause:
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyle, EnforcedShorthandSyntax, UseHashRocketsWithSymbolValues, PreferHashRocketsForNonAlnumEndingSymbols.
 # SupportedStyles: ruby19, hash_rockets, no_mixed_keys, ruby19_no_mixed_keys
-# SupportedShorthandSyntax: always, never, either, consistent
+# SupportedShorthandSyntax: always, never, either, consistent, either_consistent
 Style/HashSyntax:
   Exclude:
     - 'spec/lib/microsoft_graph/connection_spec.rb'
@@ -390,7 +391,7 @@ Style/PreferredHashMethods:
     - 'lib/mail_room/mailbox.rb'
 
 # Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
+# This cop supports unsafe autocorrection (--autocorrect-all).
 # Configuration parameters: EnforcedStyle, AllowedCompactTypes.
 # SupportedStyles: compact, exploded
 Style/RaiseArgs:
@@ -443,7 +444,7 @@ Style/StringConcatenation:
   Exclude:
     - 'lib/mail_room/logger/structured.rb'
 
-# Offense count: 142
+# Offense count: 147
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyle, ConsistentQuotesInMultiline.
 # SupportedStyles: single_quotes, double_quotes
@@ -462,7 +463,7 @@ Style/SymbolArray:
 # Offense count: 2
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyleForMultiline.
-# SupportedStylesForMultiline: comma, consistent_comma, no_comma
+# SupportedStylesForMultiline: comma, consistent_comma, diff_comma, no_comma
 Style/TrailingCommaInHashLiteral:
   Exclude:
     - 'spec/lib/mailbox_spec.rb'
@@ -489,9 +490,9 @@ Style/WordArray:
   EnforcedStyle: percent
   MinSize: 3
 
-# Offense count: 7
+# Offense count: 10
 # This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns.
+# Configuration parameters: AllowHeredoc, AllowURI, AllowQualifiedName, URISchemes, AllowRBSInlineAnnotation, AllowCopDirectives, AllowedPatterns, SplitStrings.
 # URISchemes: http, https
 Layout/LineLength:
   Max: 177

data/.ruby-version

@@ -1 +1 @@
-2.7.5
+3.2

data/.tool-versions

@@ -0,0 +1 @@
+ruby 3.4.1

data/CHANGELOG.md

@@ -1,3 +1,18 @@
+## Unreleased ##
+
+* Bump `net-imap` to `>= 0.5, < 0.7` (allows the 0.6.x line on Ruby 3.2+)
+* Raise minimum Ruby to 3.1 (`required_ruby_version >= 3.1`), required by `net-imap` 0.5+
+* Modernize runtime dependency bounds: `oauth2 >= 2.0.9, < 3`, `jwt >= 2.0, < 4`, `redis-namespace >= 1.8.2, < 2`
+* Drop Ruby 3.0 from the CI matrix (GitLab CI and GitHub Actions now test 3.1–3.4 and 4.0)
+
+## mail_room 0.11.1 ##
+
+https://github.com/tpitale/mail_room/compare/v0.11.0...v0.11.1
+
+## mail_room 0.11.0 ##
+
+https://github.com/tpitale/mail_room/compare/v0.10.1...v0.11.0
+
 ## mail_room 0.10.1 ##
 
 * Fix db attribute on redis URL PR#130 - @jarkaK

data/README.md

@@ -489,13 +489,3 @@ respectively and MailRoom will log there.
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create new Pull Request
 6. If accepted, ask for commit rights
-
-## TODO ##
-
-1. specs, this is just a (working) proof of concept √
-2. finish code for POSTing to callback with auth √
-3. accept mailbox configuration for one account directly on the commandline; or ask for it
-4. add example rails endpoint, with auth examples
-5. add example configs for upstart/init.d √
-6. log to stdout √
-7. add a development mode that opens in letter_opener by ryanb √

data/lib/mail_room/arbitration/redis.rb

@@ -3,11 +3,15 @@ require "redis"
 module MailRoom
   module Arbitration
     class Redis
-      Options = Struct.new(:redis_url, :namespace, :sentinels) do
+      Options = Struct.new(:redis_url, :redis_ssl_params, :namespace, :redis_db, :sentinels, :sentinel_username, :sentinel_password) do
         def initialize(mailbox)
           redis_url = mailbox.arbitration_options[:redis_url] || "redis://localhost:6379"
+          redis_ssl_params = mailbox.arbitration_options[:redis_ssl_params]
+          redis_db  = mailbox.arbitration_options[:redis_db] || 0
           namespace = mailbox.arbitration_options[:namespace]
           sentinels = mailbox.arbitration_options[:sentinels]
+          sentinel_username = mailbox.arbitration_options[:sentinel_username]
+          sentinel_password = mailbox.arbitration_options[:sentinel_password]
 
           if namespace
             warn <<~MSG
@@ -16,7 +20,7 @@ module MailRoom
             MSG
           end
 
-          super(redis_url, namespace, sentinels)
+          super(redis_url, redis_ssl_params, namespace, redis_db, sentinels, sentinel_username, sentinel_password)
         end
       end
 
@@ -46,8 +50,13 @@ module MailRoom
       def client
         @client ||= begin
           sentinels = options.sentinels
-          redis_options = { url: options.redis_url }
+          redis_options = { url: options.redis_url, db: options.redis_db }
+          # Needed for redis-client 0.26.2 and older: https://github.com/redis-rb/redis-client/pull/277
+          redis_options[:ssl] = true if options.redis_url.start_with?("rediss://")
+          redis_options[:ssl_params] = options.redis_ssl_params if options.redis_ssl_params
           redis_options[:sentinels] = sentinels if sentinels
+          redis_options[:sentinel_username] = options.sentinel_username if options.sentinel_username
+          redis_options[:sentinel_password] = options.sentinel_password if options.sentinel_password
 
           redis = ::Redis.new(redis_options)
 

data/lib/mail_room/configuration.rb

@@ -15,7 +15,7 @@ module MailRoom
         begin
           erb = ERB.new(File.read(options[:config_path]))
           erb.filename = options[:config_path]
-          config_file = YAML.load(erb.result)
+          config_file = YAML.load(erb.result, symbolize_names: true)
 
           set_mailboxes(config_file[:mailboxes])
           set_health_check(config_file[:health_check])

data/lib/mail_room/delivery/postback.rb

@@ -61,7 +61,6 @@ module MailRoom
       # Build a new delivery, hold the delivery options
       # @param [MailRoom::Delivery::Postback::Options]
       def initialize(delivery_options)
-        puts delivery_options
         @delivery_options = delivery_options
       end
 
@@ -71,19 +70,17 @@ module MailRoom
         connection = Faraday.new
 
         if @delivery_options.token_auth?
-          connection.token_auth @delivery_options.token
+          config_token_auth(connection)
         elsif @delivery_options.basic_auth?
           config_basic_auth(connection)
         end
 
-        connection.post do |request|
-          request.url @delivery_options.url
-          request.body = message
+        result = connection.post(@delivery_options.url, message) do |request|
           config_request_content_type(request)
           config_request_jwt_auth(request)
         end
 
-        @delivery_options.logger.info({ delivery_method: 'Postback', action: 'message pushed', url: @delivery_options.url })
+        @delivery_options.logger.info({ delivery_method: 'Postback', action: 'message pushed', url: @delivery_options.url, status_code: result.status })
         true
       end
 
@@ -101,6 +98,18 @@ module MailRoom
         request.headers[@delivery_options.jwt.header] = @delivery_options.jwt.token
       end
 
+      def config_token_auth(connection)
+        # connection.token_auth was removed in Faraday v2 in favor of connection.request(:authorization, 'Token', token)
+        if defined?(connection.token_auth)
+          connection.token_auth @delivery_options.token
+        else
+          connection.request(
+            :authorization, 'Token',
+            @delivery_options.token
+          )
+        end
+      end
+
       def config_basic_auth(connection)
         if defined?(connection.basic_auth)
           connection.basic_auth(

data/lib/mail_room/delivery/sidekiq.rb

@@ -8,12 +8,15 @@ module MailRoom
     # Sidekiq Delivery method
     # @author Douwe Maan
     class Sidekiq
-      Options = Struct.new(:redis_url, :namespace, :sentinels, :queue, :worker, :logger, :redis_db) do
+      Options = Struct.new(:redis_url, :redis_ssl_params, :namespace, :sentinels, :queue, :worker, :logger, :redis_db, :sentinel_username, :sentinel_password) do
         def initialize(mailbox)
           redis_url = mailbox.delivery_options[:redis_url] || "redis://localhost:6379"
+          redis_ssl_params = mailbox.delivery_options[:redis_ssl_params]
           redis_db  = mailbox.delivery_options[:redis_db] || 0
           namespace = mailbox.delivery_options[:namespace]
           sentinels = mailbox.delivery_options[:sentinels]
+          sentinel_username = mailbox.delivery_options[:sentinel_username]
+          sentinel_password = mailbox.delivery_options[:sentinel_password]
           queue     = mailbox.delivery_options[:queue] || "default"
           worker    = mailbox.delivery_options[:worker]
           logger = mailbox.logger
@@ -25,7 +28,7 @@ module MailRoom
             MSG
           end
 
-          super(redis_url, namespace, sentinels, queue, worker, logger, redis_db)
+          super(redis_url, redis_ssl_params, namespace, sentinels, queue, worker, logger, redis_db, sentinel_username, sentinel_password)
         end
       end
 
@@ -54,7 +57,12 @@ module MailRoom
         @client ||= begin
           sentinels = options.sentinels
           redis_options = { url: options.redis_url, db: options.redis_db }
+          # Needed for redis-client 0.26.2 and older: https://github.com/redis-rb/redis-client/pull/277
+          redis_options[:ssl] = true if options.redis_url.start_with?("rediss://")
+          redis_options[:ssl_params] = options.redis_ssl_params if options.redis_ssl_params
           redis_options[:sentinels] = sentinels if sentinels
+          redis_options[:sentinel_username] = options.sentinel_username if options.sentinel_username
+          redis_options[:sentinel_password] = options.sentinel_password if options.sentinel_password
 
           redis = ::Redis.new(redis_options)
 

data/lib/mail_room/version.rb

@@ -1,4 +1,4 @@
 module MailRoom
   # Current version of gitlab-mail_room gem
-  VERSION = "0.0.24"
+  VERSION = "1.1.0"
 end

data/mail_room.gemspec

@@ -10,23 +10,25 @@ Gem::Specification.new do |gem|
   gem.email         = ["tpitale@gmail.com"]
   gem.description   = %q{mail_room will proxy email (gmail) from IMAP to a delivery method}
   gem.summary       = %q{mail_room will proxy email (gmail) from IMAP to a callback URL, logger, or letter_opener}
-  gem.homepage      = "http://github.com/tpitale/mail_room"
+  gem.homepage      = "https://gitlab.com/gitlab-org/ruby/gems/gitlab-mail_room"
 
   gem.files         = `git ls-files`.split($/)
   gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
-  gem.add_dependency             "net-imap", ">= 0.2.1"
-  gem.add_dependency             "oauth2", [">= 1.4.4", "< 3"]
-  gem.add_dependency             "jwt", ">= 2.0"
-  gem.add_dependency             "redis", [">= 4", "< 6"]
-  gem.add_dependency             "redis-namespace", '>= 1.8.2'
+  gem.required_ruby_version = ">= 3.1"
+
+  gem.add_dependency             "net-imap", [">= 0.5", "< 0.7"]
+  gem.add_dependency             "oauth2", [">= 2.0.9", "< 3"]
+  gem.add_dependency             "jwt", [">= 2.0", "< 4"]
+  gem.add_dependency             "redis", [">= 5", "< 6"]
+  gem.add_dependency             "redis-namespace", [">= 1.8.2", "< 2"]
 
   gem.add_development_dependency "rake"
   gem.add_development_dependency "rspec", "~> 3.9"
   gem.add_development_dependency "rubocop", "~> 1.11"
-  gem.add_development_dependency "mocha", "~> 1.11"
+  gem.add_development_dependency "mocha", "~> 2.0"
   gem.add_development_dependency "simplecov"
   gem.add_development_dependency "webrick", "~> 1.6"
 

data/spec/lib/arbitration/redis_spec.rb

@@ -11,21 +11,12 @@ describe MailRoom::Arbitration::Redis do
     )
   }
   let(:options) { described_class::Options.new(mailbox) }
-  let(:redis5) { Gem::Version.new(Redis::VERSION) >= Gem::Version.new('5.0') }
   subject       { described_class.new(options) }
 
   # Private, but we don't care.
   let(:redis) { subject.send(:client) }
   let(:raw_client) { redis._client }
 
-  let(:server_url) do
-    if redis5
-      raw_client.config.server_url
-    else
-      raw_client.options[:url]
-    end
-  end
-
   describe '#deliver?' do
     context "when called the first time" do
       after do
@@ -89,7 +80,7 @@ describe MailRoom::Arbitration::Redis do
 
   context 'redis client connection params' do
     context 'when only url is present' do
-      let(:redis_url) { ENV.fetch('REDIS_URL', 'redis://localhost:6379/0') }
+      let(:redis_url) { ENV.fetch('REDIS_URL', 'redis://localhost:6379') }
       let(:mailbox) {
         build_mailbox(
           arbitration_options: {
@@ -105,7 +96,7 @@ describe MailRoom::Arbitration::Redis do
       it 'client has same specified url' do
         subject.deliver?(123)
 
-        expect(server_url).to eq redis_url
+        expect(raw_client.config.server_url).to eq redis_url
       end
 
       it 'client is a instance of Redis class' do
@@ -144,19 +135,62 @@ describe MailRoom::Arbitration::Redis do
         )
       }
 
+      before { ::RedisClient::SentinelConfig.any_instance.stubs(:resolve_master).returns(RedisClient::Config.new(**sentinels.first)) }
+
       it 'client has same specified sentinel params' do
-        if redis5
+        expect(raw_client.config).to be_a RedisClient::SentinelConfig
+        expect(raw_client.config.name).to eq('sentinel-master')
+        expect(raw_client.config.host).to eq('10.0.0.1')
+        expect(raw_client.config.password).to eq('mypassword')
+        expect(raw_client.config.sentinels.map(&:server_url)).to eq(["redis://10.0.0.1:26379"])
+      end
+
+      context 'with separate Sentinel username and password' do
+        let(:sentinel_username) { 'my-sentinel-user' }
+        let(:sentinel_password) { 'my-sentinel-pass' }
+        let(:mailbox) {
+          build_mailbox(
+            arbitration_options: {
+              redis_url: redis_url,
+              sentinels: sentinels,
+              sentinel_username: sentinel_username,
+              sentinel_password: sentinel_password
+            }
+          )
+        }
+
+        it 'client uses Sentinel username and password' do
+          expect(raw_client.config).to be_a RedisClient::SentinelConfig
           expect(raw_client.config.password).to eq('mypassword')
-          client_sentinels = raw_client.config.sentinels
-          expect(client_sentinels.length).to eq(sentinels.length)
-          expect(client_sentinels[0].host).to eq('10.0.0.1')
-          expect(client_sentinels[0].port).to eq(26379) # rubocop:disable Style/NumericLiterals
-        else
-          expect(raw_client.options[:host]).to eq('sentinel-master')
-          expect(raw_client.options[:password]).to eq('mypassword')
-          expect(raw_client.options[:sentinels]).to eq(sentinels)
+
+          sentinels = raw_client.config.sentinels
+          expect(sentinels.map(&:username).uniq).to eq([sentinel_username])
+          expect(sentinels.map(&:password).uniq).to eq([sentinel_password])
         end
       end
     end
+
+    context 'when redis_ssl_params is specified' do
+      let(:redis_url) { ENV['REDIS_URL'] }
+      let(:redis_ssl_params) { { verify_mode: OpenSSL::SSL::VERIFY_NONE } }
+      let(:mailbox) {
+        build_mailbox(
+          arbitration_options: {
+            redis_url: redis_url,
+            redis_ssl_params: redis_ssl_params
+          }
+        )
+      }
+
+      after do
+        redis.del("delivered:123")
+      end
+
+      it 'client has same specified ssl_params' do
+        subject.deliver?(123)
+
+        expect(raw_client.config.ssl_params).to eq(redis_ssl_params)
+      end
+    end
   end
 end

data/spec/lib/coordinator_spec.rb

@@ -17,7 +17,7 @@ describe MailRoom::Coordinator do
     end
 
     it 'sets the health check' do
-      health_check = MailRoom::HealthCheck.new({ address: '127.0.0.1', port: 8080})
+      health_check = MailRoom::HealthCheck.new({ address: '127.0.0.1', port: 8080 })
       coordinator = MailRoom::Coordinator.new([], health_check)
 
       expect(coordinator.health_check).to eq(health_check)

data/spec/lib/delivery/postback_spec.rb

@@ -1,138 +1,112 @@
 require 'spec_helper'
 require 'mail_room/delivery/postback'
+require 'tempfile'
+require 'base64'
+require 'webmock/rspec'
 
 describe MailRoom::Delivery::Postback do
   describe '#deliver' do
-    context 'with token auth delivery' do
-      let(:mailbox) {build_mailbox({
-        delivery_url: 'http://localhost/inbox',
-        delivery_token: 'abcdefg'
-      })}
+    let(:delivery_options) do
+      MailRoom::Delivery::Postback::Options.new(mailbox)
+    end
 
-      let(:delivery_options) {
-        MailRoom::Delivery::Postback::Options.new(mailbox)
-      }
+    before do
+      stub_request(:post, 'http://localhost/inbox')
+        .with(body: 'a message', headers: headers)
+        .to_return(status: 201)
+    end
 
+    shared_examples 'message poster' do
       it 'posts the message with faraday' do
-        connection = stub
-        request = stub
-        Faraday.stubs(:new).returns(connection)
-
-        connection.expects(:token_auth).with('abcdefg')
-        connection.expects(:post).yields(request)
-
-        request.expects(:url).with('http://localhost/inbox')
-        request.expects(:body=).with('a message')
-
         MailRoom::Delivery::Postback.new(delivery_options).deliver('a message')
       end
     end
 
-    context 'with basic auth delivery options' do
-      let(:mailbox) {build_mailbox({
-        delivery_options: {
-          url: 'http://localhost/inbox',
-          username: 'user1',
-          password: 'password123abc'
-        }
-      })}
-
-      let(:delivery_options) {
-        MailRoom::Delivery::Postback::Options.new(mailbox)
-      }
-
-      it 'posts the message with faraday' do
-        connection = stub
-        request = stub
-        Faraday.stubs(:new).returns(connection)
+    context 'with token auth delivery' do
+      let(:mailbox) do
+        build_mailbox({
+                        delivery_url: 'http://localhost/inbox',
+                        delivery_token: 'abcdefg'
+                      })
+      end
 
-        connection.expects(:basic_auth).with('user1', 'password123abc')
-        connection.expects(:post).yields(request)
+      let(:headers) { { 'Authorization' => 'Token abcdefg' } }
 
-        request.expects(:url).with('http://localhost/inbox')
-        request.expects(:body=).with('a message')
+      it_behaves_like 'message poster'
+    end
 
-        MailRoom::Delivery::Postback.new(delivery_options).deliver('a message')
+    context 'with basic auth delivery options' do
+      let(:mailbox) do
+        build_mailbox({
+                        delivery_options: {
+                          url: 'http://localhost/inbox',
+                          username: 'user1',
+                          password: 'password123abc'
+                        }
+                      })
       end
 
-      context 'with content type in the delivery options' do
-        let(:mailbox) {build_mailbox({
-          delivery_options: {
-            url: 'http://localhost/inbox',
-            username: 'user1',
-            password: 'password123abc',
-            content_type: 'text/plain'
-          }
-        })}
-
-        let(:delivery_options) {
-          MailRoom::Delivery::Postback::Options.new(mailbox)
-        }
+      let(:headers) { { 'Authorization' => 'Basic dXNlcjE6cGFzc3dvcmQxMjNhYmM=' } }
 
-        it 'posts the message with faraday' do
-          connection = stub
-          request = stub
-          Faraday.stubs(:new).returns(connection)
+      it_behaves_like 'message poster'
 
-          connection.expects(:post).yields(request)
-          request.stubs(:url)
-          request.stubs(:body=)
-          request.stubs(:headers).returns({})
-          connection.expects(:basic_auth).with('user1', 'password123abc')
-
-          MailRoom::Delivery::Postback.new(delivery_options).deliver('a message')
+      context 'with content type in the delivery options' do
+        let(:mailbox) do
+          build_mailbox({
+                          delivery_options: {
+                            url: 'http://localhost/inbox',
+                            username: 'user1',
+                            password: 'password123abc',
+                            content_type: 'text/plain'
+                          }
+                        })
+        end
 
-          expect(request.headers['Content-Type']).to eq('text/plain')
+        let(:headers) do
+          {
+            'Authorization' => 'Basic dXNlcjE6cGFzc3dvcmQxMjNhYmM=',
+            'Content-Type' => 'text/plain'
+          }
         end
+
+        it_behaves_like 'message poster'
       end
 
       context 'with jwt token in the delivery options' do
-        let(:mailbox) {build_mailbox({
-          delivery_options: {
-            url: 'http://localhost/inbox',
-            jwt_auth_header: "Mailroom-Api-Request",
-            jwt_issuer: "mailroom",
-            jwt_algorithm: "HS256",
-            jwt_secret_path: "secret_path"
+        let(:mailbox) do
+          build_mailbox({
+                          delivery_options: {
+                            url: 'http://localhost/inbox',
+                            jwt_auth_header: "Mailroom-Api-Request",
+                            jwt_issuer: "mailroom",
+                            jwt_algorithm: "HS256",
+                            jwt_secret_path: jwt_secret.path,
+                            content_type: 'application/json'
+                          }
+                        })
+        end
+
+        let(:headers) do
+          {
+            'Content-Type' => 'application/json',
+            'Mailroom-Api-Request' => /.*/
           }
-        })}
-
-        let(:delivery_options) {
-          MailRoom::Delivery::Postback::Options.new(mailbox)
-        }
-
-        it 'posts the message with faraday' do
-          connection = stub
-          request = stub
-          Faraday.stubs(:new).returns(connection)
-
-          connection.expects(:post).yields(request).twice
-          request.stubs(:url)
-          request.stubs(:body=)
-          request.stubs(:headers).returns({})
-
-          jwt = stub
-          MailRoom::JWT.expects(:new).with(
-            header: 'Mailroom-Api-Request',
-            issuer: 'mailroom',
-            algorithm: 'HS256',
-            secret_path: 'secret_path'
-          ).returns(jwt)
-          jwt.stubs(:valid?).returns(true)
-          jwt.stubs(:header).returns('Mailroom-Api-Request')
-          jwt.stubs(:token).returns('a_jwt_token')
-
-          delivery = MailRoom::Delivery::Postback.new(delivery_options)
-
-          delivery.deliver('a message')
-          expect(request.headers['Mailroom-Api-Request']).to eql('a_jwt_token')
-
-          # A different jwt token for the second time
-          jwt.stubs(:token).returns('another_jwt_token')
-
-          delivery.deliver('another message')
-          expect(request.headers['Mailroom-Api-Request']).to eql('another_jwt_token')
         end
+
+        let(:jwt_secret) do
+          file = Tempfile.new('secret')
+          # The secret is read back and Base64-decoded by MailRoom::JWT, so it
+          # must be valid Base64 and flushed to disk before it is read.
+          file.write(Base64.strict_encode64("test secret"))
+          file.flush
+          file
+        end
+
+        after do
+          jwt_secret.unlink
+        end
+
+        it_behaves_like 'message poster'
       end
     end
   end

data/spec/lib/delivery/sidekiq_spec.rb

@@ -6,17 +6,9 @@ describe MailRoom::Delivery::Sidekiq do
   let(:redis) { subject.send(:client) }
   let(:raw_client) { redis._client }
   let(:options) { MailRoom::Delivery::Sidekiq::Options.new(mailbox) }
-  let(:redis5) { Gem::Version.new(Redis::VERSION) >= Gem::Version.new('5.0') }
-  let(:server_url) do
-    if redis5
-      raw_client.config.server_url
-    else
-      raw_client.options[:url]
-    end
-  end
 
   describe '#options' do
-    let(:redis_url) { 'redis://localhost' }
+    let(:redis_url) { 'redis://localhost:6379' }
     let(:redis_options) { { redis_url: redis_url } }
 
     context 'when only redis_url is specified' do
@@ -28,10 +20,8 @@ describe MailRoom::Delivery::Sidekiq do
       }
 
       context 'with simple redis url' do
-        let(:expected_url) { redis5 ? "#{redis_url}:6379/0" : redis_url }
-
         it 'client has same specified redis_url' do
-          expect(raw_client.server_url).to eq(expected_url)
+          expect(raw_client.config.server_url).to eq(redis_url)
         end
 
         it 'client is a instance of RedisNamespace class' do
@@ -45,19 +35,17 @@ describe MailRoom::Delivery::Sidekiq do
       end
 
       context 'with redis_db specified in options' do
-        let(:expected_url) { redis5 ? "#{redis_url}:6379/4" : redis_url }
-
         before do
           redis_options[:redis_db] = 4
         end
 
         it 'client has correct redis_url' do
-          expect(raw_client.server_url).to eq(expected_url)
+          expect(raw_client.config.server_url).to eq("#{redis_url}/4")
         end
 
         it 'connection has correct values' do
-          expect(redis.connection[:host]).to eq('localhost')
-          expect(redis.connection[:db]).to eq(4)
+          expect(raw_client.config.host).to eq('localhost')
+          expect(raw_client.config.db).to eq(4)
         end
       end
     end
@@ -96,19 +84,59 @@ describe MailRoom::Delivery::Sidekiq do
         )
       }
 
+      before { ::RedisClient::SentinelConfig.any_instance.stubs(:resolve_master).returns(RedisClient::Config.new(**sentinels.first)) }
+
       it 'client has same specified sentinel params' do
-        if redis5
+        expect(raw_client.config).to be_a RedisClient::SentinelConfig
+        expect(raw_client.config.host).to eq('10.0.0.1')
+        expect(raw_client.config.name).to eq('sentinel-master')
+        expect(raw_client.config.password).to eq('mypassword')
+        expect(raw_client.config.sentinels.map(&:server_url)).to eq(["redis://10.0.0.1:26379"])
+      end
+
+      context 'with separate Sentinel username and password' do
+        let(:sentinel_username) { 'my-sentinel-user' }
+        let(:sentinel_password) { 'my-sentinel-pass' }
+        let(:mailbox) {
+          build_mailbox(
+            delivery_method: :sidekiq,
+            delivery_options: {
+              redis_url: redis_url,
+              sentinels: sentinels,
+              sentinel_username: sentinel_username,
+              sentinel_password: sentinel_password
+            }
+          )
+        }
+
+        it 'client uses Sentinel username and password' do
+          expect(raw_client.config).to be_a RedisClient::SentinelConfig
           expect(raw_client.config.password).to eq('mypassword')
-          client_sentinels = raw_client.config.sentinels
-          expect(client_sentinels.length).to eq(sentinels.length)
-          expect(client_sentinels[0].host).to eq('10.0.0.1')
-          expect(client_sentinels[0].port).to eq(26379) # rubocop:disable Style/NumericLiterals
-        else
-          expect(raw_client.options[:host]).to eq('sentinel-master')
-          expect(raw_client.options[:password]).to eq('mypassword')
-          expect(raw_client.options[:sentinels]).to eq(sentinels)
+
+          sentinels = raw_client.config.sentinels
+
+          expect(sentinels.map(&:username).uniq).to eq([sentinel_username])
+          expect(sentinels.map(&:password).uniq).to eq([sentinel_password])
         end
       end
     end
+
+    context 'when redis_ssl_params is specified' do
+      let(:redis_url) { ENV['REDIS_URL'] }
+      let(:redis_ssl_params) { { verify_mode: OpenSSL::SSL::VERIFY_NONE } }
+      let(:mailbox) {
+        build_mailbox(
+          delivery_method: :sidekiq,
+          delivery_options: {
+            redis_url: redis_url,
+            redis_ssl_params: redis_ssl_params
+          }
+        )
+      }
+
+      it 'client has same specified ssl_params' do
+        expect(raw_client.config.ssl_params).to eq(redis_ssl_params)
+      end
+    end
   end
 end

data/spec/lib/health_check_spec.rb

@@ -34,12 +34,14 @@ describe MailRoom::HealthCheck do
 
   describe '#run' do
     it 'sets running to true' do
-      server = stub(start: true)
+      server = stub(start: true, shutdown: true)
       subject.stubs(:create_server).returns(server)
 
       subject.run
 
       expect(subject.running).to be true
+
+      subject.quit
     end
   end
 

data/spec/lib/microsoft_graph/connection_spec.rb

@@ -106,7 +106,7 @@ describe MailRoom::MicrosoftGraph::Connection do
 
     shared_examples 'with a single message' do
       let(:message_id) { SecureRandom.hex }
-      let(:unread_messages_body) { { value: ['id' => message_id] } }
+      let(:unread_messages_body) { { value: [{ 'id' => message_id }] } }
       let(:message_url) { "#{message_base_url}/#{message_id}" }
       let(:message_body) { 'hello world' }
 
@@ -168,13 +168,13 @@ describe MailRoom::MicrosoftGraph::Connection do
     context 'with multiple pages of messages' do
       let(:message_ids) { [SecureRandom.hex, SecureRandom.hex] }
       let(:next_page_url) { "#{graph_endpoint}/v1.0/nextPage" }
-      let(:unread_messages_body) { { value: ['id' => message_ids.first], '@odata.nextLink' => next_page_url } }
+      let(:unread_messages_body) { { value: [{ 'id' => message_ids.first }], '@odata.nextLink' => next_page_url } }
       let(:message_body) { 'hello world' }
 
       it 'requests message ID' do
         stub_request(:get, next_page_url).to_return(
           status: 200,
-          body: { value: ['id' => message_ids[1]] }.to_json
+          body: { value: [{ 'id' => message_ids[1] }] }.to_json
         )
 
         stubs = []

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-mail_room
 version: !ruby/object:Gem::Version
-  version: 0.0.24
+  version: 1.1.0
 platform: ruby
 authors:
 - Tony Pitale
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-12-08 00:00:00.000000000 Z
+date: 2026-06-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: net-imap
@@ -16,21 +15,27 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.2.1
+        version: '0.5'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '0.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.2.1
+        version: '0.5'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: oauth2
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.4.4
+        version: 2.0.9
     - - "<"
       - !ruby/object:Gem::Version
         version: '3'
@@ -40,7 +45,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.4.4
+        version: 2.0.9
     - - "<"
       - !ruby/object:Gem::Version
         version: '3'
@@ -51,6 +56,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '2.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -58,13 +66,16 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '2.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4'
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4'
+        version: '5'
     - - "<"
       - !ruby/object:Gem::Version
         version: '6'
@@ -74,7 +85,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4'
+        version: '5'
     - - "<"
       - !ruby/object:Gem::Version
         version: '6'
@@ -85,6 +96,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.8.2
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -92,6 +106,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.8.2
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -140,14 +157,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.11'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.11'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -277,6 +294,7 @@ files:
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
 - ".ruby-version"
+- ".tool-versions"
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - CONTRIBUTING.md
@@ -337,10 +355,9 @@ files:
 - spec/lib/message_spec.rb
 - spec/lib/microsoft_graph/connection_spec.rb
 - spec/spec_helper.rb
-homepage: http://github.com/tpitale/mail_room
+homepage: https://gitlab.com/gitlab-org/ruby/gems/gitlab-mail_room
 licenses: []
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -348,15 +365,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.22
-signing_key:
+rubygems_version: 3.6.2
 specification_version: 4
 summary: mail_room will proxy email (gmail) from IMAP to a callback URL, logger, or
   letter_opener