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

Bug33491 master #1802

Closed
wants to merge 6 commits into from
Closed

Bug33491 master #1802

wants to merge 6 commits into from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@teor2345 @coveralls @torproject-pusher @dgoulet-tor
@teor2345
Copy link
Contributor

@teor2345 teor2345 commented Mar 17, 2020

No description provided.

dgoulet-tor and others added 6 commits Mar 10, 2020
@dgoulet-tor
dos: Pass transport name on new client connection
894ff2d 
For a bridge configured with a pluggable transport, the transport name is
used, with the IP address, for the GeoIP client cache entry.

However, the DoS subsystem was not aware of it and always passing NULL when
doing a lookup into the GeoIP cache.

This resulted in bridges with a PT are never able to apply DoS defenses for
newly created connections.

Fixes #33491

Signed-off-by: David Goulet <dgoulet@torproject.org>
@teor2345
Merge branch 'maint-0.3.5' into bug33491_035
032e83b
@teor2345
Merge branch 'bug33491_035' into bug33491_041
4764091
@teor2345
Merge branch 'bug33491_041' into bug33491_042
fac6009
@teor2345
Merge branch 'bug33491_042' into bug33491_043
4bfc65a
@teor2345
Merge branch 'bug33491_043' into bug33491_master
736a91a
@coveralls
Copy link

@coveralls coveralls commented Mar 17, 2020

Pull Request Test Coverage Report for Build 8405

  • 2 of 3 (66.67%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 63.872%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/core/or/channel.c 0 1 0.0%
Totals Coverage Status
Change from base Build 8403: 0.0%
Covered Lines: 50570
Relevant Lines: 79174
💛 - Coveralls

@torproject-pusher torproject-pusher deleted the branch torproject:master May 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment