[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #33428 [Core Tor/Chutney]: Make chutney check for relay microdescriptors before verifying



#33428: Make chutney check for relay microdescriptors before verifying
-------------------------------------------+-------------------------------
 Reporter:  teor                           |          Owner:  (none)
     Type:  enhancement                    |         Status:  new
 Priority:  Medium                         |      Milestone:
Component:  Core Tor/Chutney               |        Version:
 Severity:  Normal                         |     Resolution:
 Keywords:  ipv6, prop311, outreachy-ipv6  |  Actual Points:
Parent ID:  #33050                         |         Points:  1
 Reviewer:                                 |        Sponsor:  Sponsor55-can
-------------------------------------------+-------------------------------

Comment (by teor):

 Replying to [comment:13 anuradha1904]:
 > Replying to [comment:6 teor]: Hey teor, all thanks to you, I am done
 with majority of the task of the function, can you please guide me "set
 ed25519-id in the Environ to that value" step. Thank you.

 Hi, I'd like to look at the code you've written so far. Lots of developers
 get reviews on incomplete code, particularly when they are working on a
 new project.

 Please submit a pull request (PR) on the chutney GitHub, containing the
 code you have written so far:
 https://github.com/torproject/chutney

 When you submit your PR, GitHub will run chutney's continuous integration
 (CI). Let us know if CI fails, and we will help you fix it.

 (Sometimes CI fails because of the server or other programs, not your
 code.)

 > > Replying to [comment:5 anuradha1904]:
 > > You can start by writing a function called _setEd25519Id(), which
 should be called at the end of _genRouterKey():
 > >
 https://github.com/torproject/chutney/blob/master/lib/chutney/TorNet.py#L741
 > >
 > > The function should:
 > > * read keys/ed25519_master_id_public_key from datadir, if that file
 exists
 > > * extract the key data
 > > * convert it to base64
 > > * discard the trailing "="

 Your PR should do these things.

 > > * set `ed25519-id` in the Environ to that value

 You can set ed25519-id in a similar way to the code in _genRouterKey(),
 which sets fingerprint:
 {{{
 self._env['fingerprint'] = fingerprint
 }}}

 https://github.com/torproject/chutney/blob/9ffa2857fb4c6e75156159233263ac5898fc410c/lib/chutney/TorNet.py#L744

 > > You can check you have the right value, by printing it out, then
 searching for it in the cached-microdescs and cached-microdescs.new files
 in the datadir.

 Did you do this check?

 > > After you've written _setEd25519Id(), we can move on to the rest of
 this task.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/33428#comment:14>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs