autonomic-cooperative/hometown
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

feat: Enable push subscription for mobile devices by default (#4321)

Browse Source
This commit is contained in:
Sorin Davidoi 2017-07-23 23:27:23 +02:00 committed by Eugen Rochko
parent 63a2566007
commit 9075c90c46
3 changed files with 28 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
app/controllers/api/web/push_subscriptions_controller.rb
View File

@ -6,8 +6,8 @@ class Api::Web::PushSubscriptionsController < Api::BaseController
before_action :require_user! before_action :require_user!
def create def create
params.require(:data).require(:endpoint) params.require(:subscription).require(:endpoint)
params.require(:data).require(:keys).require([:auth, :p256dh]) params.require(:subscription).require(:keys).require([:auth, :p256dh])
active_session = current_session active_session = current_session
@ -16,10 +16,23 @@ class Api::Web::PushSubscriptionsController < Api::BaseController
active_session.update!(web_push_subscription: nil) active_session.update!(web_push_subscription: nil)
end end
# Mobile devices do not support regular notifications, so we enable push notifications by default
alerts_enabled = active_session.detection.device.mobile? || active_session.detection.device.tablet?
data = {
alerts: {
follow: alerts_enabled,
favourite: alerts_enabled,
reblog: alerts_enabled,
mention: alerts_enabled,
},
}
web_subscription = ::Web::PushSubscription.create!( web_subscription = ::Web::PushSubscription.create!(
endpoint: params[:data][:endpoint], endpoint: params[:subscription][:endpoint],
key_p256dh: params[:data][:keys][:p256dh], key_p256dh: params[:subscription][:keys][:p256dh],
key_auth: params[:data][:keys][:auth] key_auth: params[:subscription][:keys][:auth],
data: data
) )
active_session.update!(web_push_subscription: web_subscription) active_session.update!(web_push_subscription: web_subscription)

2
app/javascript/mastodon/web_push_subscription.js
View File

@ -37,7 +37,7 @@ const unsubscribe = ({ registration, subscription }) =>
const sendSubscriptionToBackend = (subscription) => const sendSubscriptionToBackend = (subscription) =>
axios.post('/api/web/push_subscriptions', { axios.post('/api/web/push_subscriptions', {
data: subscription, subscription,
}).then(response => response.data); }).then(response => response.data);
// Last one checks for payload support: https://web-push-book.gauntface.com/chapter-06/01-non-standards-browsers/#no-payload // Last one checks for payload support: https://web-push-book.gauntface.com/chapter-06/01-non-standards-browsers/#no-payload

26
spec/controllers/api/web/push_subscriptions_controller_spec.rb
View File

@ -9,7 +9,7 @@ describe Api::Web::PushSubscriptionsController do
let(:create_payload) do let(:create_payload) do
{ {
data: { subscription: {
endpoint: 'https://fcm.googleapis.com/fcm/send/fiuH06a27qE:APA91bHnSiGcLwdaxdyqVXNDR9w1NlztsHb6lyt5WDKOC_Z_Q8BlFxQoR8tWFSXUIDdkyw0EdvxTu63iqamSaqVSevW5LfoFwojws8XYDXv_NRRLH6vo2CdgiN4jgHv5VLt2A8ah6lUX', endpoint: 'https://fcm.googleapis.com/fcm/send/fiuH06a27qE:APA91bHnSiGcLwdaxdyqVXNDR9w1NlztsHb6lyt5WDKOC_Z_Q8BlFxQoR8tWFSXUIDdkyw0EdvxTu63iqamSaqVSevW5LfoFwojws8XYDXv_NRRLH6vo2CdgiN4jgHv5VLt2A8ah6lUX',
keys: { keys: {
p256dh: 'BEm_a0bdPDhf0SOsrnB2-ategf1hHoCnpXgQsFj5JCkcoMrMt2WHoPfEYOYPzOIs9mZE8ZUaD7VA5vouy0kEkr8=', p256dh: 'BEm_a0bdPDhf0SOsrnB2-ategf1hHoCnpXgQsFj5JCkcoMrMt2WHoPfEYOYPzOIs9mZE8ZUaD7VA5vouy0kEkr8=',
@ -36,25 +36,17 @@ describe Api::Web::PushSubscriptionsController do
it 'saves push subscriptions' do it 'saves push subscriptions' do
sign_in(user) sign_in(user)
stub_request(:post, create_payload[:data][:endpoint]).to_return(status: 200) stub_request(:post, create_payload[:subscription][:endpoint]).to_return(status: 200)
post :create, format: :json, params: create_payload post :create, format: :json, params: create_payload
user.reload user.reload
push_subscription = Web::PushSubscription.find_by(endpoint: create_payload[:data][:endpoint]) push_subscription = Web::PushSubscription.find_by(endpoint: create_payload[:subscription][:endpoint])
expect(push_subscription['endpoint']).to eq(create_payload[:data][:endpoint]) expect(push_subscription['endpoint']).to eq(create_payload[:subscription][:endpoint])
expect(push_subscription['key_p256dh']).to eq(create_payload[:data][:keys][:p256dh]) expect(push_subscription['key_p256dh']).to eq(create_payload[:subscription][:keys][:p256dh])
expect(push_subscription['key_auth']).to eq(create_payload[:data][:keys][:auth]) expect(push_subscription['key_auth']).to eq(create_payload[:subscription][:keys][:auth])
end
it 'sends welcome notification' do
sign_in(user)
stub_request(:post, create_payload[:data][:endpoint]).to_return(status: 200)
post :create, format: :json, params: create_payload
end end
end end
@ -62,15 +54,15 @@ describe Api::Web::PushSubscriptionsController do
it 'changes alert settings' do it 'changes alert settings' do
sign_in(user) sign_in(user)
stub_request(:post, create_payload[:data][:endpoint]).to_return(status: 200) stub_request(:post, create_payload[:subscription][:endpoint]).to_return(status: 200)
post :create, format: :json, params: create_payload post :create, format: :json, params: create_payload
alerts_payload[:id] = Web::PushSubscription.find_by(endpoint: create_payload[:data][:endpoint]).id alerts_payload[:id] = Web::PushSubscription.find_by(endpoint: create_payload[:subscription][:endpoint]).id
put :update, format: :json, params: alerts_payload put :update, format: :json, params: alerts_payload
push_subscription = Web::PushSubscription.find_by(endpoint: create_payload[:data][:endpoint]) push_subscription = Web::PushSubscription.find_by(endpoint: create_payload[:subscription][:endpoint])
expect(push_subscription.data['follow']).to eq(alerts_payload[:data][:follow]) expect(push_subscription.data['follow']).to eq(alerts_payload[:data][:follow])
expect(push_subscription.data['favourite']).to eq(alerts_payload[:data][:favourite]) expect(push_subscription.data['favourite']).to eq(alerts_payload[:data][:favourite])