Update comment

psiphon/serverApi.go

@@ -602,28 +602,31 @@ func confirmStatusRequestPayload(payloadInfo *statusRequestPayloadInfo) {
 	}
 }
 
-// RecordRemoteServerListStat records a completed common or OSL
-// remote server list resource download.
+// RecordRemoteServerListStat records a completed common or OSL remote server
+// list resource download.
 //
-// The RSL download event could occur when the client is unable
-// to immediately send a status request to a server, so these
-// records are stored in the persistent datastore and reported
-// via subsequent status requests sent to any Psiphon server.
+// The RSL download event could occur when the client is unable to immediately
+// send a status request to a server, so these records are stored in the
+// persistent datastore and reported via subsequent status requests sent to
+// any Psiphon server.
 //
-// Note that some common event field values may change between the
-// stat recording and reporting, including client geolocation and
-// host_id.
+// Note that some common event field values may change between the stat
+// recording and reporting, including client geolocation and host_id.
 //
-// Multiple "status" requests may be in flight at once (due
-// to multi-tunnel, asynchronous final status retry, and
-// aggressive status requests for pre-registered tunnels),
-// To avoid duplicate reporting, persistent stats records are
-// "taken-out" by a status request and then "put back" in
-// case the request fails.
+// The bytes/duration fields reflect the size and download time for the _last
+// chunk only_ in the case of a resumed download. The purpose of these fields
+// is to calculate rough data transfer rates. Both bytes and duration are
+// included in the log, to allow for filtering out of small transfers which
+// may not produce accurate rate numbers.
 //
-// Duplicate reporting may also occur when a server receives and
-// processes a status request but the client fails to receive
-// the response.
+// Multiple "status" requests may be in flight at once (due to multi-tunnel,
+// asynchronous final status retry, and aggressive status requests for
+// pre-registered tunnels), To avoid duplicate reporting, persistent stats
+// records are "taken-out" by a status request and then "put back" in case the
+// request fails.
+//
+// Duplicate reporting may also occur when a server receives and processes a
+// status request but the client fails to receive the response.
 func RecordRemoteServerListStat(
 	config *Config,
 	tunneled bool,