#25241: effective_family sometimes contains the relay's own fingerprint
-----------------------------+------------------------------
Reporter: irl | Owner: karsten
Type: defect | Status: needs_review
Priority: Medium | Milestone:
Component: Metrics/Onionoo | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: iwakeh | Sponsor:
-----------------------------+------------------------------
Comment (by karsten):
Replying to [comment:14 iwakeh]:
> Open steps:
>
> * decide for adding 'self' always to the effective family, because
clients (like RS) interpret it like that (and had to work around the
inconsistent situation so far) or against it because it would add too much
to current documents
I'm fine with this. From comments above it sounds like there are no
concerns. Let's do it.
> * explicitly state whether the relay belongs to its effective family or
not in the protocol (if not it should also explicitly be added that the
'self' doesn't show up in the other two family types either)
> * make & merge the appropriate changes (everywhere)
Do you want to do this? I just reviewed
[https://gitweb.torproject.org/user/iwakeh/onionoo.git/commit/?h=task-25241&id=df11f4de038c31f33c3f6e231854887995271201
your task-25241 branch until commit df11f4d]. Looks good! Do you want to
change it to always add a relays own fingerprint to its declared family?
And I guess we'll squash those commits in the end, to avoid making a
change and undoing it right afterwards.
If you're already juggling too many tickets, I can also take this one. Let
me know.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/25241#comment:15>
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