[metrics-bugs] #21427 [Metrics/Onionoo]: allow to filter for tor_version

Tor Bug Tracker & Wiki blackhole at torproject.org
Mon Aug 28 15:02:30 UTC 2017


#21427: allow to filter for tor_version
-----------------------------+-------------------------------
 Reporter:  cypherpunks      |          Owner:  metrics-team
     Type:  enhancement      |         Status:  merge_ready
 Priority:  Medium           |      Milestone:  Onionoo-1.4.0
Component:  Metrics/Onionoo  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:                   |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:                   |        Sponsor:
-----------------------------+-------------------------------

Comment (by iwakeh):

 Replying to [comment:5 karsten]:
 > Replying to [comment:3 iwakeh]:
 > > All checks and tests pass now with the two small commits on
 [https://gitweb.torproject.org/user/iwakeh/onionoo.git/log/?h=task-21427
 this branch].
 >
 > Thanks for the review and those two additional commits.
 >
 > > Could it be confusing to use use 'version' as search key name, because
 actually the 'platform' field of the details document is used and there
 are fields that contain the word 'version'?
 >
 > Wait, we're only looking at the last part of the "platform" line:

 True.  Maybe, use "platform_version", which is kind of long?

 >
 > ...
 >
 > And which other fields are there that contain the word "version" which
 might confuse users?

 In details we have "recommended_version".  Maybe, rather keep 'version'
 (b/c it is short) and document it well.

 >
 > Let's eliminate any confusions before merging. (But we'll also have to
 be careful not to call the parameter "platform" if we don't look at the
 entire platform string.)
 >
 > > Other than that: merge ready.
 >
 > One thing, though: Should we briefly talk about possible extensions to
 support multiple versions or version ranges? How about
 `"version=0.3.0,0.3.1"` (all 0.3.0 and 0.3.1 versions),
 `"version=0.2.4.21-0.2.4.29"` (everything between 0.2.4.21 and 0.2.4.29),
 `"version=0.2.9-"` (0.2.9 or higher), `"version=-0.2.9999`" (everything up
 to 0.2.9999), etc.?
 >
 > (Note that in particular the ranges might lead to surprises when used in
 two-digit version numbers; for example, `"version=0.2.4.2-0.2.4.3`"
 matches 0.2.4.2, 0.2.4.3, but also 0.2.4.21. I'm not sure whether we can
 do anything about that without getting into the business of parsing
 version strings.)

 I would wait and not implement this, as there are not enough sub-versions
 that would justify such an effort.  And, we offer a range search, because
 `search=version:0.2` returns all versions 'in between'/'below', i.e.,
 `0.2`, `0.2.1.1`, `0.2.2.1`, etc.

 >
 > > I'll add a ticket for documentation update.
 >
 > (Thanks!)

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21427#comment:6>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the metrics-bugs mailing list