Proposed changes

This actually fixes two things:

• When sending the same notification to multiple devices, the notification object was being modified to create a payload property as an EJSON, but since this object was a reference, it was being encoded multiple times (for each device sent), so I move the enconding to inside each push notification provider function (APN and GCM) instead of modifying the notification object itself
• If there was any error with an APN certificate, an error was thrown while instantiating apn.Provider and the server start up was getting stuck, so I added a try/catch to prevent that from hapenning.

Issue(s)

How to test or reproduce

Screenshots

Types of changes

• Bugfix (non-breaking change which fixes an issue)
• Improvement (non-breaking change which improves a current function)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Hotfix (a major bugfix that has to be merged asap)
• Documentation Update (if none of the other choices apply)

Checklist

• I have read the CONTRIBUTING doc
• I have signed the CLA
• Lint and unit tests pass locally with my changes
• I have added tests that prove my fix is effective or that my feature works (if applicable)
• I have added necessary documentation (if applicable)
• Any dependent changes have been merged and published in downstream modules

Changelog

Further comments