pushmeup icon indicating copy to clipboard operation
pushmeup copied to clipboard

Check string keys in APNS::Notifications message hash

Open lserman opened this issue 10 years ago • 0 comments

When queuing a message through Redis (Sidekiq in my case), hash parameters are serialized and unserialized via JSON dump/load, so any symbol keys are converted to strings. Since APNS::Notification only checks symbol keys, the push will fail unless you symbolize the keys again after the body is given to the Sidekiq worker.

This PR will check for string keys in addition to symbol keys so that this situation can be avoided.

lserman avatar Oct 27 '14 22:10 lserman