summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | Change renderAnswer() to take bridge lines, not Bridge classes.Isis Lovecruft2014-01-12
| | | | | | | | | | Otherwise, we'd also have to pass the `transport`, and `addressClass` parameters to renderAnswer().
* | Add missing import for bridgedb.Util in bridgedb.Bridges.Isis Lovecruft2014-01-12
| |
* | Cleanup the log statements for hashring prepopulation in bridgedb.Main.Isis Lovecruft2014-01-12
| |
* | Change loglevel on message about adding PTs to a bridge in bridgedb.Main.Isis Lovecruft2014-01-12
| |
* | Update docstring parameters for IPBasedDistributor.getBridgesForIP().Isis Lovecruft2014-01-12
| |
* | Log when a distributor's hashrings are being prepopulated.Isis Lovecruft2014-01-12
| |
* | Call Distributor.setDistributorName() in subclass __init__s.Isis Lovecruft2014-01-12
| |
* | Fix bug where open() was called even when COUNTY_BLOCK_FILE=None.Isis Lovecruft2014-01-12
| |
* | Log when we're dumping bridge assignments to a file in bridgedb.Bucket.Isis Lovecruft2014-01-12
| |
* | Add coding: utf-8 header to lib/bridgedb/Bucket.py.Isis Lovecruft2014-01-12
| |
* | Fix a potential bug where an OR ID might have been unprocessed base64.Isis Lovecruft2014-01-12
| | | | | | | | | | | | | | * CHANGE bridgedb.parse.networkstatus.parseRLine() to check the OR nickname validity only after parsing (and readding the base64 padding to) the OR ID digest. * ADD inline comments about why this is done.
* | Update bridgedb.parse.networkstatus module docstring.Isis Lovecruft2014-01-12
| |
* | Don't log the hashring filters twice, just log the number of bridges inserted.Isis Lovecruft2014-01-12
| |
* | Remove extraneous log statement from FilteredBridgeSplitter.prepopulate().Isis Lovecruft2014-01-12
| |
* | Log when we dump assignments to a file.Isis Lovecruft2014-01-12
| |
* | Name subrings after their parent hashring's distributor.Isis Lovecruft2014-01-12
| | | | | | | | | | | | | | | | | | * MOVE logic for parsing filternames from the "unique string" in method FilteredBridgeSplitter.addRing() to new method FilteredBridgeSplitter.extractFilterNames(). * ADD a FilteredBridgeSplitter.distributorName attribute which is set when Dist.Distributor.setDistributorName() is called. * REFACTOR FilteredBridgeSplitter.addRing() to use the new subring names.
* | Merge branch 'fix/10559-keyid-in-pt-lines' into developIsis Lovecruft2014-01-12
|\ \
| * | Make the 'Bridge ' prefix optional in PluggableTransport.getTransportLine().Isis Lovecruft2014-01-12
| | |
| * | Refactor PluggableTransport.getTransportLine() to remove `keyid=`.Isis Lovecruft2014-01-12
| | | | | | | | | | | | | | | | | | | | | * FIXES #10559 * FIXES a bunch of PEP8 problems. * CHANGE getTransportLine() to use `' '.join()` rather than formatting the bridge line inside the return statement. It's faster and safer.
* | | Log a message if there is a duplicate bridge from a sub-hashring.Isis Lovecruft2014-01-12
| | |
* | | Log countryCodes in BridgeRing.getBridges().Isis Lovecruft2014-01-12
| | | | | | | | | | | | | | | | | | This is to see if they are ever actually passed into this function. They weren't being used anywhere inside BridgeRing.getBridges(), nor anywhere else.
* | | Use bridges.ip in log message in BridgeRing.insert().Isis Lovecruft2014-01-12
|/ / | | | | | | * REMOVE the comment and other cruft around it.
* | Merge branch 'fix/6127-web-server-tracebacks' into developIsis Lovecruft2014-01-12
|\ \
| * | Add exception-catching wrapper to bridgedb.HTTPServer.render().Isis Lovecruft2014-01-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * ADD a wrapper which catches all unhandled exceptions which occur while generating an HTML response. If such an exception occurs, this ensures that the error is logged and the default "No bridges currently available" response is served to the client (rather than display a traceback to the client, which is what BridgeDB did previously). * FIXES #6127
| * | Separate HTML response generation logic out of getBridgeRequestAnswer().Isis Lovecruft2014-01-12
|/ / | | | | | | | | | | | | | | * MOVE logic for generating HTML responses out of bridgedb.HTTPServer.WebResource.getBridgeRequestAnswer() and into a new method, bridgedb.HTTPServer.WebResource.renderAnswer(). This allows other methods to generate responses, even if the request was malformed.
* | Update/add docstrings for bridgedb.Bridges.FilteredBridgeSplitter.Isis Lovecruft2014-01-12
| |
* | Minor PEP8 whitespace fix in bridgedb.Bridges.BridgeRing.getBridges().Isis Lovecruft2014-01-12
| |
* | Add docstring for bridgedb.Bridges.BridgeRing.getBridges().Isis Lovecruft2014-01-12
| |
* | Add docstring for bridgedb.Bridges.BridgeRing._getBridgeKeysAt().Isis Lovecruft2014-01-12
| |
* | Minor PEP8 whitespace fix in bridgedb.Bridges.BridgeRing.insert().Isis Lovecruft2014-01-12
| |
* | Add docstring for bridgedb.Bridges.BridgeRing.insert().Isis Lovecruft2014-01-12
| |
* | Add docstring for bridgedb.Bridges.BridgeRing.clear().Isis Lovecruft2014-01-12
| |
* | Add docstring for bridgedb.Bridges.BridgeRing.__len__().Isis Lovecruft2014-01-12
| |
* | Minor PEP8 whitespace fixes in bridgedb.Bridges.BridgeRing.setName().Isis Lovecruft2014-01-12
| |
* | Fix doctest example in Dist.Distributor.setDistributorName().Isis Lovecruft2014-01-12
| | | | | | | | | | | | See https://travis-ci.org/isislovecruft/bridgedb/jobs/16798991#L1591, it was erroring because it expected the default hashring name for every hashring which was prepopulated.
* | Add class docstrings for bridgedb.Bridges.BridgeRing.Isis Lovecruft2014-01-12
| |
* | Add docstring for Bridges.PluggableTransport.getTransportLine().Isis Lovecruft2014-01-11
| |
* | Add generic Dist.Distributor class for common functionality.Isis Lovecruft2014-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | This class currently serves very little purpose because I am still nervous about making major changes to the IPBasedDistributor and EmailBasedDistributor classes. However, I did add functionality for naming a Distributor's hashrings after the distributor's class, so that which hashring a bridge is being added to is much clearer in the logs. * ADD bridgedb.Dist.Distributor class. * CHANGE IPBasedDistributor and EmailBasedDistributor to inherit from Distributor. * ADD functionality for naming hashrings after their distributor.
* | Clarify log messages in EmailBasedDistributor.getBridgesForEmail().Isis Lovecruft2014-01-11
| |
* | PEP8 fixes in Dist.EmailBasedDistributor.Isis Lovecruft2014-01-11
| | | | | | | | | | * FIXES several lines which are too long. * FIXES a couple whitespace problems.
* | PEP8; remove unnecessary `pass` statements in Exception classes.Isis Lovecruft2014-01-11
| |
* | Clarify log messages in IPBasedDistributor.getBridgesForIP().Isis Lovecruft2014-01-11
| |
* | Several PEP8 fixes in Dist.IPBasedDistributor.Isis Lovecruft2014-01-11
| | | | | | | | | | | | | | | | | | * FIXES several PEP8 problems due to lines being too long. * FIXES one cute PEP8 problem; a semicolon which was placed after a `break` in a `for` loop, probably by someone whose fingers are used to coding in C. * FIXES one PEP8 problem due to continuation of an `if` clause on the same line.
* | PEP8 fixes for several lines in Dist.getNumBridgesPerAnswer().Isis Lovecruft2014-01-11
| | | | | | | | | | | | Several `if` clauses are split to the next line, and an unneeded line-continued backslash is removed from the parameters for a call to the logger.
* | Numerous whitespace fixes in bridgedb.Dist module.Isis Lovecruft2014-01-11
| | | | | | | | These are all whitespace changes ― nothing else is touched.
* | Sphinxify and improve Dist.IPBasedDistributor.getBridgesForIP docstring.Isis Lovecruft2014-01-11
| |
* | Add docstrings for HTTPServer.render() and getBridgeRequestAnswer().Isis Lovecruft2014-01-05
| |
* | Cleanup logging of memory addresses in prepopulateRings().Isis Lovecruft2014-01-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The logging before was quite messy. The sub hashrings are identified by the HMAC of a unique string. It turns out that the "unique string" which is used in this case is actually something which looks like the class.__repr__() for a bridgedb.Bridges.BridgeHolder subclass. This is *probably* unique because it contains memory addresses of the newly generated hmac_fn()s, which are likely (though not necessarily) unique. This is a bit gross, in my opinion, but it's not really a problem. However, the log messages this function generated are a problem. They contain memory addresses, which isn't the greatest idea, ignoring all other reasons, simply because it's illegible. * REFACTOR logging statements in bridgedb.Bridges.FilteredBridgeSplitter.prepopulateRings() to remove logging of hmac_fn() memory addresses. * ADD a long comment detailing the "unique strings" which identify a sub hashring, and how they should probably be changed someday.
* | Update docstring for bridgedb.Bridges.FilteredBridgeSplitter.prepopulateRings().Isis Lovecruft2014-01-05
| |
* | Return True if we were able to prepopulate sub hashrings.Isis Lovecruft2014-01-05
| | | | | | | | | | | | | | | | In bridgedb.Bridges.FilteredBridgeSplitter.prepopulateRings(), we return False if we were unable to prepopulate bridges into the sub hashring for whatever reason. In case some calling function wants to use the boolean return value as a check, we should mirror this logic and return True if prepopulation was successful.