Add the operating system to the bandwidth file headers

See #30196 (moved).

changed milestone to %sbws: 1.2.x-final

Trac:
Parent Ticket: #30255 (moved)

added component::core tor/sbws milestone::sbws: 1.2.x-final parent::30255 points::1 priority::medium severity::normal status::needs-revision type::enhancement labels

Trac:
Parent: N/A to #30255 (moved)

Copied from #30196 (moved):

teor:

irl:

For exit lists we are considering using a free-form text field with some suggested formats, e.g. something like "ExitScanner 55.5 using Tor 1.0.0 on Windows 10". If there is later something that affects the specific implementation we can adjust that string without a spec change to report library details.

Unstructured text is a nightmare to parse. Just look at the browser user-agent. Or the Tor ContactInfo field.

If we need something, let's specify it as a separate header. If we're not sure, then let's leave it out.

juga:

teor:

• we have asked authority operators about operating systems before
  • but I have never wanted to know kernel versions

What in concrete we would like to know about the operating system?, see options in https://docs.python.org/3/library/platform.html

Let's start with: platform.platform(aliased=True, terse=True) https://docs.python.org/3/library/platform.html#platform.platform

Then, if we need more info, we can add it later.

Replying to juga:

Reminder to myself: i've implemented the operating system, ssl_version and tor_version in branch https://github.com/juga0/sbws/tree/ticket30196

Here's my review of operating system:

• "system" is a confusing name, please use "operating_system"
• platform.platform(aliased=True, terse=True) uses platform.system_alias() to produce names that are better known than platform.system()
• you added tor_version in two places, but system in one place. Please refactor EXTRA_ARG_KEYVALUES and BANDWIDTH_HEADER_KEY_VALUES_INIT so the headers are consistent.
• please submit a bandwidth-file-spec patch, so the reviewer knows what your code should do
• does the "chg: v3bwfile: !refactor, obtain headers from state" commit belong in a separate pull request? It's quite large.
• please submit each feature in a separate pull request

Trac:
Status: new to needs_revision

changed time estimate to 8h

mentioned in issue #30255 (moved)

moved to tpo/network-health/sbws#30251 (moved)