ooni / probe

OONI Probe network measurement tool for detecting internet censorship
https://ooni.org/install
BSD 3-Clause "New" or "Revised" License
758 stars 142 forks source link

Not uploaded icon missing from result list view #1097

Open hellais opened 4 years ago

hellais commented 4 years ago

The first result set has not been uploaded yet in the result view we don't see the missing upload icon: Screenshot 2020-04-14 at 11 03 18

We do see it, though, when we enter the measurement itself: Screenshot 2020-04-14 at 11 03 48

It's unclear if this needs to be fixed in probe-desktop, probe-cli or both.

hellais commented 3 years ago

This is missing inside of probe-cli

sarathms commented 3 years ago

Is this now implemented in probe-cli?

hellais commented 3 years ago

Unfortunately not yet

hellais commented 3 years ago

@sarathms I checked and it was actually already present in probe-cli. Reassigning this to you.

sarathms commented 3 years ago

@hellais This is one of the items in the output from ooniprobe list --batch

{
  "fields": {
    [redacted],
    "data_usage_down": 21.9140625,
    "data_usage_up": 5.5576171875,
    "id": 21,
    "index": 15,
    "is_done": true,
    "measurement_anomaly_count": 0,
    "measurement_count": 3,
    "name": "im",
    [redacted]
    "runtime": 0,
    "start_time": "2021-03-25T16:13:27.588048991Z",
    "test_keys": "{}",
    "total_count": 16,
    "type": "result_item"
  },
  "level": "info",
  "timestamp": "2021-03-25T15:42:00.200596637-04:00",
  "message": "result item"
}

It doesn't indicate whether the measurements were uploaded or not. This particular result has measurements which are not uploaded as shown in ooniprobe list 21 --batch

  "fields": {
    [redacted],
    "failure_msg": "",
    "id": 42,
    "is_anomaly": false,
    "is_done": true,
    "is_failed": false,
    "is_first": true,
    "is_last": false,
    "is_upload_failed": false,
    "is_uploaded": false,
    [redacted],
    "report_file_path": "",
    "runtime": 0.303577437,
    "start_time": "2021-03-25T16:13:27.602353155Z",
    "test_group_name": "im",
    "test_keys": "{\"facebook_dns_blocking\":false,\"facebook_tcp_blocking\":false}",
    "test_name": "facebook_messenger",
    "type": "measurement_item",
    "upload_failure_msg": "",
    "url": "",
    "url_category_code": "",
    "url_country_code": ""
  },
hellais commented 3 years ago

Yes you are right, this is not currently present inside of probe-cli. We need to add support for it in probe-cli before we can implement it inside of the desktop app.

I documented that this needs to be done as part of: https://github.com/ooni/probe/issues/1457

hellais commented 3 years ago

This is now implemented in probe-cli