[Feature request] Connection pool about pyapns2 HOT 3 CLOSED

But does the library actually open-close connections constantly? My understanding was that connection is left opened after the first push, and then you can reuse it as many times as you want from as many threads as you want.

Thread-safety assumption comes from hyper package changelog (used for HTTP20Connection class):

The HTTP20Connection object is now thread-safe, so long as stream IDs are used on all method calls.

And as it is a HTTP/2 connection, library heavily reuses streams inside a single connection, which per my understanding is not the same as a connection pool of course, but gives you similar benefits.

Maybe the author could comment more on that.

from pyapns2.

@vinnyspb, good point! I missed out the fact that PyAPNs2 relies on http20, so I was expecting to see code managing connection at the PyAPNs2 library level and I didn't see any. As it turns out, I checked out the source code of the http20 package after seeing your comment, the connection management has already been well taken care of by http20. Thanks for your insight.

from pyapns2.

My understanding was that connection is left opened after the first push, and then you can reuse it as many times as you want from as many threads as you want.

I'm concerned there may be thread-safety issues. For example, send_notification_batch() doesn't seem to take into account that there may be open streams pending from other calls to that method on other threads. If I'm not mistaken, this could result in too many (or more than __max_concurrent_streams) simultaneous requests.

from pyapns2.