Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bug29875 master #1067

Merged
merged 5 commits into from Jun 4, 2019
Merged

Bug29875 master #1067

merged 5 commits into from Jun 4, 2019

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@nmathewson @coveralls @torproject-pusher
@nmathewson
Copy link
Contributor

@nmathewson nmathewson commented May 31, 2019

No description provided.

nmathewson added 5 commits May 31, 2019
@nmathewson
Consider dir info to have changed when the bridges change
5b3c886 
Otherwise, we won't realize that we haven't got enough bridge
information to build circuits.  Part of a fix for ticket 29875.
@nmathewson
num_bridges_usable(): only count configured bridges.
8015979 
When this function was implemented, it counted all the entry guards
in the bridge set.  But this included previously configured bridges,
as well as currently configured ones!  Instead, only count the
_filtered_ bridges (ones that are configured and possibly reachable)
as maybe usable.

Fixes bug 29875; bugfix on 0.3.0.1-alpha.
@nmathewson
changes file for bug 29875.
c09e7a4
@nmathewson
Merge branch 'bug29875_035' into bug29875_master
fd8beab
@nmathewson
Practracker excpetions.
5e59483
@coveralls
Copy link

@coveralls coveralls commented May 31, 2019

Pull Request Test Coverage Report for Build 5170

  • 0 of 4 (0.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.003%) to 62.588%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/app/config/config.c 0 2 0.0%
src/feature/client/entrynodes.c 0 2 0.0%
Totals Coverage Status
Change from base Build 5159: -0.003%
Covered Lines: 46880
Relevant Lines: 74902
💛 - Coveralls

@torproject-pusher torproject-pusher merged commit 5e59483 into torproject:master Jun 4, 2019
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment