[Feature] Multiple APNS projects and certificates

[mrzepinski]

For Apple Push (APNS) we have one certificate for one project. It could be possible to create multiple projects with this bundle?

I don't see the place in code to define more than one path to certificate for APNS.

[richsage]

You're right - it doesn't currently have support for multiple APNS certificates. Thanks - I'll mark this as a feature request :-)

[mrzepinski]

It is similar situation for Authorization key on Android platform. One Google account has one Authorization key, but if we have multiple applications with different Authorization keys because many Google accounts have been used this is a huge problem ;-)

[ZhukV]

+1!

[bassrock]

Take a look here https://github.com/DABSquared/DABSquaredPushNotificationsBundle, I made a push bundle originally based off this one, but with functionality like FOSCommentBundle. I also added support for multiple certificates and apps.

[akira28]

I've developed the ability to configure multiple pem for iOS and Mac apps in my fork https://github.com/Yameveo/RMSPushNotificationsBundle/tree/multcert

It's a little rough around the edges, but it works for our apps. It needs some changes to documentation. @richsage If you want I can create a pull request.

[danielsan80]

Hi Richsage! This bundle is very elastic. If you want multiple pem you can redefine some classes. The problem is that anyway I must set one pem in config yml. The solution could be to remove some vincula in Configuration.php. It should work without configuration keys.

[Thomasgeley]

Akira28 did a great work with the multcert branch ! Just what I needed, thank you.

[richsage]

@akira28 sorry for the delay! A PR would be great if you're able to do that? I think this is a long-overdue feature! :-)

[akira28]

@richsage I've just created the PR. It needs updates to the docs, but I don't have time at the moment to update it, sorry

[Kevinrob]

The pull request #93 make it possible apparently.

[richsage]

I can see #93 allows a second certificate to be used at runtime; I'd prefer this to be done as part of configuration, which @akira28 added in #76 but which needs rebasing and the Travis build fixing before I can merge.

[richsage]

93 has been merged.

76 is awaiting a rebase and fixing of tests.

[akira28]

Hi @richsage tomorrow I'll rebase and try to fix tests. Bye!

[davewwww]

+1

[rahit]

Any update on this feature ?

[Hasib-rafi1]

+1

[blaues0cke]

@rahit @Hasib-rafi1 what are you "+1ing" for? This feature is basically available on the main branch, see: https://github.com/richsage/RMSPushNotificationsBundle/pull/93