Correctly resolve multiple inbound connections to complete-closure #42

Merged

erinn merged 1 commits from networking into master 8 months ago

sarah commented 8 months ago

Owner

Connect() now prevents Open if there are any connections under the assumption
that one of them will be resolved valid. (Before connect allowed Open on any
error, this was a bug that would occasionally be tripped by multiple Inbounds)

erinn was assigned by sarah 8 months ago

sarah added 1 commit 8 months ago

f1e3f2ca54 Correctly resolve multiple inbound connections to complete-closure

Connect() now prevents Open if there are *any connections* under the assumption
that one of them will be resolved valid. (Before connect allowed Open on any
error, this was a bug that would occasionally be tripped by multiple Inbounds)

buildbot commented 8 months ago

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/tapir/94

erinn approved these changes 8 months ago

erinn merged commit 459e00b423 into master 8 months ago

erinn referenced this issue from a commit 8 months ago

Merge pull request 'Correctly resolve multiple inbound connections to complete-closure' (#42) from networking into master

Reviewers

erinn approved these changes 8 months ago

continuous-integration/drone/push Build is passing

Details

continuous-integration/drone/pr Build is passing

Details

The pull request has been merged as 459e00b423.

Sign in to join this conversation.

No Label

No Milestone

3 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.