Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Add connection timeouts for HTTP2 connections to APNs #1393

Open
jrconlin opened this issue May 15, 2020 · 0 comments
Open

Add connection timeouts for HTTP2 connections to APNs #1393

jrconlin opened this issue May 15, 2020 · 0 comments
Assignees
Labels
5 Estimate - l - Moderately complex, will require some effort but clearly defined.

Comments

@jrconlin
Copy link
Member

We're seeing increased connection errors after 7 days. Current theory is that our servers aren't disconnecting during idle and that's being tagged as abusive. Need to add in a reaper for idle hyper connections.

This is currently impacting iOS service.

@jrconlin jrconlin added the 5 Estimate - l - Moderately complex, will require some effort but clearly defined. label May 15, 2020
@jrconlin jrconlin self-assigned this May 15, 2020
jrconlin added a commit that referenced this issue May 15, 2020
jrconlin added a commit that referenced this issue May 16, 2020
jrconlin added a commit that referenced this issue Jun 25, 2020
Issue #1393 notes, sending push messages across APNS seems to work
great after a deploy, but then degrades after a week or so. #1394 is
a possible work-around (by double-pooling connections and using a
dedicated connection terminator), but it's messy and hacky.

What's really needed is a bit more visibility into what may be
happening, and that will involve logging all APNS communication
exceptions reliably.

Issue #1393
jrconlin added a commit that referenced this issue Jul 1, 2020
Issue #1393 notes, sending push messages across APNS seems to work
great after a deploy, but then degrades after a week or so. #1394 is
a possible work-around (by double-pooling connections and using a
dedicated connection terminator), but it's messy and hacky.

What's really needed is a bit more visibility into what may be
happening, and that will involve logging all APNS communication
exceptions reliably.

Issue #1393
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
5 Estimate - l - Moderately complex, will require some effort but clearly defined.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant