I noticed this when re-reading the spec. Not a big deal because backends are not using that. Still, for complete correctness we should possibly see to recompute this field.
I discussed the matter with @hellais and it seems the correct fix is https://github.com/ooni/explorer/issues/900 and we're going to close this issue as wontfix once that has been deployed.
I noticed this when re-reading the spec. Not a big deal because backends are not using that. Still, for complete correctness we should possibly see to recompute this field.
Explorer uses this: https://github.com/ooni/explorer/blob/3b542cfac527f26bf55167e28fad89816c89fbe0/components/measurement/nettests/WebConnectivity.js#L492
I discussed the matter with @hellais and it seems the correct fix is https://github.com/ooni/explorer/issues/900 and we're going to close this issue as wontfix once that has been deployed.