Age | Commit message (Collapse) | Author |
|
|
|
|
|
|
|
Keyword arguments tend to always be optional, but Karsten made BucketName
mandatory. Conventionally mandatory args are positional so swapping to that.
|
|
I'm still not really sure why Karsten wants to use a string field for this.
This has all the hallmarks of being an enumeration and it's trivially easy to
add new connection types as they arise.
Using a string for this field is not just unusual, but makes things a bit
harder for both controller users and anyone trying to build an alternative tor
implementation.
I'm also swapping the enums to be uppercase since that's the present convention
in the control-spec.
|
|
CircuitID is defined in section 2.4, but 'CircID' isn't. Using that general
definition in proposal 218 and also fixing a couple spots in the control-spec.
|
|
As mentioned on the tor-dev@ thread the ConnID definition should go in section
2.4 so we don't need to define it multiple times.
|
|
|
|
usage
|