SR: Control event to get the shared random values

Once all dirauth move to 029, the consensus will have a shared random value regurlarly. Having a way to get that value through the control port would be useful for many purposes including people wanting to use that value outside of tor.

I recommend we implement that only in 030 so we'll have a full stable version to debug and make sure it actually works properly before we expose it to a wider audience.

This ticket is the main one for both the spec and implementation.

changed milestone to %Tor: 0.3.0.x-final in legacy/trac

added component::core tor/tor in Legacy / Trac controller in Legacy / Trac milestone::Tor: 0.3.0.x-final in Legacy / Trac owner::dgoulet in Legacy / Trac points::3 in Legacy / Trac priority::medium in Legacy / Trac resolution::implemented in Legacy / Trac severity::normal in Legacy / Trac sponsor::R-must in Legacy / Trac status::closed in Legacy / Trac tor-spec in Legacy / Trac tor-sr in Legacy / Trac type::enhancement in Legacy / Trac labels

Same ticket as legacy/trac#17436 (moved)?

Trac:
Status: new to accepted
Summary: sr: Control event to get the shared random values to SR: Control event to get the shared random values
Keywords: spec, dirauth, control, sr deleted, tor-sr, controller, tor-spec added
Owner: N/A to dgoulet

Took a stab at this. I went dead simple here, the main purpose is to expose the value straight away thus I didn't put some key/values for the number of participants or algorithm or version.

Spec: ticket19925_01 Code: ticket19925_030_01

On the control port:

GETINFO sr/current
250-sr/current=uvk8oUuHKnynlJ1CnkzLxHcICy6wxtsTnOPh5uGoqgY=
250 OK
GETINFO sr/previous
250-sr/previous=rU+4HJZGBi07ei4TvaXbExvZvxmnoTz00Xa2i9CcFt4=
250 OK

Trac:
Status: accepted to needs_review

Hello, thanks for the code.

The basic idea looks reasonable. Why not also expose the number of participants and protocol version? If these are important things to put in the consensus, we should also expose them to users of control port.

Also, I suggest we move this to 0.3.1. I don't think we should put extra code/design/review work for 0.3.0.

Replying to asn:

Hello, thanks for the code.

The basic idea looks reasonable. Why not also expose the number of participants and protocol version? If these are important things to put in the consensus, we should also expose them to users of control port.

We can and we should at some point. Your point below is why I want dead simple at first. The main use case for this will be for third part app. to use the SR and get it easily instead of grep parsing the consensus.

I can see something like this as future addition. Actually, it would be trivial to add this:

GETINFO sr/current/num-reveals
250-sr/current/num-reveals=8
250 OK

Also, I suggest we move this to 0.3.1. I don't think we should put extra code/design/review work for 0.3.0.

My counter argument here is that SR went in 029 and 030 is the version you can get access to it through the control port. Putting that in 031 means another 8 months... This isn't a big change to the spec and code, I think we can pull it off? Feature freeze for 030 is in 21 days so we still have time in theory. :)

Merged, with a changes file. I'm okay letting this into 030 since it's not going to break anything else.

Trac:
Status: needs_review to closed
Resolution: N/A to implemented

closed

changed time estimate to 24h

moved from legacy/trac#19925 (moved)

added Feature + 1 deleted label and removed 1 deleted label

added Control API Specifications labels and removed Controller + 1 deleted label