From c2c93f8cd64704c715f5fb843e2b532779a1ca0f Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Wed, 19 Jul 2017 12:00:05 +0200 Subject: Fix deprecation warning for non-pipeline assets (#4267) --- app/models/web/push_subscription.rb | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'app/models') diff --git a/app/models/web/push_subscription.rb b/app/models/web/push_subscription.rb index baf6a1ece..dffdbbd05 100644 --- a/app/models/web/push_subscription.rb +++ b/app/models/web/push_subscription.rb @@ -45,7 +45,7 @@ class Web::PushSubscription < ApplicationRecord title: title, dir: dir, image: image, - badge: full_asset_url('badge.png'), + badge: full_asset_url('badge.png', skip_pipeline: true), tag: notification.id, timestamp: notification.created_at, icon: notification.from_account.avatar_static_url, @@ -115,7 +115,7 @@ class Web::PushSubscription < ApplicationRecord when :mention then [ { title: translate('push_notifications.mention.action_favourite'), - icon: full_asset_url('emoji/2764.png'), + icon: full_asset_url('emoji/2764.png', skip_pipeline: true), todo: 'request', method: 'POST', action: "/api/v1/statuses/#{notification.target_status.id}/favourite", @@ -156,8 +156,8 @@ class Web::PushSubscription < ApplicationRecord Webpush.payload_send( message: JSON.generate( title: translate('push_notifications.subscribed.title'), - icon: full_asset_url('android-chrome-192x192.png'), - badge: full_asset_url('badge.png'), + icon: full_asset_url('android-chrome-192x192.png', skip_pipeline: true), + badge: full_asset_url('badge.png', skip_pipeline: true), data: { content: translate('push_notifications.subscribed.body'), actions: [], -- cgit From bbdcfd6baf8da01098eb377c3d3579b23ae54d80 Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Wed, 19 Jul 2017 17:06:46 +0200 Subject: Add ActivityPub attributes to accounts (#4273) --- app/models/account.rb | 7 +++++++ app/services/send_interaction_service.rb | 2 +- app/services/subscribe_service.rb | 2 ++ app/services/unsubscribe_service.rb | 2 ++ db/migrate/20170718211102_add_activitypub_to_accounts.rb | 9 +++++++++ db/schema.rb | 7 ++++++- 6 files changed, 27 insertions(+), 2 deletions(-) create mode 100644 db/migrate/20170718211102_add_activitypub_to_accounts.rb (limited to 'app/models') diff --git a/app/models/account.rb b/app/models/account.rb index 58b0a1086..82c9b58d5 100644 --- a/app/models/account.rb +++ b/app/models/account.rb @@ -36,6 +36,11 @@ # followers_count :integer default(0), not null # following_count :integer default(0), not null # last_webfingered_at :datetime +# inbox_url :string default(""), not null +# outbox_url :string default(""), not null +# shared_inbox_url :string default(""), not null +# followers_url :string default(""), not null +# protocol :integer default("ostatus"), not null # class Account < ApplicationRecord @@ -49,6 +54,8 @@ class Account < ApplicationRecord include Remotable include EmojiHelper + enum protocol: [:ostatus, :activitypub] + # Local users has_one :user, inverse_of: :account diff --git a/app/services/send_interaction_service.rb b/app/services/send_interaction_service.rb index ef38a748b..ab0d3aeed 100644 --- a/app/services/send_interaction_service.rb +++ b/app/services/send_interaction_service.rb @@ -10,7 +10,7 @@ class SendInteractionService < BaseService @source_account = source_account @target_account = target_account - return if block_notification? + return if !target_account.ostatus? || block_notification? delivery = build_request.perform diff --git a/app/services/subscribe_service.rb b/app/services/subscribe_service.rb index f58067038..c1c0a4c8b 100644 --- a/app/services/subscribe_service.rb +++ b/app/services/subscribe_service.rb @@ -2,6 +2,8 @@ class SubscribeService < BaseService def call(account) + return unless account.ostatus? + @account = account @account.secret = SecureRandom.hex @response = build_request.perform diff --git a/app/services/unsubscribe_service.rb b/app/services/unsubscribe_service.rb index c2f022d7d..c5e0e73fe 100644 --- a/app/services/unsubscribe_service.rb +++ b/app/services/unsubscribe_service.rb @@ -2,6 +2,8 @@ class UnsubscribeService < BaseService def call(account) + return unless account.ostatus? + @account = account @response = build_request.perform diff --git a/db/migrate/20170718211102_add_activitypub_to_accounts.rb b/db/migrate/20170718211102_add_activitypub_to_accounts.rb new file mode 100644 index 000000000..c08e38bb9 --- /dev/null +++ b/db/migrate/20170718211102_add_activitypub_to_accounts.rb @@ -0,0 +1,9 @@ +class AddActivityPubToAccounts < ActiveRecord::Migration[5.1] + def change + add_column :accounts, :inbox_url, :string, null: false, default: '' + add_column :accounts, :outbox_url, :string, null: false, default: '' + add_column :accounts, :shared_inbox_url, :string, null: false, default: '' + add_column :accounts, :followers_url, :string, null: false, default: '' + add_column :accounts, :protocol, :integer, null: false, default: 0 + end +end diff --git a/db/schema.rb b/db/schema.rb index 5ec78a7c9..cf7b0722f 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -10,7 +10,7 @@ # # It's strongly recommended that you check this file into your version control system. -ActiveRecord::Schema.define(version: 20170714184731) do +ActiveRecord::Schema.define(version: 20170718211102) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" @@ -56,6 +56,11 @@ ActiveRecord::Schema.define(version: 20170714184731) do t.integer "followers_count", default: 0, null: false t.integer "following_count", default: 0, null: false t.datetime "last_webfingered_at" + t.string "inbox_url", default: "", null: false + t.string "outbox_url", default: "", null: false + t.string "shared_inbox_url", default: "", null: false + t.string "followers_url", default: "", null: false + t.integer "protocol", default: 0, null: false t.index "(((setweight(to_tsvector('simple'::regconfig, (display_name)::text), 'A'::\"char\") || setweight(to_tsvector('simple'::regconfig, (username)::text), 'B'::\"char\")) || setweight(to_tsvector('simple'::regconfig, (COALESCE(domain, ''::character varying))::text), 'C'::\"char\")))", name: "search_index", using: :gin t.index "lower((username)::text), lower((domain)::text)", name: "index_accounts_on_username_and_domain_lower" t.index ["uri"], name: "index_accounts_on_uri" -- cgit From dcb94971484c46e470b89032f174267ace86be5c Mon Sep 17 00:00:00 2001 From: Sorin Davidoi Date: Wed, 19 Jul 2017 22:37:52 +0200 Subject: fix(push_subscription): Guard against malformed favourite notification (#4271) --- app/models/web/push_subscription.rb | 10 ++++------ app/services/notify_service.rb | 7 ++++++- 2 files changed, 10 insertions(+), 7 deletions(-) (limited to 'app/models') diff --git a/app/models/web/push_subscription.rb b/app/models/web/push_subscription.rb index dffdbbd05..86df9b591 100644 --- a/app/models/web/push_subscription.rb +++ b/app/models/web/push_subscription.rb @@ -26,8 +26,6 @@ class Web::PushSubscription < ApplicationRecord before_create :send_welcome_notification def push(notification) - return unless pushable? notification - name = display_name notification.from_account title = title_str(name, notification) body = body_str notification @@ -69,6 +67,10 @@ class Web::PushSubscription < ApplicationRecord ) end + def pushable?(notification) + data && data.key?('alerts') && data['alerts'][notification.type.to_s] + end + def as_payload payload = { id: id, @@ -148,10 +150,6 @@ class Web::PushSubscription < ApplicationRecord rtl?(body) ? 'rtl' : 'ltr' end - def pushable?(notification) - data && data.key?('alerts') && data['alerts'][notification.type.to_s] - end - def send_welcome_notification Webpush.payload_send( message: JSON.generate( diff --git a/app/services/notify_service.rb b/app/services/notify_service.rb index c7d8ad50a..a44df5180 100644 --- a/app/services/notify_service.rb +++ b/app/services/notify_service.rb @@ -65,7 +65,12 @@ class NotifyService < BaseService end def send_push_notifications - sessions_with_subscriptions_ids = @recipient.user.session_activations.where.not(web_push_subscription: nil).pluck(:id) + # HACK: Can be caused by quickly unfavouriting a status, since creating + # a favourite and creating a notification are not wrapped in a transaction. + return if @notification.activity.nil? + + sessions_with_subscriptions = @recipient.user.session_activations.where.not(web_push_subscription: nil) + sessions_with_subscriptions_ids = sessions_with_subscriptions.select { |session| session.web_push_subscription.pushable? @notification }.map(&:id) WebPushNotificationWorker.push_bulk(sessions_with_subscriptions_ids) do |session_activation_id| [session_activation_id, @notification.id] -- cgit