schleuder 4.0.2 → 4.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9f8275838177195ab78283d9c737de9cbd2d39e816c9f41db2e31d785d787dd8
-  data.tar.gz: 464e220965c5d0ee43d300a1f988e7a27edd9344d751bae9e06a9a02fc5c2a43
+  metadata.gz: c44b12c722680d91cee45a80ae2ee6a4ff47a9544dfd171cc83a5549bdd72be2
+  data.tar.gz: 77773ef59d05b1faff86cf2fe5d22ef657cd5683e6184db1a55b28d1592b07d2
 SHA512:
-  metadata.gz: d6ba6aab5c19ae0f8f74a0fe44e84e5a10f4adf434f55454a1d7ca49943557fbad5db23ff06fee1a3e0efa5a248034f8a6a17dcbeb91864bf04ec68f8cd14d1f
-  data.tar.gz: bd361a32b63c6bcfa146e1369a45fa54e9c3c94d29baaefbbff01a1629b468c896a34c17cd6ff957357eaf67611a5b09b5c2b2d9a023130430a86f29892b19cb
+  metadata.gz: eb351c776f1773cf0dd5ef0f04f310fe70a0155d9344fd75eb78fce9b0d69cad0f0d9bfad664d1a8ab004a55c2d29a849cf1ea2cd0a295c61c3b56b96a25d9a8
+  data.tar.gz: 97ab8283c694fac04196ee752ffe3153f57acb1e37c8754c8c9f1840fbf5109d47fac3426aa8fa3f168341efbfcf38c8d4292e9a360b3d81dff5d9231ac9b700

data/README.md

