Skip to content

Conversation

@zone117x
Copy link
Contributor

@zone117x zone117x commented Oct 30, 2024

Workaround for bug in stacks-core event emitter http code. Regression in stacks-core where http requests must now have their POST body read before closing the connection.

chainhook-1    | {"msg":"POST /drop_mempool_tx","level":"DEBUG","ts":"2024-10-30T10:41:34.030361652Z"}
stacks-node-1  | WARN [1730284894.031170] [testnet/stacks-node/src/event_dispatcher.rs:496] [relayer-http://0.0.0.0:20443/] Event dispatcher: connection or request failed to chainhook:20455 - Custom { kind: Other, error: "Failed to send 8192 bytes: \"Failed to send socket data\"" }, backoff: 158.191s, attempts: 10

Chainhook ignored some events by returning a 200 http response and closing the connection after reading the http request headers, and ignoring the request body.

@zone117x zone117x requested a review from rafaelcr October 30, 2024 11:41
@zone117x zone117x merged commit a01470e into stacks-signers Oct 31, 2024
8 checks passed
@zone117x zone117x deleted the fix/ignored-events-post-body branch October 31, 2024 14:24
@zone117x zone117x had a problem deploying to Development-mainnet October 31, 2024 14:36 — with GitHub Actions Failure
@zone117x zone117x had a problem deploying to Development-testnet October 31, 2024 14:36 — with GitHub Actions Failure
github-actions bot pushed a commit that referenced this pull request Oct 31, 2024
## [1.9.0-stacks-signers.6](v1.9.0-stacks-signers.5...v1.9.0-stacks-signers.6) (2024-10-31)

### Bug Fixes

* read stacks-core http event POST payloads for ignored events ([#673](#673)) ([a01470e](a01470e))
@github-actions
Copy link

🎉 This PR is included in version 1.9.0-stacks-signers.6 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

hirosystems-release-bot bot added a commit that referenced this pull request Jun 16, 2025
## [1.9.0](v1.8.0...v1.9.0) (2025-06-16)

### Features

* add mock signature pubkey ([#670](#670)) ([c414d79](c414d79))
* add mock signer message support ([#669](#669)) ([ad5fd54](ad5fd54))
* add various new nakamoto block fields to `/new_block` ingestion and `StacksPayload` ([#659](#659)) ([f48cda5](f48cda5))
* callback to intercept and modify predicates before re-registration ([#675](#675)) ([5f13912](5f13912))
* include recovered signer pubkeys in new block payload ([#662](#662)) ([b5ad4ba](b5ad4ba))
* include signer messages in Stacks predicate payloads ([#656](#656)) ([aee14bc](aee14bc))
* parse `/stackerdb_chunks` Stacks node event ([#653](#653)) ([e44d84a](e44d84a))
* store signer messages in local sqlite database ([#664](#664)) ([d12acd9](d12acd9))
* support tenure-height in block messages ([#661](#661)) ([99e5a61](99e5a61))
* **ts-client:** add persisted predicates and periodic health check ([#658](#658)) ([535226a](535226a))
* **ts-client:** add signer messages to stacks payloads ([#657](#657)) ([ff66bb2](ff66bb2))

### Bug Fixes

* add a configurable max timeout for outgoing predicate payload requests ([#642](#642)) ([6c1dfa9](6c1dfa9)), closes [#643](#643)
* add missing fields to ordinal inscription reveals ([#703](#703)) ([4e49608](4e49608))
* bump stacks codec ([3c117b5](3c117b5))
* increase stacks event server body size limit to 500MB ([#660](#660)) ([21234c9](21234c9))
* mock signature retrieval ([#671](#671)) ([8bb59c1](8bb59c1))
* only import stacks tsv if chainstate is empty ([#684](#684)) ([1f9f37e](1f9f37e)), closes [#677](#677)
* propagate error when trying to store peer info ([1bc949a](1bc949a))
* read contract_interface from Stacks node block events ([#712](#712)) ([6f9c718](6f9c718))
* read stacks-core http event POST payloads for ignored events ([#673](#673)) ([a01470e](a01470e))
* return 500 status code to Stacks node when event ingestion fails ([#648](#648)) ([63c753c](63c753c)), closes [#646](#646)
* signer pubkey calculation ([#665](#665)) ([cc93873](cc93873))
* ts client missing matches_regex option for print_event ([#701](#701)) ([8247900](8247900))
* update SignerMessage deserializing ([#663](#663)) ([71364c1](71364c1))
* validate ts client predicates before registering ([#639](#639)) ([2d08f72](2d08f72))
@hirosystems-release-bot
Copy link

🎉 This PR is included in version 1.9.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

edge-dragonmkop added a commit to edge-dragonmkop/chainhook that referenced this pull request Sep 29, 2025
## [1.9.0-stacks-signers.6](hirosystems/chainhook@v1.9.0-stacks-signers.5...v1.9.0-stacks-signers.6) (2024-10-31)

### Bug Fixes

* read stacks-core http event POST payloads for ignored events ([#673](hirosystems/chainhook#673)) ([a01470e](hirosystems/chainhook@a01470e))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants