Réécriture du package bus pour éviter les deadlocks #21

wpetit · 2023-11-30T15:08:31+01:00

wpetit commented

2023-11-30 15:08:31 +01:00

Cette PR introduit une réécriture en profondeur du package bus qui est au coeur des échanges entre les applications et leurs backends.

L'ancienne implémentation souffrait de problèmes de "deadlock" dans certains cas spécifiques (lorsque le context.Context passé à la méthode bus.Request() avait "expiré" ¹). Cette implémentation en est exempt, comme le prouve les tests intégrés.

Plusieurs améliorations annexes liées à la réécriture sont également intégrées dans les changements.

Lié à #20
Corrige #10

How to use Contexts in Go ↩︎

Cette PR introduit une réécriture en profondeur du package `bus` qui est au coeur des échanges entre les applications et leurs backends. L'ancienne implémentation souffrait de problèmes de "deadlock" dans certains cas spécifiques (lorsque le `context.Context` passé à la méthode `bus.Request()` avait "expiré" [^1]). Cette implémentation en est exempt, comme le prouve les tests intégrés. Plusieurs améliorations annexes liées à la réécriture sont également intégrées dans les changements. [^1]: [How to use Contexts in Go](https://www.digitalocean.com/community/tutorials/how-to-use-contexts-in-go) Lié à https://forge.cadoles.com/arcad/edge/issues/20 Corrige https://forge.cadoles.com/arcad/edge/issues/10

wpetit added 1 commit 2023-11-30 15:08:33 +01:00