@@ -36,15 +36,15 @@ Additionally these **rubygems** are required (will be installed automatically un
 Installing Schleuder
 ------------
 
-1. Download [the gem](https://schleuder.org/download/schleuder-4.0.2.gem) and [the OpenPGP-signature](https://schleuder.org/download/schleuder-4.0.2.gem.sig) and verify:
+1. Download [the gem](https://schleuder.org/download/schleuder-4.0.3.gem) and [the OpenPGP-signature](https://schleuder.org/download/schleuder-4.0.3.gem.sig) and verify:
    ```
    gpg --recv-key 0xB3D190D5235C74E1907EACFE898F2C91E2E6E1F3
-   gpg --verify schleuder-4.0.2.gem.sig
+   gpg --verify schleuder-4.0.3.gem.sig
    ```
 
 2. If all went well install the gem:
    ```
-   gem install schleuder-4.0.2.gem
+   gem install schleuder-4.0.3.gem
    ```
 
 3. Set up schleuder:
@@ -134,4 +134,4 @@ GNU GPL 3.0. Please see [LICENSE.txt](LICENSE.txt).
 Alternative Download
 --------------------
 
-Alternatively to the gem-files you can download the latest release as [a tarball](https://schleuder.org/download/schleuder-4.0.2.tar.gz) and [its OpenPGP-signature](https://schleuder.org/download/schleuder-4.0.2.tar.gz.sig).
+Alternatively to the gem-files you can download the latest release as [a tarball](https://schleuder.org/download/schleuder-4.0.3.tar.gz) and [its OpenPGP-signature](https://schleuder.org/download/schleuder-4.0.3.tar.gz.sig).

data/db/migrate/20211106112020_change_boolean_values_to_integers.rb

@@ -0,0 +1,46 @@
+# Since ActiveRecord >= 6.0, the SQLite3 connection adapter relies on boolean
+# serialization to use 1 and 0, but does not natively recognize 't' and 'f' as
+# booleans were previously serialized.
+#
+# Accordingly, this migration handles conversion of both column defaults and
+# stored data provided by a user.
+#
+# In contrast to other migrations, only a 'forward' method is provided, a
+# mechanism to 'reverse' is not. Given the nature of this migration, the later
+# is not really required.
+#
+# Unfortunately, we missed this breaking change when bumping ActiveRecord to >=
+# 6.0 in Schleuder version 4.0. This caused quite some work upstream, but also
+# in downstream environments and, last but not least, at the side of users.
+#
+# We should extend our CI to explicitly test, and ensure things work as
+# expected, if running a Schleuder setup in real world. As of now, we don't
+# ensure data provided by a user in Schleuder version x still works after
+# upgrading to version y.
+
+class ChangeBooleanValuesToIntegers < ActiveRecord::Migration[6.0]
+  class Lists < ActiveRecord::Base
+  end
+
+  class Subscriptions < ActiveRecord::Base
+  end
+
+  def up
+    [Lists, Subscriptions].each do |table|
+      unless table.connection.is_a?(ActiveRecord::ConnectionAdapters::SQLite3Adapter)
+        return
+      end
+
+      bool_columns_defaults = table.columns.select { |column| column.type == :boolean }.map{ |column| [column.name, column.default] }
+
+      bool_columns_defaults.each do |column_name, column_default|
+        column_bool = ActiveRecord::Type::Boolean.new.deserialize(column_default)
+        
+        change_column_default :"#{table.table_name}", :"#{column_name}", column_bool
+        
+        table.where("#{column_name} = 'f'").update_all("#{column_name}": 0)
+        table.where("#{column_name} = 't'").update_all("#{column_name}": 1)
+      end
+    end
+  end
+end

data/db/migrate/20211107151309_add_limits_to_string_columns.rb

@@ -0,0 +1,28 @@
+# It seems, the change to ActiveRecord >= 6.0 makes this necessary.
+# Without doing this, the auto-generated database schema file would drop
+# these limits, if running migrations.
+#
+# In contrast to other migrations, only a 'forward' method is provided, a
+# mechanism to 'reverse' is not. Given the nature of this migration, the later
+# is not really required.
+#
+# This has been an upstream issue for quite some time. For details, see
+# https://github.com/rails/rails/issues/19001.
+
+class AddLimitsToStringColumns < ActiveRecord::Migration[6.0]
+  class Lists < ActiveRecord::Base
+  end
+
+  class Subscriptions < ActiveRecord::Base
+  end
+
+  def up
+    [Lists, Subscriptions].each do |table|
+      string_columns = table.columns.select { |column| column.type == :string }.map(&:name)
+
+      string_columns.each do |column_name|
+        change_column :"#{table.table_name}", :"#{column_name}", :string, :limit => 255
+      end
+    end
+  end
+end

data/db/schema.rb

@@ -10,18 +10,18 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2020_01_18_170110) do
+ActiveRecord::Schema.define(version: 2021_11_07_151309) do
 
   create_table "lists", force: :cascade do |t|
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.string "email"
-    t.string "fingerprint"
-    t.string "log_level", default: "warn"
-    t.string "subject_prefix", default: ""
-    t.string "subject_prefix_in", default: ""
-    t.string "subject_prefix_out", default: ""
-    t.string "openpgp_header_preference", default: "signencrypt"
+    t.string "email", limit: 255
+    t.string "fingerprint", limit: 255
+    t.string "log_level", limit: 255, default: "warn"
+    t.string "subject_prefix", limit: 255, default: ""
+    t.string "subject_prefix_in", limit: 255, default: ""
+    t.string "subject_prefix_out", limit: 255, default: ""
+    t.string "openpgp_header_preference", limit: 255, default: "signencrypt"
     t.text "public_footer", default: ""
     t.text "headers_to_meta", default: "[\"from\", \"to\", \"cc\", \"date\", \"sig\", \"enc\"]"
     t.text "bounces_drop_on_headers", default: "{\"x-spam-flag\":\"yes\"}"
@@ -39,7 +39,7 @@ ActiveRecord::Schema.define(version: 2020_01_18_170110) do
     t.boolean "include_list_headers", default: true
     t.boolean "include_openpgp_header", default: true
     t.integer "max_message_size_kb", default: 10240
-    t.string "language", default: "en"
+    t.string "language", limit: 255, default: "en"
     t.boolean "forward_all_incoming_to_admins", default: false
     t.integer "logfiles_to_keep", default: 2
     t.text "internal_footer", default: ""
@@ -51,8 +51,8 @@ ActiveRecord::Schema.define(version: 2020_01_18_170110) do
 
   create_table "subscriptions", force: :cascade do |t|
     t.integer "list_id"
-    t.string "email"
-    t.string "fingerprint"
+    t.string "email", limit: 255
+    t.string "fingerprint", limit: 255
     t.boolean "admin", default: false
     t.boolean "delivery_enabled", default: true
     t.datetime "created_at"

data/lib/schleuder/cli.rb

@@ -132,7 +132,7 @@ module Schleuder
       def shellexec(cmd)
         result = `#{cmd} 2>&1`
         if $?.exitstatus > 0
-          exit $?.exitstatus
+          fatal result, $?.exitstatus
         end
         result
       end

data/lib/schleuder/filters_runner.rb

@@ -13,11 +13,7 @@ module Schleuder
           list.logger.debug "Calling filter #{cmd}"
           response = Filters.send(cmd, list, mail)
           if stop?(response)
-            if bounce?(response, mail)
-              return response
-            else
-              return nil
-            end
+            return response
           end
         end
         nil
@@ -32,31 +28,6 @@ module Schleuder
         response.kind_of?(StandardError)
       end
 
-      def bounce?(response, mail)
-        if list.bounces_drop_all
-          list.logger.debug 'Dropping bounce as configurated'
-          notify_admins(I18n.t('.bounces_drop_all'), mail.original_message)
-          return false
-        end
-
-        list.bounces_drop_on_headers.each do |key, value|
-          if mail[key].to_s.match(/#{value}/i)
-            list.logger.debug "Incoming message header key '#{key}' matches value '#{value}': dropping the bounce."
-            notify_admins(I18n.t('.bounces_drop_on_headers', key: key, value: value), mail.original_message)
-            return false
-          end
-        end
-
-        list.logger.debug 'Bouncing message'
-        true
-      end
-
-      def notify_admins(reason, original_message)
-        if list.bounces_notify_admins?
-          list.logger.notify_admin reason, original_message, I18n.t('notice')
-        end
-      end
-
       def load_filters
         list.logger.debug "Loading #{filter_type}_decryption filters"
         sorted_filters.map do |filter_name|

data/lib/schleuder/runner.rb

@@ -31,7 +31,13 @@ module Schleuder
       end
 
       error = run_filters('pre')
-      return error if error
+      if error
+        if bounce?(error, @mail)
+          return error
+        else
+          return nil
+        end
+      end
 
       begin
         # This decrypts, verifies, etc.
@@ -48,7 +54,13 @@ module Schleuder
       end
 
       error = run_filters('post')
-      return error if error
+      if error
+        if bounce?(error, @mail)
+          return error
+        else
+          return nil
+        end
+      end
 
       if ! @mail.was_validly_signed?
         logger.debug 'Message was not validly signed, adding subject_prefix_in'
@@ -85,16 +97,34 @@ module Schleuder
       @list
     end
 
-    def run_filters(filter_type)
-      error = filters_runner(filter_type).run(@mail)
-      if error
-        if list.bounces_notify_admins?
-          text = "#{I18n.t('.bounces_notify_admins')}\n\n#{error}"
-          # TODO: raw_source is mostly blank?
-          logger.notify_admin text, @mail.original_message, I18n.t('notice')
+    def notify_admins(reason, original_message)
+      if list.bounces_notify_admins
+        list.logger.notify_admin reason, original_message, I18n.t('notice')
+      end
+    end
+
+    def bounce?(response, mail)
+      if list.bounces_drop_all
+        list.logger.debug 'Dropping bounce as configurated'
+        notify_admins(I18n.t('.bounces_drop_all'), mail.original_message)
+        return false
+      end
+
+      list.bounces_drop_on_headers.each do |key, value|
+        if mail[key].to_s.match(/#{value}/i)
+          list.logger.debug "Incoming message header key '#{key}' matches value '#{value}': dropping the bounce."
+          notify_admins(I18n.t('.bounces_drop_on_headers', key: key, value: value), mail.original_message)
+          return false
         end
-        return error
       end
+
+      list.logger.debug 'Bouncing message'
+      notify_admins("#{I18n.t('.bounces_notify_admins')}\n\n#{response}", mail.original_message)
+      true
+    end
+
+    def run_filters(filter_type)
+      filters_runner(filter_type).run(@mail)
     end
 
     def filters_runner(filter_type)

data/lib/schleuder/version.rb

@@ -1,3 +1,3 @@
 module Schleuder
-  VERSION = '4.0.2'
+  VERSION = '4.0.3'
 end

data/locales/de.yml

@@ -11,12 +11,12 @@ de:
         invalid: "enthält nicht druckbare Zeichen"
       public_footer:
         invalid: "enthält nicht druckbare Zeichen"
-    invalid_email: "ist keine valide E-Mail-Adresse"
-    invalid_fingerprint: "ist kein valider OpenPGP-Fingerabdruck"
-    list_fingerprint_missing: "Fingerabdruck der Liste ist nicht gesetzt, kann nicht arbeiten! (In `%{listdir}`.)"
-    list_key_missing: "Schlüssel der Liste nicht im Schlüsselring gefunden, kann nicht arbeiten! (In `%{listdir}`.)"
-    list_secret_key_missing: "Geheimer Schlüssel der Liste nicht im Schlüsselring gefunden, kann nicht arbeiten! (In `%{listdir}`.)"
-    list_admins_missing: "List hat keine Admins konfiguriert, kann nicht arbeiten! (In `%{listdir}`.)"
+    invalid_email: "ist keine gültige E-Mail-Adresse"
+    invalid_fingerprint: "ist kein gültiger OpenPGP-Fingerabdruck"
+    list_fingerprint_missing: "Fingerabdruck der Liste ist nicht gesetzt, kann nicht verarbeiten! (In `%{listdir}`.)"
+    list_key_missing: "Schlüssel der Liste nicht im Schlüsselring gefunden, kann nicht verarbeiten! (In `%{listdir}`.)"
+    list_secret_key_missing: "Geheimer Schlüssel der Liste nicht im Schlüsselring gefunden, kann nicht verarbeiten! (In `%{listdir}`.)"
+    list_admins_missing: "List hat keine Admins konfiguriert, kann nicht verarbeiten! (In `%{listdir}`.)"
     fatalerror: |
       Es ist ein schwerwiegender Fehler aufgetreten. Administratoren wurden benachrichtigt.
       Bitte versuche es später noch ein Mal.
@@ -37,7 +37,7 @@ de:
     message_signature_unknown: |
       Emails an diese Adresse müssen mit dem OpenPGP-Schlüssel signiert sein, der 
       deiner Mitgliedschaft zugewiesen ist. Wenn du nicht weisst, welcher Schlüssel das ist, frage
-      die Administrator/innen. Die erreichst du per Email an
+      die Administrator/innen. Diese erreichst du per Email an
       <%{owner_email}>.
       (Vorzugsweise verschlüssele die Email mit dem Schlüssel dieser Adresse:
       %{list_fingerprint}).
@@ -147,7 +147,7 @@ de:
         Mitgliedschaft von %{email} nicht gelöscht:
         %{errors}
       cannot_unsubscribe_last_admin: |
-        %{email} ist die einzige admin-Mitgliedschaft für diese Liste, daher darf sie nicht gelöscht werden.
+        %{email} ist die einzige Admin-Mitgliedschaft für diese Liste, daher darf sie nicht gelöscht werden.
       subscribed: |
         Mitgliedschaft von %{email} mit diesen Werten eingetragen:
 
@@ -159,7 +159,7 @@ de:
 
         %{errors}.
       list_of_subscriptions: "Mitgliedschaften:\n"
-      set_fingerprint_only_self: Nur admins dürfen den Fingerabdruck für andere Mitgliedschaften festlegen.
+      set_fingerprint_only_self: Nur Admins dürfen den Fingerabdruck für andere Mitgliedschaften festlegen.
       fingerprint_set: Fingerabdruck für %{email} auf %{fingerprint} gesetzt.
       setting_fingerprint_failed: |
         Fingerabdruck für %{email} konnte nicht auf %{fingerprint} gesetzt werden:
@@ -172,7 +172,7 @@ de:
         Benötigt werden ein oder zwei Werte, bspw.:
         X-SET-FINGERPRINT: 0xB3D190D5235C74E1907EACFE898F2C91E2E6E1F3
 
-        oder (als admin):
+        oder (als Admin):
         X-SET-FINGERPRINT: subscription2@hostname 0xB3D190D5235C74E1907EACFE898F2C91E2E6E1F3
 
         Wobei der Fingerprint in der gesamten Länge (40 Zeichen) angegeben werden muss. Optional mit 0x als Präfix.
@@ -184,11 +184,11 @@ de:
         Benötigt werden ein oder zwei Werte, bspw.:
         X-SET-FINGERPRINT: 0xB3D190D5235C74E1907EACFE898F2C91E2E6E1F3
 
-        oder (als admin):
+        oder (als Admin):
         X-SET-FINGERPRINT: subscription2@hostname 0xB3D190D5235C74E1907EACFE898F2C91E2E6E1F3
 
         Um einen Fingerprint zu entfernen kannst du das Schlüsselwort 'UNSET-FINGERPRINT' verwenden.
-      unset_fingerprint_only_self: Nur admins dürfen den Fingerabdruck für andere Mitgliedschaften löschen.
+      unset_fingerprint_only_self: Nur Admins dürfen den Fingerabdruck für andere Mitgliedschaften löschen.
       fingerprint_unset: Fingerabdruck für %{email} wurde entfernt.
       unsetting_fingerprint_failed: |
         Fingerabdruck für %{email} konnte nicht entfernt werden:
@@ -199,7 +199,7 @@ de:
         Benötigt werden ein Wert, bspw.:
         X-UNSET-FINGERPRINT: subscription2@hostname
 
-        Als admin musst du um deinen eigenen Fingerabdruck zu entfernen, noch zusätzlich das Argument force mitgeben. bspw.:
+        Als Admin musst du um deinen eigenen Fingerabdruck zu entfernen, noch zusätzlich das Argument force mitgeben. bspw.:
         X-UNSET-FINGERPRINT: adminsubscription2@hostname force
       subscribe_requires_arguments: |
         Fehler: Du hast zu dem Schlüsselwort 'SUBSCRIBE' keinen Wert angegeben.
@@ -210,13 +210,13 @@ de:
         Oder, um den Schlüssel der neuen Mitgliedschaft zuzuweisen:
         X-SUBSCRIBE: new-subscription@hostname 0xB3D190D5235C74E1907EACFE898F2C91E2E6E1F3
 
-        Oder, um den Schlüssel zuzuweisen, und die Mitgliedschaft als admin einzutragen:
+        Oder, um den Schlüssel zuzuweisen, und die Mitgliedschaft als Admin einzutragen:
         X-SUBSCRIBE: new-subscription@hostname 0xB3D190D5235C74E1907EACFE898F2C91E2E6E1F3 true
 
-        Oder, um den Schlüssel zuzuweisen, die Mitgliedschaft als admin einzutragen, und die Zustellung von Listen-Emails für diese Mitgliedschaft abzuschalten:
+        Oder, um den Schlüssel zuzuweisen, die Mitgliedschaft als Admin einzutragen, und die Zustellung von Listen-Emails für diese Mitgliedschaft abzuschalten:
         X-SUBSCRIBE: new-subscription@hostname 0xB3D190D5235C74E1907EACFE898F2C91E2E6E1F3 true false
 
-        Wenn du die optionalen Werte weglässt hat die Mitgliedschaft keinen Schlüssel zugewiesen, ist nicht admin, und hat die Zustellung von Listen-Email aktiviert.
+        Wenn du die optionalen Werte weglässt hat die Mitgliedschaft keinen Schlüssel zugewiesen, ist nicht Admin, und hat die Zustellung von Listen-Email aktiviert.
     sign_this:
       signatures_attached: Die Signaturen hängen an.
       no_content_found: Deine Email enthielt keine Anhänge und keinen Text-Inhalt, daher konnte nichts signiert werden.

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: schleuder
 version: !ruby/object:Gem::Version
-  version: 4.0.2
+  version: 4.0.3
 platform: ruby
 authors:
 - schleuder dev team
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-07-31 00:00:00.000000000 Z
+date: 2022-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -240,20 +240,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: irb
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
@@ -341,6 +327,8 @@ files:
 - db/migrate/20180723173900_add_deliver_selfsent_to_list.rb
 - db/migrate/20190906194820_add_autocrypt_header_to_list.rb
 - db/migrate/20200118170110_add_set_reply_to_to_sender_and_munge_from.rb
+- db/migrate/20211106112020_change_boolean_values_to_integers.rb
+- db/migrate/20211107151309_add_limits_to_string_columns.rb
 - db/schema.rb
 - etc/init.d/schleuder-api-daemon
 - etc/list-defaults.yml
@@ -460,8 +448,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.21
-signing_key:
+rubyforge_project: 
+rubygems_version: 2.7.6.2
+signing_key: 
 specification_version: 4
 summary: Schleuder is an encrypting mailing list manager with remailing-capabilities.
 test_files: []