From abecaba317b3c28040efba2d3b84397ffbe85603 Mon Sep 17 00:00:00 2001 From: ThibG Date: Sat, 30 Mar 2019 18:15:23 +0100 Subject: Fix failing keybase-related test (#10428) --- spec/services/activitypub/process_account_service_spec.rb | 2 ++ 1 file changed, 2 insertions(+) diff --git a/spec/services/activitypub/process_account_service_spec.rb b/spec/services/activitypub/process_account_service_spec.rb index ab8fb5cc3..fee502949 100644 --- a/spec/services/activitypub/process_account_service_spec.rb +++ b/spec/services/activitypub/process_account_service_spec.rb @@ -42,6 +42,8 @@ RSpec.describe ActivityPub::ProcessAccountService, type: :service do end it 'parses out of attachment' do + allow(ProofProvider::Keybase::Worker).to receive(:perform_async) + account = subject.call('alice', 'example.com', payload) expect(account.identity_proofs.count).to eq 1 -- cgit From 59cce07601cbac40c8e6f5af593ea957a57d209f Mon Sep 17 00:00:00 2001 From: slice Date: Sat, 30 Mar 2019 20:03:56 -0700 Subject: Add thumbnail, hero, and mascot back to Form::AdminSettings::KEYS (#10430) These will allow these images to be uploaded again on the admin settings page. --- app/models/form/admin_settings.rb | 3 +++ 1 file changed, 3 insertions(+) diff --git a/app/models/form/admin_settings.rb b/app/models/form/admin_settings.rb index 2d3aa726d..85a6e6dff 100644 --- a/app/models/form/admin_settings.rb +++ b/app/models/form/admin_settings.rb @@ -25,6 +25,9 @@ class Form::AdminSettings preview_sensitive_media custom_css profile_directory + thumbnail + hero + mascot ).freeze BOOLEAN_KEYS = %i( -- cgit From 68c797bece0ed0801cfc8718d77841b455bf4d0d Mon Sep 17 00:00:00 2001 From: ThibG Date: Sun, 31 Mar 2019 17:27:24 +0200 Subject: Ensure request.body isn't emptied out before signature verification (#10432) Fixes #10429 --- app/controllers/activitypub/inboxes_controller.rb | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/app/controllers/activitypub/inboxes_controller.rb b/app/controllers/activitypub/inboxes_controller.rb index 1501b914e..a0b7532c2 100644 --- a/app/controllers/activitypub/inboxes_controller.rb +++ b/app/controllers/activitypub/inboxes_controller.rb @@ -32,7 +32,10 @@ class ActivityPub::InboxesController < Api::BaseController end def body - @body ||= request.body.read.force_encoding('UTF-8') + return @body if defined?(@body) + @body = request.body.read.force_encoding('UTF-8') + request.body.rewind if request.body.respond_to?(:rewind) + @body end def upgrade_account -- cgit From abdf225353dfb9791933b5d2dbcd038dcdb656df Mon Sep 17 00:00:00 2001 From: mayaeh Date: Mon, 1 Apr 2019 12:50:05 +0900 Subject: i18n: Update Japanese translations (#10427) * Update Japanese translations. run yarn manage:translations && i18n-tasks add-missing * Update Japanese translations. * Fix mistake * Change to be concise. --- .../mastodon/locales/defaultMessages.json | 12 +++ app/javascript/mastodon/locales/en.json | 1 + app/javascript/mastodon/locales/ja.json | 41 +++++----- config/locales/activerecord.ja.yml | 7 +- config/locales/en.yml | 1 + config/locales/ja.yml | 91 ++++++++++++++++------ 6 files changed, 106 insertions(+), 47 deletions(-) diff --git a/app/javascript/mastodon/locales/defaultMessages.json b/app/javascript/mastodon/locales/defaultMessages.json index cf0da6e47..5d1367020 100644 --- a/app/javascript/mastodon/locales/defaultMessages.json +++ b/app/javascript/mastodon/locales/defaultMessages.json @@ -551,6 +551,10 @@ }, { "descriptors": [ + { + "defaultMessage": "You are blocked", + "id": "empty_column.account_timeline_blocked" + }, { "defaultMessage": "No toots here!", "id": "empty_column.account_timeline" @@ -1251,6 +1255,10 @@ }, { "descriptors": [ + { + "defaultMessage": "You are blocked", + "id": "empty_column.account_timeline_blocked" + }, { "defaultMessage": "No one follows this user yet.", "id": "account.followers.empty" @@ -1260,6 +1268,10 @@ }, { "descriptors": [ + { + "defaultMessage": "You are blocked", + "id": "empty_column.account_timeline_blocked" + }, { "defaultMessage": "This user doesn't follow anyone yet.", "id": "account.follows.empty" diff --git a/app/javascript/mastodon/locales/en.json b/app/javascript/mastodon/locales/en.json index bf0c1bfab..45548eb55 100644 --- a/app/javascript/mastodon/locales/en.json +++ b/app/javascript/mastodon/locales/en.json @@ -117,6 +117,7 @@ "emoji_button.symbols": "Symbols", "emoji_button.travel": "Travel & Places", "empty_column.account_timeline": "No toots here!", + "empty_column.account_timeline_blocked": "You are blocked", "empty_column.blocks": "You haven't blocked any users yet.", "empty_column.community": "The local timeline is empty. Write something publicly to get the ball rolling!", "empty_column.direct": "You don't have any direct messages yet. When you send or receive one, it will show up here.", diff --git a/app/javascript/mastodon/locales/ja.json b/app/javascript/mastodon/locales/ja.json index 766ae0ec5..9f00332fc 100644 --- a/app/javascript/mastodon/locales/ja.json +++ b/app/javascript/mastodon/locales/ja.json @@ -1,12 +1,12 @@ { - "account.add_or_remove_from_list": "リストに追加または外す", + "account.add_or_remove_from_list": "リストから追加または外す", "account.badges.bot": "Bot", "account.block": "@{name}さんをブロック", "account.block_domain": "{domain}全体を非表示", "account.blocked": "ブロック済み", "account.direct": "@{name}さんにダイレクトメッセージ", "account.domain_blocked": "ドメイン非表示中", - "account.edit_profile": "プロフィールを編集", + "account.edit_profile": "プロフィール編集", "account.endorse": "プロフィールで紹介する", "account.follow": "フォロー", "account.followers": "フォロワー", @@ -16,7 +16,7 @@ "account.follows_you": "フォローされています", "account.hide_reblogs": "@{name}さんからのブーストを非表示", "account.link_verified_on": "このリンクの所有権は{date}に確認されました", - "account.locked_info": "このアカウントは承認制アカウントです。相手が確認するまでフォローは完了しません。", + "account.locked_info": "このアカウントは承認制アカウントです。相手が承認するまでフォローは完了しません。", "account.media": "メディア", "account.mention": "@{name}さんにトゥート", "account.moved_to": "{name}さんは引っ越しました:", @@ -30,7 +30,7 @@ "account.share": "@{name}さんのプロフィールを共有する", "account.show_reblogs": "@{name}さんからのブーストを表示", "account.unblock": "@{name}さんのブロックを解除", - "account.unblock_domain": "{domain}を表示", + "account.unblock_domain": "{domain}の非表示を解除", "account.unendorse": "プロフィールから外す", "account.unfollow": "フォロー解除", "account.unmute": "@{name}さんのミュートを解除", @@ -67,14 +67,14 @@ "community.column_settings.media_only": "メディアのみ表示", "compose_form.direct_message_warning": "このトゥートはメンションされた人にのみ送信されます。", "compose_form.direct_message_warning_learn_more": "もっと詳しく", - "compose_form.hashtag_warning": "このトゥートは未収載なのでハッシュタグの一覧に表示されません。公開トゥートだけがハッシュタグで検索できます。", + "compose_form.hashtag_warning": "このトゥートは公開設定ではないのでハッシュタグの一覧に表示されません。公開トゥートだけがハッシュタグで検索できます。", "compose_form.lock_disclaimer": "あなたのアカウントは{locked}になっていません。誰でもあなたをフォローすることができ、フォロワー限定の投稿を見ることができます。", "compose_form.lock_disclaimer.lock": "承認制", "compose_form.placeholder": "今なにしてる？", - "compose_form.poll.add_option": "Add a choice", - "compose_form.poll.duration": "投票期間", - "compose_form.poll.option_placeholder": "Choice {number}", - "compose_form.poll.remove_option": "Remove this choice", + "compose_form.poll.add_option": "追加", + "compose_form.poll.duration": "アンケート期間", + "compose_form.poll.option_placeholder": "選択肢 {number}", + "compose_form.poll.remove_option": "この選択肢を削除", "compose_form.publish": "トゥート", "compose_form.publish_loud": "{publish}！", "compose_form.sensitive.marked": "メディアに閲覧注意が設定されています", @@ -83,7 +83,7 @@ "compose_form.spoiler.unmarked": "閲覧注意が設定されていません", "compose_form.spoiler_placeholder": "ここに警告を書いてください", "confirmation_modal.cancel": "キャンセル", - "confirmations.block.block_and_report": "Block & Report", + "confirmations.block.block_and_report": "ブロックし通報", "confirmations.block.confirm": "ブロック", "confirmations.block.message": "本当に{name}さんをブロックしますか？", "confirmations.delete.confirm": "削除", @@ -117,6 +117,7 @@ "emoji_button.symbols": "記号", "emoji_button.travel": "旅行と場所", "empty_column.account_timeline": "トゥートがありません！", + "empty_column.account_timeline_blocked": "ブロックされています", "empty_column.blocks": "まだ誰もブロックしていません。", "empty_column.community": "ローカルタイムラインはまだ使われていません。何か書いてみましょう！", "empty_column.direct": "ダイレクトメッセージはまだありません。ダイレクトメッセージをやりとりすると、ここに表示されます。", @@ -154,8 +155,8 @@ "home.column_settings.basic": "基本設定", "home.column_settings.show_reblogs": "ブースト表示", "home.column_settings.show_replies": "返信表示", - "intervals.full.days": "{number, plural, one {# day} other {# days}}", - "intervals.full.hours": "{number, plural, one {# hour} other {# hours}}", + "intervals.full.days": "{number}日", + "intervals.full.hours": "{number}時間", "intervals.full.minutes": "{number}分", "introduction.federation.action": "次へ", "introduction.federation.federated.headline": "連合タイムライン", @@ -173,7 +174,7 @@ "introduction.interactions.reply.text": "自身や人々のトゥートに返信することで、一連の会話に繋げることができます。", "introduction.welcome.action": "はじめる！", "introduction.welcome.headline": "はじめに", - "introduction.welcome.text": "Fediverseの世界へようこそ！あと少しでメッセージを配信したり、さまざまなサーバーを越えた友達と話せるようになります。ところでここ{domain}は特別なサーバーです…あなたのプロフィールを持つ主体のサーバーですので、名前を覚えておきましょう。", + "introduction.welcome.text": "Fediverseの世界へようこそ！あと少しでメッセージを配信したり、さまざまなサーバーを越えた友達と話せるようになります。ところで、ここ{domain}は特別なサーバーです…あなたのプロフィールを持つ主体のサーバーですので、名前を覚えておきましょう。", "keyboard_shortcuts.back": "戻る", "keyboard_shortcuts.blocked": "ブロックしたユーザーのリストを開く", "keyboard_shortcuts.boost": "ブースト", @@ -246,7 +247,7 @@ "notification.favourite": "{name}さんがあなたのトゥートをお気に入りに登録しました", "notification.follow": "{name}さんにフォローされました", "notification.mention": "{name}さんがあなたに返信しました", - "notification.poll": "A poll you have voted in has ended", + "notification.poll": "アンケートが終了しました", "notification.reblog": "{name}さんがあなたのトゥートをブーストしました", "notifications.clear": "通知を消去", "notifications.clear_confirmation": "本当に通知を消去しますか？", @@ -257,7 +258,7 @@ "notifications.column_settings.filter_bar.show": "表示", "notifications.column_settings.follow": "新しいフォロワー:", "notifications.column_settings.mention": "返信:", - "notifications.column_settings.poll": "Poll results:", + "notifications.column_settings.poll": "アンケート結果:", "notifications.column_settings.push": "プッシュ通知", "notifications.column_settings.reblog": "ブースト:", "notifications.column_settings.show": "カラムに表示", @@ -267,14 +268,14 @@ "notifications.filter.favourites": "お気に入り", "notifications.filter.follows": "フォロー", "notifications.filter.mentions": "返信", - "notifications.filter.polls": "Poll results", + "notifications.filter.polls": "アンケート結果", "notifications.group": "{count} 件の通知", "poll.closed": "終了", "poll.refresh": "更新", "poll.total_votes": "{count}票", - "poll.vote": "Vote", - "poll_button.add_poll": "Add a poll", - "poll_button.remove_poll": "投票を削除", + "poll.vote": "投票", + "poll_button.add_poll": "アンケートを追加", + "poll_button.remove_poll": "アンケートを削除", "privacy.change": "公開範囲を変更", "privacy.direct.long": "メンションしたユーザーだけに公開", "privacy.direct.short": "ダイレクト", @@ -366,7 +367,7 @@ "upload_area.title": "ドラッグ＆ドロップでアップロード", "upload_button.label": "メディアを追加 (JPEG, PNG, GIF, WebM, MP4, MOV)", "upload_error.limit": "アップロードできる上限を超えています。", - "upload_error.poll": "File upload not allowed with polls.", + "upload_error.poll": "アンケートではファイルをアップロードできません。", "upload_form.description": "視覚障害者のための説明", "upload_form.focus": "プレビューを変更", "upload_form.undo": "削除", diff --git a/config/locales/activerecord.ja.yml b/config/locales/activerecord.ja.yml index 7bc4fc308..c3b4b7484 100644 --- a/config/locales/activerecord.ja.yml +++ b/config/locales/activerecord.ja.yml @@ -2,6 +2,9 @@ ja: activerecord: attributes: + poll: + expires_at: 期限 + options: 選択肢 user: email: メールアドレス errors: @@ -9,8 +12,8 @@ ja: account: attributes: username: - invalid: アルファベット・数値・アンダーバー(_)で入力してください + invalid: アルファベット・数字・アンダーバーの組み合わせで入力してください status: attributes: reblog: - taken: のブーストはすでに存在します + taken: は既にブーストされています diff --git a/config/locales/en.yml b/config/locales/en.yml index 64467be39..705024ff5 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -633,6 +633,7 @@ en: all: All changes_saved_msg: Changes successfully saved! copy: Copy + order_by: Order by save_changes: Save changes validation_errors: one: Something isn't quite right yet! Please review the error below diff --git a/config/locales/ja.yml b/config/locales/ja.yml index 6eab21a49..09214a937 100644 --- a/config/locales/ja.yml +++ b/config/locales/ja.yml @@ -163,8 +163,8 @@ ja: search: 検索 shared_inbox_url: Shared inbox URL show: - created_reports: このアカウントで作られたレポート - targeted_reports: このアカウントについてのレポート + created_reports: このアカウントで作られた通報 + targeted_reports: このアカウントについての通報 silence: サイレンス silenced: サイレンス済み statuses: トゥート数 @@ -180,7 +180,7 @@ ja: web: Web action_logs: actions: - assigned_to_self_report: "%{name} さんがレポート %{target} を自身の担当に割り当てました" + assigned_to_self_report: "%{name} さんが通報 %{target} を自身の担当に割り当てました" change_email_user: "%{name} さんが %{target} さんのメールアドレスを変更しました" confirm_user: "%{name} さんが %{target} さんのメールアドレスを確認済みにしました" create_account_warning: "%{name} さんが %{target} さんに警告メールを送信しました" @@ -200,12 +200,12 @@ ja: memorialize_account: "%{name} さんが %{target} さんを追悼アカウントページに登録しました" promote_user: "%{name} さんが %{target} さんを昇格しました" remove_avatar_user: "%{name} さんが %{target} さんのアイコンを削除しました" - reopen_report: "%{name} さんがレポート %{target} を再び開きました" + reopen_report: "%{name} さんが通報 %{target} を再び開きました" reset_password_user: "%{name} さんが %{target} さんのパスワードをリセットしました" - resolve_report: "%{name} さんがレポート %{target} を解決済みにしました" + resolve_report: "%{name} さんが通報 %{target} を解決済みにしました" silence_account: "%{name} さんが %{target} さんをサイレンスにしました" suspend_account: "%{name} さんが %{target} さんを停止しました" - unassigned_report: "%{name} さんがレポート %{target} の担当を外しました" + unassigned_report: "%{name} さんが通報 %{target} の担当を外しました" unsilence_account: "%{name} さんが %{target} さんのサイレンスを解除しました" unsuspend_account: "%{name} さんが %{target} さんの停止を解除しました" update_custom_emoji: "%{name} さんがカスタム絵文字 %{target} を更新しました" @@ -245,9 +245,10 @@ ja: feature_profile_directory: ディレクトリ feature_registrations: 新規登録 feature_relay: 連合リレー + feature_timeline_preview: タイムラインプレビュー features: 機能 hidden_service: 秘匿サービスとの連合 - open_reports: 未解決のレポート + open_reports: 未解決の通報 recent_users: 最近登録したユーザー search: 全文検索 single_user_mode: シングルユーザーモード @@ -275,10 +276,10 @@ ja: title: 新規ドメインブロック reject_media: メディアファイルを拒否 reject_media_hint: ローカルに保存されたメディアファイルを削除し、今後のダウンロードを拒否します。停止とは無関係です - reject_reports: レポートを拒否 - reject_reports_hint: このドメインからのレポートをすべて無視します。停止とは無関係です + reject_reports: 通報を拒否 + reject_reports_hint: このドメインからの通報をすべて無視します。停止とは無関係です rejecting_media: メディアファイルを拒否中 - rejecting_reports: レポートを拒否中 + rejecting_reports: 通報を拒否中 severity: silence: サイレンス中 suspend: 停止中 @@ -319,7 +320,7 @@ ja: total_blocked_by_us: ブロック合計 total_followed_by_them: 被フォロー合計 total_followed_by_us: フォロー合計 - total_reported: レポート合計 + total_reported: 通報合計 total_storage: 添付されたメディア invites: deactivate_all: すべて無効化 @@ -345,19 +346,19 @@ ja: status: ステータス title: リレー report_notes: - created_msg: レポートメモを書き込みました！ - destroyed_msg: レポートメモを削除しました！ + created_msg: 通報メモを書き込みました！ + destroyed_msg: 通報メモを削除しました！ reports: account: note: メモ - report: レポート - action_taken_by: レポート処理者 + report: 通報 + action_taken_by: 通報処理者 are_you_sure: 本当に実行しますか？ assign_to_self: 担当になる assigned: 担当者 comment: none: なし - created_at: レポート日時 + created_at: 通報日時 mark_as_resolved: 解決済みとしてマーク mark_as_unresolved: 未解決として再び開く notes: @@ -367,13 +368,13 @@ ja: delete: 削除 placeholder: どのような措置が取られたか、または関連する更新を記述してください… reopen: 再び開く - report: レポート#%{id} + report: 通報#%{id} reported_account: 報告対象アカウント reported_by: 報告者 resolved: 解決済み - resolved_msg: レポートを解決済みにしました！ + resolved_msg: 通報を解決済みにしました！ status: ステータス - title: レポート + title: 通報 unassign: 担当を外す unresolved: 未解決 updated_at: 更新日時 @@ -386,7 +387,7 @@ ja: title: 新規ユーザーが自動フォローするアカウント contact_information: email: ビジネスメールアドレス - username: 連絡先のユーザー名 + username: 連絡先ユーザー名 custom_css: desc_html: 全ページに適用されるCSSの編集 title: カスタムCSS @@ -586,6 +587,9 @@ ja: content: もうしわけありませんが、なにかが間違っています。 title: このページは正しくありません noscript_html: Mastodonのウェブアプリケーションを利用する場合はJavaScriptを有効にしてください。またはあなたのプラットフォーム向けのMastodonネイティブアプリを探すことができます。 + existing_username_validator: + not_found: そのようなユーザー名はローカルに見つかりませんでした + not_found_multiple: "%{usernames} は見つかりませんでした" exports: archive_takeout: date: 日時 @@ -629,10 +633,31 @@ ja: all: すべて changes_saved_msg: 正常に変更されました！ copy: コピー + order_by: 並び順 save_changes: 変更を保存 validation_errors: one: エラーが発生しました！ 以下のエラーを確認してください other: エラーが発生しました！ 以下の%{count}個のエラーを確認してください + html_validator: + invalid_markup: '無効なHTMLマークアップが含まれています: %{error}' + identity_proofs: + active: アクティブ + authorize: 許可する + authorize_connection_prompt: この暗号化接続を許可しますか？ + errors: + failed: 暗号化接続に失敗しました。%{provider}からもう一度やり直してください。 + keybase: + invalid_token: Keybaseトークンは16進数で66文字のハッシュである必要があります + verification_failed: KeybaseはこのトークンをKeybaseユーザー%{kb_username}の署名として認識しませんでした。Keybaseから再試行してください。 + wrong_user: "%{current}としてログインしている間%{proving}の証明を作成することはできません。%{proving}としてログインし、もう一度やり直してください。" + explanation_html: ここではKeybaseのような他のサービスのアカウントと暗号化し関連づけることができます。これにより他の人が暗号化されたメッセージを送信したり、その内容を信用できるようになります。 + i_am_html: I am %{username} on %{service}. + identity: Identity + inactive: 非アクティブ + publicize_checkbox: 'そしてこれをトゥートしてください:' + publicize_toot: 'It is proven! I am %{username} on %{service}: %{url}' + status: 認証状態 + view_proof: 証明を表示 imports: modes: merge: 統合 @@ -736,10 +761,11 @@ ja: truncate: "…" polls: errors: + already_voted: このアンケートには投票済みです duplicate_options: に同じものがあります duration_too_long: が長過ぎます duration_too_short: が短過ぎます - expired: 既に終了している投票です + expired: アンケートは既に終了しました over_character_limit: は%{max}文字より長くすることはできません too_few_options: は複数必要です too_many_options: は%{max}個までです @@ -748,6 +774,19 @@ ja: other: その他 publishing: 投稿 web: ウェブ + relationships: + activity: 活動 + dormant: 非アクティブ + last_active: 最後の活動 + most_recent: 新着 + moved: 引っ越し済み + mutual: 相互 + primary: 標準 + relationship: 関係性 + remove_selected_domains: 選択したドメインのフォロワーを全て解除 + remove_selected_followers: 選択したフォロワーを解除 + remove_selected_follows: 選択したユーザーをフォロー解除 + status: 状態 remote_follow: acct: あなたの ユーザー名@ドメイン を入力してください missing_resource: リダイレクト先が見つかりませんでした @@ -822,10 +861,12 @@ ja: edit_profile: プロフィールを編集 export: データのエクスポート featured_tags: 注目のハッシュタグ + identity_proofs: Identity proofs import: データのインポート migrate: アカウントの引っ越し notifications: 通知 preferences: ユーザー設定 + relationships: フォロー・フォロワー settings: 設定 two_factor_authentication: 二段階認証 your_apps: アプリ @@ -850,12 +891,12 @@ ja: limit: 固定できるトゥート数の上限に達しました ownership: 他人のトゥートを固定することはできません private: 非公開のトゥートを固定することはできません - reblog: ブーストされたトゥートを固定することはできません + reblog: ブーストを固定することはできません poll: total_votes: - one: "%{count} vote" - other: "%{count} votes" - vote: Vote + one: "%{count}票" + other: "%{count}票" + vote: 投票 show_more: もっと見る sign_in_to_participate: ログインして会話に参加 title: '%{name}: "%{quote}"' -- cgit From 85973f4f37c8b94422f267654018d0d7f88ba32b Mon Sep 17 00:00:00 2001 From: slice Date: Sun, 31 Mar 2019 22:30:46 -0700 Subject: Improvements to image upload validation and creation (#10431) * Check if image value is nil? before creating an image Check if uploaded images aren't nil before creating SiteUpload models for them. * Validate presence of file in SiteUpload * Fix file presence validation * Fabricate SiteUpload#file * Add link to Creative Commons license --- app/models/form/admin_settings.rb | 2 +- app/models/site_upload.rb | 1 + spec/fabricators/assets/TEAPOT | 6 ++++++ spec/fabricators/assets/utah_teapot.png | Bin 0 -> 248232 bytes spec/fabricators/site_upload_fabricator.rb | 1 + 5 files changed, 9 insertions(+), 1 deletion(-) create mode 100644 spec/fabricators/assets/TEAPOT create mode 100644 spec/fabricators/assets/utah_teapot.png diff --git a/app/models/form/admin_settings.rb b/app/models/form/admin_settings.rb index 85a6e6dff..d3af8c30f 100644 --- a/app/models/form/admin_settings.rb +++ b/app/models/form/admin_settings.rb @@ -67,7 +67,7 @@ class Form::AdminSettings KEYS.each do |key| value = instance_variable_get("@#{key}") - if UPLOAD_KEYS.include?(key) + if UPLOAD_KEYS.include?(key) && !value.nil? upload = SiteUpload.where(var: key).first_or_initialize(var: key) upload.update(file: value) else diff --git a/app/models/site_upload.rb b/app/models/site_upload.rb index 14d683767..cf10b30fc 100644 --- a/app/models/site_upload.rb +++ b/app/models/site_upload.rb @@ -18,6 +18,7 @@ class SiteUpload < ApplicationRecord has_attached_file :file validates_attachment_content_type :file, content_type: /\Aimage\/.*\z/ + validates :file, presence: true validates :var, presence: true, uniqueness: true before_save :set_meta diff --git a/spec/fabricators/assets/TEAPOT b/spec/fabricators/assets/TEAPOT new file mode 100644 index 000000000..e624ecb4c --- /dev/null +++ b/spec/fabricators/assets/TEAPOT @@ -0,0 +1,6 @@ +This "Utah teapot" photograph is licensed under the Creative Commons +Attribution-Share Alike 3.0 Unported license: + https://creativecommons.org/licenses/by-sa/3.0/deed.en + +Original source of work: + https://commons.wikimedia.org/wiki/File:Utah_teapot_simple_2.png diff --git a/spec/fabricators/assets/utah_teapot.png b/spec/fabricators/assets/utah_teapot.png new file mode 100644 index 000000000..6708361e5 Binary files /dev/null and b/spec/fabricators/assets/utah_teapot.png differ diff --git a/spec/fabricators/site_upload_fabricator.rb b/spec/fabricators/site_upload_fabricator.rb index 4a171486f..2efc57e28 100644 --- a/spec/fabricators/site_upload_fabricator.rb +++ b/spec/fabricators/site_upload_fabricator.rb @@ -1,2 +1,3 @@ Fabricator(:site_upload) do + file { File.open(File.join(Rails.root, 'spec', 'fabricators', 'assets', 'utah_teapot.png')) } end -- cgit From 2acd8940de596ab119e35bd204fe55e06b914403 Mon Sep 17 00:00:00 2001 From: ThibG Date: Mon, 1 Apr 2019 19:10:44 +0200 Subject: Fix more keybase-related test failures (#10443) --- spec/services/activitypub/process_account_service_spec.rb | 2 ++ 1 file changed, 2 insertions(+) diff --git a/spec/services/activitypub/process_account_service_spec.rb b/spec/services/activitypub/process_account_service_spec.rb index fee502949..5141e3f16 100644 --- a/spec/services/activitypub/process_account_service_spec.rb +++ b/spec/services/activitypub/process_account_service_spec.rb @@ -56,6 +56,8 @@ RSpec.describe ActivityPub::ProcessAccountService, type: :service do end it 'removes no longer present proofs' do + allow(ProofProvider::Keybase::Worker).to receive(:perform_async) + account = Fabricate(:account, username: 'alice', domain: 'example.com') old_proof = Fabricate(:account_identity_proof, account: account, provider: 'keybase', provider_username: 'Bob', token: 'b' * 66) -- cgit From 67eb47e25f1e7704dd9ecd32695ff3b53c807375 Mon Sep 17 00:00:00 2001 From: Jeong Arm Date: Tue, 2 Apr 2019 02:17:21 +0900 Subject: Simplify culling and fix race condition (#10440) --- lib/mastodon/accounts_cli.rb | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/lib/mastodon/accounts_cli.rb b/lib/mastodon/accounts_cli.rb index dca31cdca..f02b2149b 100644 --- a/lib/mastodon/accounts_cli.rb +++ b/lib/mastodon/accounts_cli.rb @@ -236,11 +236,7 @@ module Mastodon end if [404, 410].include?(code) - unless options[:dry_run] - SuspendAccountService.new.call(account) - account.destroy - end - + SuspendAccountService.new.call(account, destroy: true) unless options[:dry_run] culled += 1 say('+', :green, false) else -- cgit From 62bafa20a112ccdddaedb25723fc819dbbcd8e9a Mon Sep 17 00:00:00 2001 From: ThibG Date: Mon, 1 Apr 2019 20:06:13 +0200 Subject: Hide blocking accounts from blocked users (#10442) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Revert "Add indication that you have been blocked in web UI (#10420)" This reverts commit bd02ec6daa974dcd3231e73826a56e08dbeedadc. * Revert "Add `blocked_by` relationship to the REST API (#10373)" This reverts commit 9745de883b198375ba23f7fde879f6d75ce2df0f. * Hide blocking accounts from search results * Filter blocking accouts from account followers * Filter blocking accouts from account's following accounts * Filter blocking accounts from “reblogged by” and “favourited by” lists * Remove blocking account from URL search * Return 410 on trying to fetch user data from a user who blocked us * Return 410 in /api/v1/account/statuses for suspended or blocking accounts * Fix status filtering when performing URL search * Restore some React improvements Restore some cleanup from bd02ec6daa974dcd3231e73826a56e08dbeedadc * Refactor by adding `without_blocking` scope --- .../v1/accounts/follower_accounts_controller.rb | 2 +- .../v1/accounts/following_accounts_controller.rb | 2 +- .../api/v1/accounts/statuses_controller.rb | 10 ++++++++ app/controllers/api/v1/accounts_controller.rb | 5 ++++ .../statuses/favourited_by_accounts_controller.rb | 1 + .../statuses/reblogged_by_accounts_controller.rb | 2 +- .../mastodon/features/account/components/header.js | 2 +- .../mastodon/features/account_timeline/index.js | 22 +++++----------- .../mastodon/features/followers/index.js | 8 +++--- .../mastodon/features/following/index.js | 8 +++--- app/javascript/styles/mastodon/stream_entries.scss | 6 ++--- app/models/account.rb | 1 + app/presenters/account_relationships_presenter.rb | 6 +---- app/serializers/rest/relationship_serializer.rb | 6 +---- app/services/account_search_service.rb | 10 +++++++- app/services/search_service.rb | 2 ++ .../accounts/follower_accounts_controller_spec.rb | 29 +++++++++++++++++++-- .../accounts/following_accounts_controller_spec.rb | 29 +++++++++++++++++++-- spec/services/account_search_service_spec.rb | 17 ++++++++++++ spec/services/search_service_spec.rb | 30 ++++++++++++---------- 20 files changed, 136 insertions(+), 62 deletions(-) diff --git a/app/controllers/api/v1/accounts/follower_accounts_controller.rb b/app/controllers/api/v1/accounts/follower_accounts_controller.rb index ec15debb0..7a45e6dd2 100644 --- a/app/controllers/api/v1/accounts/follower_accounts_controller.rb +++ b/app/controllers/api/v1/accounts/follower_accounts_controller.rb @@ -25,7 +25,7 @@ class Api::V1::Accounts::FollowerAccountsController < Api::BaseController end def default_accounts - Account.includes(:active_relationships, :account_stat).references(:active_relationships) + Account.without_blocking(current_account).includes(:active_relationships, :account_stat).references(:active_relationships) end def paginated_follows diff --git a/app/controllers/api/v1/accounts/following_accounts_controller.rb b/app/controllers/api/v1/accounts/following_accounts_controller.rb index f3e112f2c..0369cb25e 100644 --- a/app/controllers/api/v1/accounts/following_accounts_controller.rb +++ b/app/controllers/api/v1/accounts/following_accounts_controller.rb @@ -25,7 +25,7 @@ class Api::V1::Accounts::FollowingAccountsController < Api::BaseController end def default_accounts - Account.includes(:passive_relationships, :account_stat).references(:passive_relationships) + Account.without_blocking(current_account).includes(:passive_relationships, :account_stat).references(:passive_relationships) end def paginated_follows diff --git a/app/controllers/api/v1/accounts/statuses_controller.rb b/app/controllers/api/v1/accounts/statuses_controller.rb index 8cd8f8e79..7aba2d0bd 100644 --- a/app/controllers/api/v1/accounts/statuses_controller.rb +++ b/app/controllers/api/v1/accounts/statuses_controller.rb @@ -3,6 +3,8 @@ class Api::V1::Accounts::StatusesController < Api::BaseController before_action -> { authorize_if_got_token! :read, :'read:statuses' } before_action :set_account + before_action :check_account_suspension + before_action :check_account_block after_action :insert_pagination_headers respond_to :json @@ -18,6 +20,14 @@ class Api::V1::Accounts::StatusesController < Api::BaseController @account = Account.find(params[:account_id]) end + def check_account_suspension + gone if @account.suspended? + end + + def check_account_block + gone if current_account.present? && @account.blocking?(current_account) + end + def load_statuses cached_account_statuses end diff --git a/app/controllers/api/v1/accounts_controller.rb b/app/controllers/api/v1/accounts_controller.rb index b0c62778e..685e044c3 100644 --- a/app/controllers/api/v1/accounts_controller.rb +++ b/app/controllers/api/v1/accounts_controller.rb @@ -10,6 +10,7 @@ class Api::V1::AccountsController < Api::BaseController before_action :require_user!, except: [:show, :create] before_action :set_account, except: [:create] before_action :check_account_suspension, only: [:show] + before_action :check_account_block, only: [:show] before_action :check_enabled_registrations, only: [:create] respond_to :json @@ -75,6 +76,10 @@ class Api::V1::AccountsController < Api::BaseController gone if @account.suspended? end + def check_account_block + gone if current_account.present? && @account.blocking?(current_account) + end + def account_params params.permit(:username, :email, :password, :agreement, :locale) end diff --git a/app/controllers/api/v1/statuses/favourited_by_accounts_controller.rb b/app/controllers/api/v1/statuses/favourited_by_accounts_controller.rb index 657e57831..e00c4d708 100644 --- a/app/controllers/api/v1/statuses/favourited_by_accounts_controller.rb +++ b/app/controllers/api/v1/statuses/favourited_by_accounts_controller.rb @@ -22,6 +22,7 @@ class Api::V1::Statuses::FavouritedByAccountsController < Api::BaseController def default_accounts Account + .without_blocking(current_account) .includes(:favourites, :account_stat) .references(:favourites) .where(favourites: { status_id: @status.id }) diff --git a/app/controllers/api/v1/statuses/reblogged_by_accounts_controller.rb b/app/controllers/api/v1/statuses/reblogged_by_accounts_controller.rb index 6851099f6..9b2d0e59e 100644 --- a/app/controllers/api/v1/statuses/reblogged_by_accounts_controller.rb +++ b/app/controllers/api/v1/statuses/reblogged_by_accounts_controller.rb @@ -21,7 +21,7 @@ class Api::V1::Statuses::RebloggedByAccountsController < Api::BaseController end def default_accounts - Account.includes(:statuses, :account_stat).references(:statuses) + Account.without_blocking(current_account).includes(:statuses, :account_stat).references(:statuses) end def paginated_statuses diff --git a/app/javascript/mastodon/features/account/components/header.js b/app/javascript/mastodon/features/account/components/header.js index f21ba8a9c..9d15bc28f 100644 --- a/app/javascript/mastodon/features/account/components/header.js +++ b/app/javascript/mastodon/features/account/components/header.js @@ -111,7 +111,7 @@ class Header extends ImmutablePureComponent { } else if (account.getIn(['relationship', 'requested'])) { actionBtn =