Merge branch 'develop' into 0valt/1229/post-history-fixes

Author: trichoplax

Gemfile

@@ -41,6 +41,7 @@ gem 'will_paginate-bootstrap', '~> 1.0'
 
 # AWS for S3 (image storage) and SES (emails).
 gem 'aws-sdk-s3', '~> 1.61', require: false
+gem 'aws-sdk-sns', '~> 1.72'
 gem 'aws-ses-v4', require: 'aws/ses'
 
 # Task scheduler.

Gemfile.lock

@@ -66,16 +66,16 @@ GEM
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
-    addressable (2.8.1)
+    addressable (2.8.6)
       public_suffix (>= 2.0.2, < 6.0)
     ast (2.4.2)
     awesome_print (1.9.2)
-    aws-eventstream (1.2.0)
-    aws-partitions (1.628.0)
-    aws-sdk-core (3.145.0)
-      aws-eventstream (~> 1, >= 1.0.2)
-      aws-partitions (~> 1, >= 1.525.0)
-      aws-sigv4 (~> 1.1)
+    aws-eventstream (1.3.0)
+    aws-partitions (1.908.0)
+    aws-sdk-core (3.191.6)
+      aws-eventstream (~> 1, >= 1.3.0)
+      aws-partitions (~> 1, >= 1.651.0)
+      aws-sigv4 (~> 1.8)
       jmespath (~> 1, >= 1.6.1)
     aws-sdk-kms (1.58.0)
       aws-sdk-core (~> 3, >= 3.127.0)
@@ -84,12 +84,15 @@ GEM
       aws-sdk-core (~> 3, >= 3.127.0)
       aws-sdk-kms (~> 1)
       aws-sigv4 (~> 1.4)
+    aws-sdk-sns (1.72.0)
+      aws-sdk-core (~> 3, >= 3.191.0)
+      aws-sigv4 (~> 1.1)
     aws-ses-v4 (0.8.1)
       builder
       mail (> 2.2.5)
       mime-types
       xml-simple
-    aws-sigv4 (1.5.1)
+    aws-sigv4 (1.8.0)
       aws-eventstream (~> 1, >= 1.0.2)
     bcrypt (3.1.18)
     bindex (0.8.1)
@@ -126,7 +129,7 @@ GEM
       thor (>= 0.19.4, < 2.0)
       tins (~> 1.6)
     crass (1.0.6)
-    css_parser (1.11.0)
+    css_parser (1.16.0)
       addressable
     date (3.3.4)
     devise (4.8.1)
@@ -209,7 +212,7 @@ GEM
     net-smtp (0.4.0)
       net-protocol
     nio4r (2.7.0)
-    nokogiri (1.15.4-x86_64-linux)
+    nokogiri (1.16.2-x86_64-linux)
       racc (~> 1.4)
     omniauth (2.1.0)
       hashie (>= 3.4.6)
@@ -227,7 +230,7 @@ GEM
     premailer-rails (1.11.1)
       actionmailer (>= 3)
       premailer (~> 1.7, >= 1.7.9)
-    public_suffix (5.0.0)
+    public_suffix (5.0.4)
     puma (5.6.8)
       nio4r (~> 2.0)
     racc (1.7.3)
@@ -391,6 +394,7 @@ PLATFORMS
 DEPENDENCIES
   awesome_print (~> 1.9)
   aws-sdk-s3 (~> 1.61)
+  aws-sdk-sns (~> 1.72)
   aws-ses-v4
   byebug (~> 11.1)
   capybara (~> 3.38)
@@ -452,4 +456,4 @@ RUBY VERSION
    ruby 2.7.6p219
 
 BUNDLED WITH
-   2.4.15
+   2.4.13

INSTALLATION.md

@@ -118,11 +118,11 @@ You will need to set the Redis connection details there too. If you've followed
 the sample file should already contain the correct values for you, but if you've customised your
 setup you'll need to correct them.
 
+You'll also need to copy the Active Storage configuration from `config/storage.sample.yml` to `config/storage.yml`.
+
 If you are using MariaDB instead of MySQL, you will need to replace all occurrences of
 `utf8mb4_0900_ai_ci` with `utf8mb4_unicode_ci` in `db/schema.rb`.
 
-You'll also need to copy the Active Storage configuration from `config/storage.sample.yml` to `config/storage.yml`.
-
 Set up the database:
 
     rails db:create

app/assets/javascripts/posts.js

@@ -93,6 +93,13 @@ $(() => {
     tags: true
   });
 
+  /**
+   * Attempts to save a draft with a given body
+   * @param {string} postText draft text
+   * @param {JQuery<Element>} $field body input element
+   * @param {boolean} [manual] whether manual draft saving is enabled
+   * @returns {Promise<void>}
+   */
   const saveDraft = async (postText, $field, manual = false) => {
     const autosavePref = await QPixel.preference('autosave', true);
     if (autosavePref !== 'on' && !manual) {
@@ -111,10 +118,15 @@ $(() => {
         path: location.pathname
       })
     });
+
     if (resp.status === 200) {
-      const $el = $(`<span>&middot; <span class="has-color-green-600">draft saved</span></span>`);
-      $field.parents('.widget').find('.js-post-field-footer').append($el);
-      $el.fadeOut(1500, function () { $(this).remove() });
+      const $statusEl = $field.parents('.widget').find('.js-post-draft-status');
+
+      $statusEl.removeClass('transparent');
+
+      setTimeout(() => {
+        $statusEl.addClass('transparent');
+      }, 1500);
     }
   };
 

app/assets/stylesheets/posts.scss

@@ -89,7 +89,25 @@ h1 .badge.is-tag.is-master-tag {
   width: calc(100% + 2px);
 
   + .widget--footer {
-    margin-bottom: 0;
+    border-top: none;
+    align-items: center;
+    margin: 0;
+
+    &.mdhint {
+      display: flex;
+      flex-wrap: wrap;
+      gap: 1em 0;
+      justify-content: space-between;
+
+      & > * {
+        padding: 0;
+      }
+    }
+
+    & > .draft-status {
+      text-align: center;
+      transition: opacity 0.5s ease-in-out;
+    }
   }
 }
 

app/assets/stylesheets/utilities.scss

@@ -284,4 +284,8 @@ span.spoiler {
 
 .clearfix {
   overflow: hidden;
-}
+}
+
+.transparent {
+  opacity: 0;
+}

app/controllers/email_logs_controller.rb

@@ -0,0 +1,24 @@
+class EmailLogsController < ApplicationController
+  skip_forgery_protection only: [:log]
+  skip_before_action :set_globals, only: [:log]
+  skip_before_action :distinguish_fake_community, only: [:log]
+  skip_before_action :enforce_signed_in, only: [:log]
+
+  def log
+    message_type = request.headers['X-Amz-SNS-Message-Type']
+    if ['SubscriptionConfirmation', 'Notification'].include? message_type
+      verifier = Aws::SNS::MessageVerifier.new
+      body = request.body.read
+      if verifier.authentic? body
+        aws_data = JSON.parse body
+        message_data = JSON.parse aws_data['Message']
+        log_type = message_data['notificationType']
+        destination = message_data['mail']['destination'].join(', ')
+        EmailLog.create(log_type: log_type, destination: destination, data: aws_data['Message'])
+        render plain: 'OK'
+      else
+        render plain: "You're not AWS. Go away.", status: 401
+      end
+    end
+  end
+end

app/controllers/tags_controller.rb

@@ -32,13 +32,15 @@ def category
               @tag_set.tags.where(excerpt: '').or(@tag_set.tags.where(excerpt: nil))
                       .order(Arel.sql('COUNT(posts.id) DESC'))
             else
-              @tag_set.tags.order(Arel.sql('COUNT(posts.id) DESC'))
+              @tag_set&.tags&.order(Arel.sql('COUNT(posts.id) DESC'))
             end
-    @count = @tags.count
+    @count = @tags&.size || 0
     table = params[:hierarchical].present? ? 'tags_paths' : 'tags'
-    @tags = @tags.left_joins(:posts).group(Arel.sql("#{table}.id"))
-                 .select(Arel.sql("#{table}.*, COUNT(DISTINCT IF(posts.deleted = 0, posts.id, NULL)) AS post_count"))
-                 .paginate(per_page: 96, page: params[:page])
+    if @count.positive?
+      @tags = @tags.left_joins(:posts).group(Arel.sql("#{table}.id"))
+                   .select(Arel.sql("#{table}.*, COUNT(DISTINCT IF(posts.deleted = 0, posts.id, NULL)) AS post_count"))
+                   .paginate(per_page: 96, page: params[:page])
+    end
   end
 
   def show

app/controllers/users_controller.rb

@@ -174,7 +174,7 @@ def set_preference
   end
 
   def posts
-    @posts = if current_user&.privilege?('flag_curate')
+    @posts = if current_user&.privilege?('flag_curate') || @user == current_user
                Post.all
              else
                Post.undeleted
@@ -579,7 +579,7 @@ def vote_summary
   def avatar
     respond_to do |format|
       format.png do
-        size = params[:size]&.to_i&.positive? ? params[:size]&.to_i : 64
+        size = params[:size]&.to_i&.positive? ? [params[:size]&.to_i, 256].min : 64
         send_data helpers.user_auto_avatar(size, user: @user).to_blob, type: 'image/png', disposition: 'inline'
       end
     end

app/helpers/email_logs_helper.rb

@@ -0,0 +1,2 @@
+module EmailLogsHelper
+end