Skip to content

Missing push notifications for all but first device when sending through Delayed::Job #67

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
morrissinger wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Missing push notifications for all but first device when sending through Delayed::Job #67

Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
6b50e7b
debug
morrissinger Oct 16, 2013
abdc650
debug
morrissinger Oct 16, 2013
51358f8
debug
morrissinger Oct 16, 2013
ef8d87c
debug
morrissinger Oct 16, 2013
9c60851
debug
morrissinger Oct 16, 2013
ed59d7a
debug
morrissinger Oct 16, 2013
f3a9bce
debug
morrissinger Oct 16, 2013
f822a66
debug
morrissinger Oct 16, 2013
57bb1b6
debug
morrissinger Oct 16, 2013
833352b
debug
morrissinger Oct 16, 2013
b523e74
debug
morrissinger Oct 16, 2013
6a8d314
debug
morrissinger Oct 16, 2013
83b07d0
debug
morrissinger Oct 16, 2013
70ce9df
debug
morrissinger Oct 16, 2013
41cbb59
debug
morrissinger Oct 16, 2013
5d7add6
debug
morrissinger Oct 16, 2013
8a069b3
debug
morrissinger Oct 16, 2013
b7b9f68
debug
morrissinger Oct 16, 2013
914457b
debug
morrissinger Oct 16, 2013
70d87e9
debug
morrissinger Oct 16, 2013
6cd5a99
debug
morrissinger Oct 16, 2013
d211b0f
debug
morrissinger Oct 16, 2013
415c087
fixed issue with delayed::job and apns getting missed.
morrissinger Oct 16, 2013
cbf9e34
removed some debug output.
morrissinger Oct 16, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions lib/apn_on_rails/app/models/apn/app.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,12 @@ def self.send_notifications_for_cert(the_cert, app_id)
APN::Device.find_each(:conditions => conditions) do |dev|
dev.unsent_notifications.each do |noty|
conn.write(noty.message_for_sending)

# This seems to fix the bug where multiple notifications get
# missed by Apple when sending Push notifications as a worker
# task through Delayed::Job
sleep 0.00000001

noty.sent_at = Time.now
noty.save
end
Expand Down