From 6eb60260b1b771e8cd42d3b58b82b2781a067991 Mon Sep 17 00:00:00 2001 From: ThibG Date: Fri, 22 Dec 2017 02:14:17 +0100 Subject: [PATCH 1/3] =?UTF-8?q?Display=20deleted=20users'=20role=20as=20?= =?UTF-8?q?=E2=80=9CSuspended=E2=80=9D=20(#6080)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Deleted users are technically suspended, but the code displaying their status in the admin interface was broken and displayed a javascript object holding translations of the possible user roles instead. --- app/views/admin/accounts/_account.html.haml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/app/views/admin/accounts/_account.html.haml b/app/views/admin/accounts/_account.html.haml index 598f6cddd0b..dfa7c564918 100644 --- a/app/views/admin/accounts/_account.html.haml +++ b/app/views/admin/accounts/_account.html.haml @@ -6,7 +6,10 @@ = link_to account.domain, admin_accounts_path(by_domain: account.domain) %td - if account.local? - = t("admin.accounts.roles.#{account.user&.role}") + - if account.user.nil? + = t("admin.accounts.moderation.suspended") + - else + = t("admin.accounts.roles.#{account.user.role}") - else = account.protocol.humanize %td From cea98e0c12759528d4a17f59c0e7616b7d426c8c Mon Sep 17 00:00:00 2001 From: ThibG Date: Fri, 22 Dec 2017 02:15:08 +0100 Subject: [PATCH 2/3] Reduce the number of synchronous resolves when posting toots (#6075) --- app/services/process_mentions_service.rb | 24 +++++++++++++++--------- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/app/services/process_mentions_service.rb b/app/services/process_mentions_service.rb index e12721c464c..46401f298a9 100644 --- a/app/services/process_mentions_service.rb +++ b/app/services/process_mentions_service.rb @@ -11,18 +11,20 @@ class ProcessMentionsService < BaseService return unless status.local? status.text = status.text.gsub(Account::MENTION_RE) do |match| - begin - mentioned_account = resolve_remote_account_service.call($1) - rescue Goldfinger::Error, HTTP::Error - mentioned_account = nil + username, domain = $1.split('@') + mentioned_account = Account.find_remote(username, domain) + + if mention_undeliverable?(status, mentioned_account) + begin + mentioned_account = resolve_remote_account_service.call($1) + rescue Goldfinger::Error, HTTP::Error + mentioned_account = nil + end end - if mentioned_account.nil? - username, domain = $1.split('@') - mentioned_account = Account.find_remote(username, domain) - end + mentioned_account ||= Account.find_remote(username, domain) - next match if mentioned_account.nil? || (!mentioned_account.local? && mentioned_account.ostatus? && status.stream_entry.hidden?) + next match if mention_undeliverable?(status, mentioned_account) mentioned_account.mentions.where(status: status).first_or_create(status: status) "@#{mentioned_account.acct}" @@ -37,6 +39,10 @@ class ProcessMentionsService < BaseService private + def mention_undeliverable?(status, mentioned_account) + mentioned_account.nil? || (!mentioned_account.local? && mentioned_account.ostatus? && status.stream_entry.hidden?) + end + def create_notification(status, mention) mentioned_account = mention.account From 9592b5e31e19ce470252ade7b8df73625e2592e6 Mon Sep 17 00:00:00 2001 From: nightpool Date: Thu, 21 Dec 2017 19:17:59 -0600 Subject: [PATCH 3/3] enforce LOCAL_HTTPS=true in production (#6061) * enforce https in production * note changes in production env sample * typo fix --- .env.production.sample | 5 +++-- config/initializers/ostatus.rb | 3 ++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/.env.production.sample b/.env.production.sample index 91fcce6ac45..3f0edd72ff6 100644 --- a/.env.production.sample +++ b/.env.production.sample @@ -11,10 +11,11 @@ DB_PASS= DB_PORT=5432 # Federation -# Note: Changing LOCAL_DOMAIN or LOCAL_HTTPS at a later time will cause unwanted side effects. +# Note: Changing LOCAL_DOMAIN at a later time will cause unwanted side effects, including breaking all existing federation. # LOCAL_DOMAIN should *NOT* contain the protocol part of the domain e.g https://example.com. LOCAL_DOMAIN=example.com -LOCAL_HTTPS=true + +# Changing LOCAL_HTTPS in production is no longer supported. (Mastodon will always serve https:// links) # Use this only if you need to run mastodon on a different domain than the one used for federation. # You can read more about this option on https://github.com/tootsuite/documentation/blob/master/Running-Mastodon/Serving_a_different_domain.md diff --git a/config/initializers/ostatus.rb b/config/initializers/ostatus.rb index bb8591f7475..5773b729039 100644 --- a/config/initializers/ostatus.rb +++ b/config/initializers/ostatus.rb @@ -3,11 +3,12 @@ port = ENV.fetch('PORT') { 3000 } host = ENV.fetch('LOCAL_DOMAIN') { "localhost:#{port}" } web_host = ENV.fetch('WEB_DOMAIN') { host } -https = ENV['LOCAL_HTTPS'] == 'true' alternate_domains = ENV.fetch('ALTERNATE_DOMAINS') { '' } Rails.application.configure do + https = Rails.env.production? || ENV['LOCAL_HTTPS'] == 'true' + config.x.local_domain = host config.x.web_domain = web_host config.x.use_https = https