Fix RabbitMQ connection timeouts during long operations.#856
Merged
Conversation
82cbc4f to
3a0f281
Compare
nesitor
approved these changes
Sep 18, 2025
amalcaraz
pushed a commit
that referenced
this pull request
Sep 18, 2025
nesitor
added a commit
that referenced
this pull request
Oct 8, 2025
…ycle management (#836) * feature: support aleph credits * feat: parse new credit airdrop messages and refactor credit_balances table and primary key * fix: change credit amount type from Decimal to int * fix: lint errors * feat: implement credit cost calculation and garbage collector * point to tmp branch for aleph-message package * fix: added missing migration for error code * fix: balance Decimal type in http response * fix: fix migrations order * feat: filter accepted credit messages with channel + address + post types * feat: remove /api/v0/addresses/{address}/credit_balance and integrate it in /api/v0/addresses/{address}/balance * feat: support credit distribution, transfer, expense * fix: added more tests * fix: credit_history table and fifo calculation taking into account expiration and expenses * feat: credit history endpoint + lint fixes * fix: validate_balance_for_payment to take into account all resources + new one for the next 24 hours * fix: validate_balance_for_payment to take into account all resources + new one for the next 24 hours * Solve format issue on balance pre-check (#855) * Fix: Solve issue on format getting IPFS file sizes before downloading the entire content. * Fix: Added test cae for string dag_node format * Fix RabbitMQ connection timeouts during long operations. (#856) * Fix IPFS conf: use the recommended method to update IPFS configuration (#846) * fix: lint * fix: pending_message queue getting stuck due to bad error handling * Fix: Added a default internal code for errors. * fix: bugfix caching expired credit balances * fix: lint fix * fix: CI fixes due to hatch problematic dep * feat: new index in credit_history.expiration_date column * fix: added cost_credit in cost http response * fix: change pricing aggregate owner for testing * feat: added charged_address in /price and /price/estimate response * Fix: Upgrade `aleph_message` version to `1.0.5` * Fix: Pin pydantic version to the previous one from latest. * Fix: Solved failing test and put legacy and current PAYG message tests. * fix: fix account addresses for credit and settings * fix: new credit channel --------- Co-authored-by: nesitor <[email protected]> Co-authored-by: Alie.E <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
During long operations, the mq client is disconnected by the server and loses the context of processed messages from the mq exchange.
This behavior introduces inconsistency in the processing order between each processing task.
Changes
Setting the heartbeat timeout to 10 minutes is sufficient to avoid this problem.