This package would be a great, shared foundation for building APNS support into higher level frameworks. Its only dependencies are NIO, NIOHTTP2, and NIOOpenSSL which means any existing NIO-based projects should be able to integrate this easily without needing to drag in large dependencies.
One thing we should be thinking of is that NIO will likely be dropping OpenSSL soon. This will make NIOAPNS' dependency on OpenSSL less ideal. In that case, we have a few options:
- Find a pure Swift solution
- Embed a more lightweight C library that does ECDSA (like NIOPostgres does for MD5)
- Continue linking OpenSSL and wait for broader solution to Crypto
Option three seems fine as frameworks like Vapor will likely continue to link OpenSSL until a broader Crypto solution is agreed upon. But this is something we should consider.
Overall, huge +1, really nice work on this!