Skip to content

Conversation

@ruslandoga
Copy link
Contributor

@ruslandoga ruslandoga commented Mar 10, 2025

@cnkk cnkk added the preview label Mar 10, 2025
@github-actions
Copy link

Preview environment👷🏼‍♀️🏗️
PR-5166

@cnkk
Copy link
Member

cnkk commented Mar 10, 2025

@ruslandoga looks good! tested this and no errors anymore :)

@aerosol aerosol added this pull request to the merge queue Mar 11, 2025
Merged via the queue into master with commit 1dbad70 Mar 11, 2025
9 checks passed
@cnkk cnkk deleted the update-ch branch March 11, 2025 08:47
@ruslandoga
Copy link
Contributor Author

ruslandoga commented Mar 12, 2025

👋 @cnkk

I think I am still seeing these errors with ch v0.3.2. Maybe something like plausible/ch#249 could be tried next? WDYT?

More info on the differences between this and the other PRs:

  • this PR attempted re-connecting a connection that has received connection: close in the response headers
  • that other PR would try re-connecting a connection that has received a FIN (or RST?) packet
  • final attempt (not ready) would try re-connecting an abnormally closed connection

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants