Skip to content

Set CloseClientOnQueueFull to false by default#434

Merged
mathieucarbou merged 1 commit intomainfrom
issue/433
Apr 21, 2026
Merged

Set CloseClientOnQueueFull to false by default#434
mathieucarbou merged 1 commit intomainfrom
issue/433

Conversation

@mathieucarbou
Copy link
Copy Markdown
Member

@mathieucarbou mathieucarbou commented Apr 21, 2026

This PR changes the default flag for CloseClientOnQueueFull to set it to false.

false should be the sensible default for this library especially after the recent refactorings allowing for a very fast message delivery.

Applications have to check the status when sending and queue status to check if they need to decrease the sending rate or close the client.

Ref: #433

This PR changes the default flag for CloseClientOnQueueFull to set it to false.

false should be the sensible default for this library especially after the recent refactorings allowing for a very fast message delivery.

Applications have to check the status when sending and queue status to check if they need to decrease the sending rate or close the client.

Ref: #433
@mathieucarbou
Copy link
Copy Markdown
Member Author

Also updated doc (website) and examples

@mathieucarbou mathieucarbou merged commit f520559 into main Apr 21, 2026
34 checks passed
@mathieucarbou mathieucarbou deleted the issue/433 branch April 21, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LoadProhibited in AsyncMiddlewareChain::_runChain when setCloseClientOnQueueFull closes a client during concurrent HTTP request

2 participants