aboutsummaryrefslogtreecommitdiff
path: root/src/or
Commit message (Collapse)AuthorAge
* Only count timeout data for 3 hop circuits.Mike Perry2010-09-29
| | | | | Use 4/3 of this timeout value for 4 hop circuits, and use half of it for canabalized circuits.
* Merge branch 'bug1511'Nick Mathewson2010-09-23
|\
| * Even more accurate handling for shifting accounting intervalsNick Mathewson2010-09-21
| | | | | | | | | | | | | | Roger correctly pointed out that my code was broken for accounting periods that shifted forwards, since start_of_accounting_period_containing(interval_start_time) would not be equal to interval_start_time, but potentially much earlier.
| * Tolerate a little skew in accounting intervals.Nick Mathewson2010-09-03
| | | | | | | | | | | | | | This will make changes for DST still work, and avoid double-spending bytes when there are slight changes to configurations. Fixes bug 1511; the DST issue is a bugfix on 0.0.9pre5.
* | Rename has_completed_circuit to can_complete_circuitNick Mathewson2010-09-22
| | | | | | | | Also redocument it. Related to #1362.
* | Merge remote branch 'arma/bug1362'Nick Mathewson2010-09-22
|\ \
| * | log when we finish a circuit after being offlineRoger Dingledine2010-09-21
| | |
* | | Changes to bug1959_part1 on review from arma.Nick Mathewson2010-09-22
| | | | | | | | | | | | | | | Significant one: we want to say "not enough entry nodes descriptors, so we can't build circuits" only when we have 0 descriptors.
* | | Make our min-info check also check for entry node presenceNick Mathewson2010-09-21
| | | | | | | | | | | | Part of a fix for bug1959
* | | Merge branch 'bug1789'Nick Mathewson2010-09-21
|\ \ \ | |/ / |/| |
| * | Bug1789 cleanups suggested by armaNick Mathewson2010-09-21
| | | | | | | | | | | | The significant one is that we look at RelayBandwidthRate if it is set.
| * | finish a comment, lower a variableNick Mathewson2010-09-15
| | |
| * | How many seconds until != timestamp of that dateSebastian Hahn2010-09-15
| | |
| * | Base our expected bw accounting usage on time before soft limitNick Mathewson2010-09-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, we were also considering the time spent in soft-hibernation. If this was a long time, we would wind up underestimating our bandwidth by a lot, and skewing our wakeup time towards the start of the accounting interval. This patch also makes us store a few more fields in the state file, including the time at which we entered soft hibernation. Fixes bug 1789. Bugfix on 0.0.9pre5.
| * | Use a more sophisticated soft-hibernation-limit calculationNick Mathewson2010-09-03
| | | | | | | | | | | | This should help address bug 1789.
| * | Remove an obsolete comment from hibernate.cNick Mathewson2010-09-03
| |/
* | Merge branch 'bug1956'Nick Mathewson2010-09-21
|\ \
| * | Count unknown authorities as unknown only once, not once per signature.Nick Mathewson2010-09-20
| | | | | | | | | | | | | | | | | | Do not double-report signatures from unrecognized authorities both as "from unknown authority" and "not present". Fixes bug 1956, bugfix on 0.2.2.16-alpha.
* | | A start at a patch for bug 1943 (alignment issues)Roger Dingledine2010-09-20
|/ /
* | Attempt to fix compilation on WindowsSebastian Hahn2010-09-19
| | | | | | | | | | | | | | Our attempt to make compilation work on old versions of Windows again while keeping wince compatibility broke the build for Win2k+. helix reports this patch fixes the issue for WinXP. Bugfix on 0.2.2.15-alpha; related to bug 1797.
* | I hear we are close to a release. Clean up the whitespace.Nick Mathewson2010-09-16
| |
* | Merge remote branch 'sebastian/bug1921'Nick Mathewson2010-09-16
|\ \
| * | Demote a warn when enabling CellStatisticsSebastian Hahn2010-09-16
| | | | | | | | | | | | | | | | | | | | | | | | When the CellStatistics option is off, we don't store cell insertion times. Doing so would also not be very smart, because there seem to still be some performance issues with this type of statistics. Nothing harmful happens when we don't have insertion times, so we don't need to alarm the user.
* | | perconnbwrate and perconnbwburst consensus paramsRoger Dingledine2010-09-16
|/ /
* | Merge branch 'bug1184'Nick Mathewson2010-09-15
|\ \
| * | Never queue a cell on a marked circuitNick Mathewson2010-09-15
| | |
| * | Clear cell queues when marking or truncating a circuit.Nick Mathewson2010-07-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | At best, this patch helps us avoid sending queued relayed cells that would get ignored during the time between when a destroy cell is sent and when the circuit is finally freed. At worst, it lets us release some memory a little earlier than it would otherwise. Fix for bug #1184. Bugfix on 0.2.0.1-alpha.
* | | Add a simple integer-ceiling-division macro before we get it wrongNick Mathewson2010-09-14
| | |
* | | Merge branch 'bug1899'Nick Mathewson2010-09-14
|\ \ \
| * | | Extract the "do these routers have the same addr:orport" logic into a fnNick Mathewson2010-09-14
| | | |
| * | | Launch reachability tests for routers whose IP or ORPort changeNick Mathewson2010-09-03
| | | | | | | | | | | | | | | | Implements #1899, suggested by Sebastian. Depends on #911 fix.
* | | | Merge branch 'bug911'Nick Mathewson2010-09-14
|\ \ \ \
| * | | | Tweak some issues found by arma in bug911 review.Nick Mathewson2010-09-14
| |/ / /
| * | | Move code for launching tests out of router_add_to_routerlist()Nick Mathewson2010-08-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | router_add_to_routerlist() is supposed to be a nice minimal function that only touches the routerlist structures, but it included a call to dirserv_single_reachability_test(). We have a function that gets called _after_ adding descriptors successfully: routerlist_descriptors_added. This patch moves the responsibility for testing there. Because the decision of whether to test or not depends on whether there was an old routerinfo for this router or not, we have to first detect whether we _will_ want to run the tests if the router is added. We make this the job of routers_update_status_from_consensus_networkstatus(). Finally, this patch makes the code notice if a router is going from hibernating to non-hibernating, and if so causes a reachability test to get launched.
| * | | Allow some skew in checking when a router said it was hibernatingNick Mathewson2010-08-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This solves the problem Roger noted as: What if the router has a clock that's 5 minutes off, so it publishes a descriptor for 5 minutes in the future, and we test it three minutes in. In this edge case, we will continue to advertise it as Running for the full 45 minute period.
| * | | Clarify AssumeReachable semantics wrt hibernationNick Mathewson2010-08-18
| | | |
| * | | Add some braces to make arma happyNick Mathewson2010-08-18
| | | |
| * | | If a router is hibernating, never vote that it is Running.Nick Mathewson2010-07-31
| |/ / | | | | | | | | | | | | Also, clean up and comment some of the logic in dirserv_set_router_is_running.
* | | Merge remote branch 'sebastian/bug1776_v3'Nick Mathewson2010-09-14
|\ \ \
| * | | Allow clients to use relays as bridgesSebastian Hahn2010-08-20
| | | |
* | | | Make circuit_resume_edge_reading_helper treat streams more fairly.Nick Mathewson2010-09-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously[*], the function would start with the first stream on the circuit, and let it package as many cells as it wanted before proceeding to the next stream in turn. If a circuit had many live streams that all wanted to package data, the oldest would get preference, and the newest would get ignored. Now, we figure out how many cells we're willing to send per stream, and try to allocate them fairly. Roger diagnosed this in the comments for bug 1298. [*] This bug has existed since before the first-ever public release of Tor. It was added by r152 of Tor on 26 Jan 2003, which was the first commit to implement streams (then called "topics"). This is not the oldest bug to be fixed in 0.2.2.x: that honor goes to the windowing bug in r54, which got fixed in e50b7768 by Roger with diagnosis by Karsten. This is, however, the most long-lived bug to be fixed in 0.2.2.x: the r54 bug was fixed 2580 days after it was introduced, whereas I am writing this commit message 2787 days after r152.
* | | | Add a max_cells arg to connection_edge_process_raw_inbufNick Mathewson2010-09-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm going to use this to implement more fairness in circuit_resume_edge_reading_helper in an attempt to fix bug 1298. (Updated with fixes from arma and Sebastian)
* | | | Fix function declaration linebreaks at start of relay.cNick Mathewson2010-09-13
| | | |
* | | | Merge remote branch 'public/pretty-signature-log'Nick Mathewson2010-09-13
|\ \ \ \
| * | | | Complicate the rules on WARN vs INFO in consensus verificationNick Mathewson2010-09-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's normal when bootstrapping to have a lot of different certs missing, so we don't want missing certs to make us warn... unless the certs we're missing are ones that we've tried to fetch a couple of times and failed at. May fix bug 1145.
| * | | | Make the "2 unknown, 7 missing key..." messages easier to understandNick Mathewson2010-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch should fix the cases we care about for bugs 1290 and 1145. "30-56-99 are correct. Limited 4 and 8 are missing".
* | | | | Merge remote branch 'sebastian/bug1525'Nick Mathewson2010-09-13
|\ \ \ \ \
| * | | | | Update some documentation for the bug 1525 fixSebastian Hahn2010-08-08
| | | | | |
| * | | | | Ensure controller RESOLVE commands respect __LeaveStreamsUnattachedRobert Hogan2010-08-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://trac.torproject.org/projects/tor/ticket/1525 "The codepath taken by the control port "RESOLVE" command to create a synthetic SOCKS resolve request isn't the same as the path taken by a real SOCKS request from 'tor-resolve'. This prevents controllers who set LeaveStreamsUnattached=1 from being able to attach RESOLVE streams to circuits of their choosing." Create a new function connection_ap_rewrite_and_attach_if_allowed() and call that when Tor needs to attach a stream to a circuit but needs to know if the controller permits it. No tests added.
* | | | | | Merge branch 'bug1138'Nick Mathewson2010-09-12
|\ \ \ \ \ \