Skip to content
This repository has been archived by the owner on Mar 29, 2022. It is now read-only.

Fix race conditions in AMQPServer, TopicExchange & Binding #28

Merged
merged 1 commit into from
Aug 24, 2019

Conversation

cjorba
Copy link

@cjorba cjorba commented Apr 15, 2019

  • Added mutex for TopicExchange for synchronization.
  • Added mutex for Binding for synchronization.
  • Added mutex for AMQPServer for synchronization.

@codecov-io
Copy link

codecov-io commented Apr 15, 2019

Codecov Report

Merging #28 into master will increase coverage by 0.22%.
The diff coverage is 68.18%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   48.45%   48.67%   +0.22%     
==========================================
  Files          20       20              
  Lines        1197     1210      +13     
==========================================
+ Hits          580      589       +9     
- Misses        593      597       +4     
  Partials       24       24
Impacted Files Coverage Δ
amqptest/server/server.go 3.6% <0%> (-0.04%) ⬇️
amqptest/server/vhost.go 59.82% <100%> (-0.68%) ⬇️
amqptest/server/exchange.go 72.91% <71.42%> (-9.44%) ⬇️
amqptest/dial.go 89.81% <0%> (+3.7%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e762dd0...25d9d0d. Read the comment docs.

@i4ki
Copy link
Contributor

i4ki commented Aug 24, 2019

Thanks @cjorba !

@i4ki i4ki merged commit 3bb9b07 into NeowayLabs:master Aug 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants